[Pharo-project] FSPlatformResolver class>>forCurrentPlatform use a deprecated method
In the package Filesystem, FSPlatformResolver class >> forCurrentPlatform | platformName | platformName := SmalltalkImage current platformName. ^ (self allSubclasses detect: [:ea | ea platformName = platformName]) new should be replaced by FSPlatformResolver class >> forCurrentPlatform | platformName | platformName := OSPlatform platformName. ^ (self allSubclasses detect: [:ea | ea platformName = platformName]) new
Why? OSPlatform doesn't even exist in Pharo 1.1. Lukas 2010/9/3 Cyrille Delaunay <cy.delaunay@gmail.com>:
In the package Filesystem, FSPlatformResolver class >> forCurrentPlatform | platformName | platformName := Â SmalltalkImage current platformName. ^ (self allSubclasses detect: [:ea | ea platformName = platformName]) new should be replaced by FSPlatformResolver class >> forCurrentPlatform | platformName | platformName := Â OSPlatform platformName. ^ (self allSubclasses detect: [:ea | ea platformName = platformName]) new _______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
-- Lukas Renggli www.lukas-renggli.ch
A better choice might be Smalltalk os platformName, which works on Pharo 1.1, 1.2, Squeak 4.1 and 4.2. (well, as soon as it's properly undeprecated in 1.2 at least, #platform should be removed instead...) Cheers, Henry On Sep 3, 2010, at 2:30 23PM, Lukas Renggli wrote:
Why? OSPlatform doesn't even exist in Pharo 1.1.
Lukas
2010/9/3 Cyrille Delaunay <cy.delaunay@gmail.com>:
In the package Filesystem, FSPlatformResolver class >> forCurrentPlatform | platformName | platformName := SmalltalkImage current platformName. ^ (self allSubclasses detect: [:ea | ea platformName = platformName]) new should be replaced by FSPlatformResolver class >> forCurrentPlatform | platformName | platformName := OSPlatform platformName. ^ (self allSubclasses detect: [:ea | ea platformName = platformName]) new _______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
-- Lukas Renggli www.lukas-renggli.ch
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Yes, I naively repeated what was telling me the 'deprecated' message :) But indeed, use 'Smalltalk platformName' seems to work fine. 2010/9/3 Henrik Johansen <henrik.s.johansen@veloxit.no>
A better choice might be Smalltalk os platformName, which works on Pharo 1.1, 1.2, Squeak 4.1 and 4.2. (well, as soon as it's properly undeprecated in 1.2 at least, #platform should be removed instead...)
Cheers, Henry
On Sep 3, 2010, at 2:30 23PM, Lukas Renggli wrote:
Why? OSPlatform doesn't even exist in Pharo 1.1.
Lukas
2010/9/3 Cyrille Delaunay <cy.delaunay@gmail.com>:
In the package Filesystem, FSPlatformResolver class >> forCurrentPlatform | platformName | platformName := SmalltalkImage current platformName. ^ (self allSubclasses detect: [:ea | ea platformName = platformName]) new should be replaced by FSPlatformResolver class >> forCurrentPlatform | platformName | platformName := OSPlatform platformName. ^ (self allSubclasses detect: [:ea | ea platformName = platformName]) new _______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
-- Lukas Renggli www.lukas-renggli.ch
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
indeed may be we should do that http://code.google.com/p/pharo/issues/detail?id=2907 On Sep 3, 2010, at 2:46 PM, Henrik Johansen wrote:
A better choice might be Smalltalk os platformName, which works on Pharo 1.1, 1.2, Squeak 4.1 and 4.2. (well, as soon as it's properly undeprecated in 1.2 at least, #platform should be removed instead...)
Cheers, Henry
On Sep 3, 2010, at 2:30 23PM, Lukas Renggli wrote:
Why? OSPlatform doesn't even exist in Pharo 1.1.
Lukas
2010/9/3 Cyrille Delaunay <cy.delaunay@gmail.com>:
In the package Filesystem, FSPlatformResolver class >> forCurrentPlatform | platformName | platformName := SmalltalkImage current platformName. ^ (self allSubclasses detect: [:ea | ea platformName = platformName]) new should be replaced by FSPlatformResolver class >> forCurrentPlatform | platformName | platformName := OSPlatform platformName. ^ (self allSubclasses detect: [:ea | ea platformName = platformName]) new _______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
-- Lukas Renggli www.lukas-renggli.ch
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
participants (4)
-
Cyrille Delaunay -
Henrik Johansen -
Lukas Renggli -
Stéphane Ducasse