Dear All,
For people who do not know, O2 is the browser David has implemented in the last few years. It replaces class-categories on the left-hand side pane with a list of packages. We are still away from having packages as real first class entities, but this is a good step toward. Note that some people find O2 a bit slower than other available code browsers. David and I spent some time on fixing O2. O2 has now no dependencies to OB anymore. We would like that people try it and submit bug reports on the google bug tracker. In a fresh 10505, you can load O2 by evaluating: Gofer new squeaksource: 'MetacelloRepository'; package: 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform: #loadDefault O2 does not need OB whatsoever. O2 loads in a Core (I tried in a 11165). Before and after loading O2, running OB tests give "548 run, 547 passes, 1 failures" [*]. Cheers, Alexandre [*] the test have been loaded using (ConfigurationOfOmniBrowser project version: '1.1') load: 'Dev Tests' -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
Thanks this is good.
And yes I did not make any progress on the package rewrite. Doing too much pharo cleaning and project writing. Stef On Jan 19, 2010, at 11:03 PM, Alexandre Bergel wrote: > Dear All, > > For people who do not know, O2 is the browser David has implemented in > the last few years. It replaces class-categories on the left-hand side > pane with a list of packages. We are still away from having packages > as real first class entities, but this is a good step toward. Note > that some people find O2 a bit slower than other available code > browsers. > > David and I spent some time on fixing O2. O2 has now no dependencies > to OB anymore. > We would like that people try it and submit bug reports on the google > bug tracker. In a fresh 10505, you can load O2 by evaluating: > > Gofer new squeaksource: 'MetacelloRepository'; package: > 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform: > #loadDefault > > O2 does not need OB whatsoever. O2 loads in a Core (I tried in a > 11165). Before and after loading O2, running OB tests give "548 run, > 547 passes, 1 failures" [*]. > > Cheers, > Alexandre > > [*] the test have been loaded using (ConfigurationOfOmniBrowser > project version: '1.1') load: 'Dev Tests' > -- > _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: > Alexandre Bergel http://www.bergel.eu > ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. > > > > > > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
In reply to this post by Alexandre Bergel-4
On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel <[hidden email]> wrote: Dear All, Yes, but be aware that that piece of code load only the core, as you can see here: group: 'default' with: #('Core' ); group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-Morphic' 'O2-Enhancements' ); group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' ); As I remember, most of the problems of O2 were sometimes related to the integration to RB. So, if you really want to tests it, please, test it using the RB: ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev' As I already said to Alexandre and David in private, I will put O2 again in the next image. So, it would be cool if it is tested "completely". O2 does not need OB whatsoever. O2 loads in a Core (I tried in a Yes, if someone can help with that failure would be cool, I have already reported it. Cheers, _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
Thanks, Alex
I gave it a quick try. Installation into PharoCore 10505 worked and briefly clicking through the browser I haven't found any obvious problems. However, there are 6 undeclared classes referenced in the code. Very likely when this code is run, O2 will fall over. Also 2 tests have errors and one fails. HTH, Adrian On Jan 19, 2010, at 23:11 , Mariano Martinez Peck wrote: > On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel < > [hidden email]> wrote: > >> Dear All, >> >> For people who do not know, O2 is the browser David has implemented in >> the last few years. It replaces class-categories on the left-hand side >> pane with a list of packages. We are still away from having packages >> as real first class entities, but this is a good step toward. Note >> that some people find O2 a bit slower than other available code >> browsers. >> >> David and I spent some time on fixing O2. O2 has now no dependencies >> to OB anymore. >> We would like that people try it and submit bug reports on the google >> bug tracker. In a fresh 10505, you can load O2 by evaluating: >> >> Gofer new squeaksource: 'MetacelloRepository'; package: >> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform: >> #loadDefault >> >> > Yes, but be aware that that piece of code load only the core, as you can see > here: > > group: 'default' with: #('Core' ); > group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-Morphic' > 'O2-Enhancements' ); > group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' ); > > > As I remember, most of the problems of O2 were sometimes related to the > integration to RB. So, if you really want to tests it, please, test it using > the RB: > > ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev' > > > As I already said to Alexandre and David in private, I will put O2 again in > the next image. So, it would be cool if it is tested "completely". > > > >> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a >> 11165). Before and after loading O2, running OB tests give "548 run, >> 547 passes, 1 failures" [*]. >> >> > Yes, if someone can help with that failure would be cool, I have already > reported it. > > > >> Cheers, >> Alexandre >> >> [*] the test have been loaded using (ConfigurationOfOmniBrowser >> project version: '1.1') load: 'Dev Tests' >> -- >> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >> Alexandre Bergel http://www.bergel.eu >> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >> >> >> >> >> >> >> _______________________________________________ >> Pharo-project mailing list >> [hidden email] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
In reply to this post by Alexandre Bergel-4
Is it virtually possible to run it in Squeak image, since its not
depends on OB anymore? 2010/1/20 Alexandre Bergel <[hidden email]>: > Dear All, > > For people who do not know, O2 is the browser David has implemented in > the last few years. It replaces class-categories on the left-hand side > pane with a list of packages. We are still away from having packages > as real first class entities, but this is a good step toward. Note > that some people find O2 a bit slower than other available code > browsers. > > David and I spent some time on fixing O2. O2 has now no dependencies > to OB anymore. > We would like that people try it and submit bug reports on the google > bug tracker. In a fresh 10505, you can load O2 by evaluating: > > Gofer new squeaksource: 'MetacelloRepository'; package: > 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform: > #loadDefault > > O2 does not need OB whatsoever. O2 loads in a Core (I tried in a > 11165). Before and after loading O2, running OB tests give "548 run, > 547 passes, 1 failures" [*]. > > Cheers, > Alexandre > > [*] the test have been loaded using (ConfigurationOfOmniBrowser > project version: '1.1') load: 'Dev Tests' > -- > _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: > Alexandre Bergel http://www.bergel.eu > ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. > > > > > > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > -- Best regards, Igor Stasenko AKA sig. _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
In reply to this post by Mariano Martinez Peck
Hi:
Browser loaded like follows: Gofer new squeaksource: 'MetacelloRepository'; package: 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform: #loadDefault. ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev' In general, it works stable for me. Have used it the whole afternoon. Some small points I've found: - selecting the 'all' protocol does not allow testing with cmd+t (and context menu) it reports that 0 tests have been executed - the icons should have a transparent background, does not look nice when an item is selected - code completion popups are not removed correctly, but I think that is not specific to O2 Reproduce: 1. type somewhere (code pane or search bar) to bring up completion 2. then move the whole browser window That is only one of the few cases where such a popup is not removed. Best Stefan On 19 Jan 2010, at 23:11, Mariano Martinez Peck wrote: > > > On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel <[hidden email]> wrote: > Dear All, > > For people who do not know, O2 is the browser David has implemented in > the last few years. It replaces class-categories on the left-hand side > pane with a list of packages. We are still away from having packages > as real first class entities, but this is a good step toward. Note > that some people find O2 a bit slower than other available code > browsers. > > David and I spent some time on fixing O2. O2 has now no dependencies > to OB anymore. > We would like that people try it and submit bug reports on the google > bug tracker. In a fresh 10505, you can load O2 by evaluating: > > Gofer new squeaksource: 'MetacelloRepository'; package: > 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform: > #loadDefault > > > Yes, but be aware that that piece of code load only the core, as you can see here: > > group: 'default' with: #('Core' ); > group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-Morphic' 'O2-Enhancements' ); > group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' ); > > > As I remember, most of the problems of O2 were sometimes related to the integration to RB. So, if you really want to tests it, please, test it using the RB: > > ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev' > > > As I already said to Alexandre and David in private, I will put O2 again in the next image. So, it would be cool if it is tested "completely". > > > O2 does not need OB whatsoever. O2 loads in a Core (I tried in a > 11165). Before and after loading O2, running OB tests give "548 run, > 547 passes, 1 failures" [*]. > > > Yes, if someone can help with that failure would be cool, I have already reported it. > > > Cheers, > Alexandre > > [*] the test have been loaded using (ConfigurationOfOmniBrowser > project version: '1.1') load: 'Dev Tests' > -- > _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: > Alexandre Bergel http://www.bergel.eu > ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. > > > > > > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project -- Stefan Marr Software Languages Lab Vrije Universiteit Brussel Pleinlaan 2 / B-1050 Brussels / Belgium http://soft.vub.ac.be/~smarr Phone: +32 2 629 3956 Fax: +32 2 629 3525 _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
you are a good tester :) I checked and it also happens with OB. Can you open a bug ticket please? If you identify other reproducible situations, please write them too Thanks!
_______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
In reply to this post by Adrian Lienhard
> However, there are 6 undeclared classes referenced in the code. Very
> likely when this code is run, O2 will fall over. Also 2 tests have > errors and one fails. That cannot be left like that. I will seat down with David asap... Alexandre > > > On Jan 19, 2010, at 23:11 , Mariano Martinez Peck wrote: > >> On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel < >> [hidden email]> wrote: >> >>> Dear All, >>> >>> For people who do not know, O2 is the browser David has >>> implemented in >>> the last few years. It replaces class-categories on the left-hand >>> side >>> pane with a list of packages. We are still away from having packages >>> as real first class entities, but this is a good step toward. Note >>> that some people find O2 a bit slower than other available code >>> browsers. >>> >>> David and I spent some time on fixing O2. O2 has now no dependencies >>> to OB anymore. >>> We would like that people try it and submit bug reports on the >>> bug tracker. In a fresh 10505, you can load O2 by evaluating: >>> >>> Gofer new squeaksource: 'MetacelloRepository'; package: >>> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) >>> perform: >>> #loadDefault >>> >>> >> Yes, but be aware that that piece of code load only the core, as >> you can see >> here: >> >> group: 'default' with: #('Core' ); >> group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2- >> Morphic' >> 'O2-Enhancements' ); >> group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' ); >> >> >> As I remember, most of the problems of O2 were sometimes related to >> the >> integration to RB. So, if you really want to tests it, please, test >> it using >> the RB: >> >> ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: >> 'Dev' >> >> >> As I already said to Alexandre and David in private, I will put O2 >> again in >> the next image. So, it would be cool if it is tested "completely". >> >> >> >>> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a >>> 11165). Before and after loading O2, running OB tests give "548 run, >>> 547 passes, 1 failures" [*]. >>> >>> >> Yes, if someone can help with that failure would be cool, I have >> already >> reported it. >> >> >> >>> Cheers, >>> Alexandre >>> >>> [*] the test have been loaded using (ConfigurationOfOmniBrowser >>> project version: '1.1') load: 'Dev Tests' >>> -- >>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >>> Alexandre Bergel http://www.bergel.eu >>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >>> >>> >>> >>> >>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [hidden email] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> _______________________________________________ >> Pharo-project mailing list >> [hidden email] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
In reply to this post by Igor Stasenko
On 20.01.2010 15:59, Igor Stasenko wrote:
> Is it virtually possible to run it in Squeak image, since its not > depends on OB anymore? > Not quite. 1. In MCClassDefinition>> loadClass, isEmpty is sent to traitComposition, since O2 includes classes with trait definitions as a single trait, it'd have to be rewritten to: composition := self traitComposition ifNotNil:[: trait | Compiler evaluate: trait]. (composition isNil and:[class traitComposition isEmpty]) ifFalse:[ class setTraitComposition: composition asTraitComposition. ]. in order to load them correctly. 2. In ConfigurationOfO2, the baseline10 would have to change spec for: #pharo into spec for: #common. After those two changes, it still won't work, as O2 relies on one of the classes originally included (but really unrelated to) in the Traits category, and this has been removed from Squeak. If you make an external package with that, and modify the baseline to include a spec for: #squeak which imports said package, it might. Cheers, Henry _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
In reply to this post by Alexandre Bergel-4
On 19.01.2010 23:03, Alexandre Bergel wrote:
> Dear All, > > For people who do not know, O2 is the browser David has implemented in > the last few years. It replaces class-categories on the left-hand side > pane with a list of packages. We are still away from having packages > as real first class entities, but this is a good step toward. Note > that some people find O2 a bit slower than other available code > browsers. > > David and I spent some time on fixing O2. O2 has now no dependencies > to OB anymore. > We would like that people try it and submit bug reports on the google > bug tracker. In a fresh 10505, you can load O2 by evaluating: > > Gofer new squeaksource: 'MetacelloRepository'; package: > 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform: > #loadDefault > > O2 does not need OB whatsoever. O2 loads in a Core (I tried in a > 11165). Before and after loading O2, running OB tests give "548 run, > 547 passes, 1 failures" [*]. > > Cheers, > Alexandre > > [*] the test have been loaded using (ConfigurationOfOmniBrowser > project version: '1.1') load: 'Dev Tests' > OR2PackageBrowser does not open, it lacks a defaultRootNode method returning other than nil. Is the focus mainly on the O2* browsers, or should OR2* also work? Cheers, Henry _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
Well spotted. David?
Alexandre On 20 Jan 2010, at 18:44, Henrik Sperre Johansen wrote: > On 19.01.2010 23:03, Alexandre Bergel wrote: >> Dear All, >> >> For people who do not know, O2 is the browser David has implemented >> in >> the last few years. It replaces class-categories on the left-hand >> side >> pane with a list of packages. We are still away from having packages >> as real first class entities, but this is a good step toward. Note >> that some people find O2 a bit slower than other available code >> browsers. >> >> David and I spent some time on fixing O2. O2 has now no dependencies >> to OB anymore. >> We would like that people try it and submit bug reports on the google >> bug tracker. In a fresh 10505, you can load O2 by evaluating: >> >> Gofer new squeaksource: 'MetacelloRepository'; package: >> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) >> perform: >> #loadDefault >> >> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a >> 11165). Before and after loading O2, running OB tests give "548 run, >> 547 passes, 1 failures" [*]. >> >> Cheers, >> Alexandre >> >> [*] the test have been loaded using (ConfigurationOfOmniBrowser >> project version: '1.1') load: 'Dev Tests' >> > > OR2PackageBrowser does not open, it lacks a defaultRootNode method > returning other than nil. > Is the focus mainly on the O2* browsers, or should OR2* also work? > > Cheers, > Henry > > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
In reply to this post by Henrik Sperre Johansen
> > OR2PackageBrowser does not open, it lacks a defaultRootNode method > returning other than nil. > Is the focus mainly on the O2* browsers, or should OR2* also work? How did you spot this problem? From within the O2 browser, OR2PackageBrowser works. OR2PackageBrowser is not meant to be used independently from the O2 browser. Something like 'OR2PackageBrowser open' is not supposed to work, as the OR2PackageBrowser gets the root node from the O2 browser, so there is no reasonable default root node. Or do you see other problems when using OR2PackageBrowser from within O2? David _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
On Jan 21, 2010, at 2:30 18PM, David Röthlisberger wrote: > >> >> OR2PackageBrowser does not open, it lacks a defaultRootNode method >> returning other than nil. >> Is the focus mainly on the O2* browsers, or should OR2* also work? > > How did you spot this problem? From within the O2 browser, OR2PackageBrowser works. > OR2PackageBrowser is not meant to be used independently from the O2 browser. > Something like 'OR2PackageBrowser open' is not supposed to work, as the OR2PackageBrowser > gets the root node from the O2 browser, so there is no reasonable default root node. > > Or do you see other problems when using OR2PackageBrowser from within O2? > > David Not quite sure how/when I use it from within O2, but the refactoring items I've tried in O2PackageBrowser seem to work :) Quick question: How do you open a browser of methods referring an instance variable from the O2PackageBrowser? Cheers, Henry _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
Indeed, nice job. That would remove my reason for sticking to the conventional browser. Just one (no two) tiny UI issues: if you right click a package -> Refactoring scope , -> *package* is greyed out. If you do the same from one of the classes of the package, it works. Am I right in thinking that a method name is italic iff it comes from a trait? Should it also be in a dynamic protocol called traits, or similar, and/or an icon? It shows an icon for override of superclass method, for which there is bubble help text, but there is no comparable way to find out where the trait method comes from. Cheers, ...Stan |
In reply to this post by Adrian Lienhard
Hi Adrian,
> However, there are 6 undeclared classes referenced in the code. ok, I fixed these undeclared references. > Also 2 tests have errors and one fails. Which tests fail? I only got one error in a test which I fixed now. Cheers, David > > On Jan 19, 2010, at 23:11 , Mariano Martinez Peck wrote: > >> On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel < >> [hidden email]> wrote: >> >>> Dear All, >>> >>> For people who do not know, O2 is the browser David has implemented in >>> the last few years. It replaces class-categories on the left-hand side >>> pane with a list of packages. We are still away from having packages >>> as real first class entities, but this is a good step toward. Note >>> that some people find O2 a bit slower than other available code >>> browsers. >>> >>> David and I spent some time on fixing O2. O2 has now no dependencies >>> to OB anymore. >>> We would like that people try it and submit bug reports on the google >>> bug tracker. In a fresh 10505, you can load O2 by evaluating: >>> >>> Gofer new squeaksource: 'MetacelloRepository'; package: >>> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform: >>> #loadDefault >>> >>> >> Yes, but be aware that that piece of code load only the core, as you can see >> here: >> >> group: 'default' with: #('Core' ); >> group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-Morphic' >> 'O2-Enhancements' ); >> group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' ); >> >> >> As I remember, most of the problems of O2 were sometimes related to the >> integration to RB. So, if you really want to tests it, please, test it using >> the RB: >> >> ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev' >> >> >> As I already said to Alexandre and David in private, I will put O2 again in >> the next image. So, it would be cool if it is tested "completely". >> >> >> >>> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a >>> 11165). Before and after loading O2, running OB tests give "548 run, >>> 547 passes, 1 failures" [*]. >>> >>> >> Yes, if someone can help with that failure would be cool, I have already >> reported it. >> >> >> >>> Cheers, >>> Alexandre >>> >>> [*] the test have been loaded using (ConfigurationOfOmniBrowser >>> project version: '1.1') load: 'Dev Tests' >>> -- >>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >>> Alexandre Bergel http://www.bergel.eu >>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >>> >>> >>> >>> >>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [hidden email] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> _______________________________________________ >> Pharo-project mailing list >> [hidden email] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
Hi David,
in my PharoCore-1.0-10505 image I had #testSmartGroupSwitch that failed and #testExtensionNodes and #testStyling that errored. Cheers, Adrian On Jan 21, 2010, at 16:09 , David Röthlisberger wrote: > Hi Adrian, > >> However, there are 6 undeclared classes referenced in the code. > > ok, I fixed these undeclared references. > >> Also 2 tests have errors and one fails. > > Which tests fail? I only got one error in a test which I fixed now. > > Cheers, > David > > >> >> On Jan 19, 2010, at 23:11 , Mariano Martinez Peck wrote: >> >>> On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel < >>> [hidden email]> wrote: >>> >>>> Dear All, >>>> >>>> For people who do not know, O2 is the browser David has implemented in >>>> the last few years. It replaces class-categories on the left-hand side >>>> pane with a list of packages. We are still away from having packages >>>> as real first class entities, but this is a good step toward. Note >>>> that some people find O2 a bit slower than other available code >>>> browsers. >>>> >>>> David and I spent some time on fixing O2. O2 has now no dependencies >>>> to OB anymore. >>>> We would like that people try it and submit bug reports on the google >>>> bug tracker. In a fresh 10505, you can load O2 by evaluating: >>>> >>>> Gofer new squeaksource: 'MetacelloRepository'; package: >>>> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform: >>>> #loadDefault >>>> >>>> >>> Yes, but be aware that that piece of code load only the core, as you can see >>> here: >>> >>> group: 'default' with: #('Core' ); >>> group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-Morphic' >>> 'O2-Enhancements' ); >>> group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' ); >>> >>> >>> As I remember, most of the problems of O2 were sometimes related to the >>> integration to RB. So, if you really want to tests it, please, test it using >>> the RB: >>> >>> ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev' >>> >>> >>> As I already said to Alexandre and David in private, I will put O2 again in >>> the next image. So, it would be cool if it is tested "completely". >>> >>> >>> >>>> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a >>>> 11165). Before and after loading O2, running OB tests give "548 run, >>>> 547 passes, 1 failures" [*]. >>>> >>>> >>> Yes, if someone can help with that failure would be cool, I have already >>> reported it. >>> >>> >>> >>>> Cheers, >>>> Alexandre >>>> >>>> [*] the test have been loaded using (ConfigurationOfOmniBrowser >>>> project version: '1.1') load: 'Dev Tests' >>>> -- >>>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >>>> Alexandre Bergel http://www.bergel.eu >>>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >>>> >>>> >>>> >>>> >>>> >>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> [hidden email] >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [hidden email] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> >> >> _______________________________________________ >> Pharo-project mailing list >> [hidden email] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
> in my PharoCore-1.0-10505 image I had #testSmartGroupSwitch that failed and #testExtensionNodes and #testStyling that errored. ah, then probably I fixed the other test methods by fixing the undeclared references. Thanks, David > On Jan 21, 2010, at 16:09 , David Röthlisberger wrote: > >> Hi Adrian, >> >>> However, there are 6 undeclared classes referenced in the code. >> ok, I fixed these undeclared references. >> >>> Also 2 tests have errors and one fails. >> Which tests fail? I only got one error in a test which I fixed now. >> >> Cheers, >> David >> >> >>> On Jan 19, 2010, at 23:11 , Mariano Martinez Peck wrote: >>> >>>> On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel < >>>> [hidden email]> wrote: >>>> >>>>> Dear All, >>>>> >>>>> For people who do not know, O2 is the browser David has implemented in >>>>> the last few years. It replaces class-categories on the left-hand side >>>>> pane with a list of packages. We are still away from having packages >>>>> as real first class entities, but this is a good step toward. Note >>>>> that some people find O2 a bit slower than other available code >>>>> browsers. >>>>> >>>>> David and I spent some time on fixing O2. O2 has now no dependencies >>>>> to OB anymore. >>>>> We would like that people try it and submit bug reports on the google >>>>> bug tracker. In a fresh 10505, you can load O2 by evaluating: >>>>> >>>>> Gofer new squeaksource: 'MetacelloRepository'; package: >>>>> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform: >>>>> #loadDefault >>>>> >>>>> >>>> Yes, but be aware that that piece of code load only the core, as you can see >>>> here: >>>> >>>> group: 'default' with: #('Core' ); >>>> group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-Morphic' >>>> 'O2-Enhancements' ); >>>> group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' ); >>>> >>>> >>>> As I remember, most of the problems of O2 were sometimes related to the >>>> integration to RB. So, if you really want to tests it, please, test it using >>>> the RB: >>>> >>>> ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev' >>>> >>>> >>>> As I already said to Alexandre and David in private, I will put O2 again in >>>> the next image. So, it would be cool if it is tested "completely". >>>> >>>> >>>> >>>>> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a >>>>> 11165). Before and after loading O2, running OB tests give "548 run, >>>>> 547 passes, 1 failures" [*]. >>>>> >>>>> >>>> Yes, if someone can help with that failure would be cool, I have already >>>> reported it. >>>> >>>> >>>> >>>>> Cheers, >>>>> Alexandre >>>>> >>>>> [*] the test have been loaded using (ConfigurationOfOmniBrowser >>>>> project version: '1.1') load: 'Dev Tests' >>>>> -- >>>>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >>>>> Alexandre Bergel http://www.bergel.eu >>>>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >>>>> >>>>> >>>>> >>>>> >>>>> >>>>> >>>>> _______________________________________________ >>>>> Pharo-project mailing list >>>>> [hidden email] >>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> [hidden email] >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [hidden email] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> >> _______________________________________________ >> Pharo-project mailing list >> [hidden email] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
Free forum by Nabble | Edit this page |