Issue 6503 in pharo: ScriptLoader deletePackage: does not say what it says

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

Issue 6503 in pharo: ScriptLoader deletePackage: does not say what it says

pharo
Status: Accepted
Owner: marianopeck
CC: [hidden email],  [hidden email],  [hidden email]
Labels: Type-Bug Milestone-2.0

New issue 6503 by marianopeck: ScriptLoader deletePackage: does not say  
what it says
http://code.google.com/p/pharo/issues/detail?id=6503

ScriptLoader >> #deletePackage: does not delete the package....it just  
unregister it. I mean, it does NOT remove the classes of the package. This  
message should be renamed tp unregisterPackage if that is desired.  
Anyway....I have refactored it to really remove packages.

ScriptLoader >> deletePackage: aString

        | toRemove toUnregister |
        toRemove := (MCWorkingCopy allManagers asSortedCollection:
                [ :a :b | a package name <= b package name ]) detect: [:each | each  
package name = aString].
        toRemove unload.
        "With RPackage, each 'sub-package' is considered a separate package so we  
need to also unregister them"
        toUnregister := RPackageOrganizer default packages select: [:each | (each  
name = aString) or:  [each name beginsWith: aString,'-']].
        RPackageOrganizer default unregisterPackageNamed: aString.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 6503 in pharo: ScriptLoader deletePackage: does not say what it says

pharo
Updates:
        Status: HumanReviewNeeded

Comment #1 on issue 6503 by marianopeck: ScriptLoader deletePackage: does  
not say what it says
http://code.google.com/p/pharo/issues/detail?id=6503

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 6503 in pharo: ScriptLoader deletePackage: does not say what it says

pharo
Updates:
        Status: FixToInclude

Comment #2 on issue 6503 by [hidden email]: ScriptLoader  
deletePackage: does not say what it says
http://code.google.com/p/pharo/issues/detail?id=6503

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 6503 in pharo: ScriptLoader deletePackage: does not say what it says

pharo
Updates:
        Status: Integrated

Comment #3 on issue 6503 by [hidden email]: ScriptLoader  
deletePackage: does not say what it says
http://code.google.com/p/pharo/issues/detail?id=6503

in 20250


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker