Issue 6036 in pharo: Let DataStream only for Monticello

Status: Accepted
Owner: marianopeck
Labels: Type-Bug

New issue 6036 by marianopeck: Let DataStream only for Monticello
http://code.google.com/p/pharo/issues/detail?id=6036

So this issue is the discussion:  
http://forum.world.st/Next-step-in-cleaning-DataStream-td4633167.html

Basically, I clean (removed) all unused stuff of DataStream and  
ReferenceStream. I only let in DataStream what is needed for Monticello.



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

Updates:
        Status: FixReviewNeeded
        Cc: [hidden email]
        Labels: -Type-Bug Type-Cleanup Milestone-2.0

Comment #1 on issue 6036 by marianopeck: Let DataStream only for Monticello
http://code.google.com/p/pharo/issues/detail?id=6036

So...I did:

1) Remove DiskProxy
2) Let the minimum needed stuff in DataStream ONLY for Monticello.

Notice that this could affect Monticello. I run Monticello tests and tried  
to commit and things like that and seems to work.

The next step are:
a) remove ReferenceStream
b) rename DataStream to MCDataStream and move it to Monticello package
c) Move DeepCopier somewhere and remove the package SystemObject Storage

Name:  
SLICE-Issue-6036-Let-DataStream-only-for-Monticello-MarianoMartinezPeck.1
Author: MarianoMartinezPeck
Time: 5 June 2012, 7:22:53.829 pm
UUID: 0ada0c08-d38c-4ac6-9db3-4c7cd8f5ea06
Ancestors:
Dependencies: System-Object Storage-MarianoMartinezPeck.163

Fix to issue 6036


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

Comment #2 on issue 6036 by [hidden email]: Let DataStream only for  
Monticello
http://code.google.com/p/pharo/issues/detail?id=6036

May be we should not rename DataStream...


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

Comment #3 on issue 6036 by marianopeck: Let DataStream only for Monticello
http://code.google.com/p/pharo/issues/detail?id=6036

Ok, but I didn't do that in this slice. IN this slice I just cleaned. The  
renamed (and move to Monticello package) was intended in a future slice.  
THis must be the first step.


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

Comment #4 on issue 6036 by [hidden email]: Let DataStream only for  
Monticello
http://code.google.com/p/pharo/issues/detail?id=6036

Ok I will look at it and integrate it soon then.


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

Updates:
        Status: integrated

Comment #5 on issue 6036 by [hidden email]: Let DataStream only for  
Monticello
http://code.google.com/p/pharo/issues/detail?id=6036

in 20124


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