Re: Issue 7178 in pharo: Saving extension methods with MC is badly brokencement>> handlesAnnouncement:

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

Re: Issue 7178 in pharo: Saving extension methods with MC is badly brokencement>> handlesAnnouncement:

pharo
Updates:
        Summary: Saving extension methods with MC is badly brokencement>>  
handlesAnnouncement:

Comment #10 on issue 7178 by [hidden email]: Saving extension  
methods with MC is badly brokencement>> handlesAnnouncement:
http://code.google.com/p/pharo/issues/detail?id=7178

When merging only two packages are getting dirty: Ring-Core-kernel and  
ring-monticello.
Now when integrating it:
Announcements, AST,..... ASMJit,....... got touched.

So there is something wrong. We will have to do another pass on the fix.

It looks like something destabilizes scriptLoader:  as if the marking of  
the packages existing in the system (that are used to compute the  
difference) got impacted and that the system believes that all the packages  
are dirty.

Before the system blocked at letter A (may be it was a network problem) I  
see the packages filling up my package cache up to F and more probably

Stef


_______________________________________________
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 7178 in pharo: Saving extension methods with MC is badly brokencement>> handlesAnnouncement:

pharo

Comment #11 on issue 7178 by [hidden email]: Saving extension  
methods with MC is badly brokencement>> handlesAnnouncement:
http://code.google.com/p/pharo/issues/detail?id=7178

There is something really weird happening. This is not really logical. So  
we should build hypotheses and check them one by one.
- I'm not sure that this change has an impact on the dirtiness of a  
package. I thought that snapshot is invoke on dirty packages not to  
determine are dirty.


_______________________________________________
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 7178 in pharo: Saving extension methods with MC is badly brokencement>> handlesAnnouncement:

pharo

Comment #12 on issue 7178 by [hidden email]: Saving extension  
methods with MC is badly brokencement>> handlesAnnouncement:
http://code.google.com/p/pharo/issues/detail?id=7178

I may have some time tomorrow afternoon to have a look at it.
So far I got all the packages dirty and saved on my local cache by  
scriptLoader.... :(


_______________________________________________
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 7178 in pharo: Saving extension methods with MC is badly brokencement>> handlesAnnouncement:

pharo

Comment #13 on issue 7178 by [hidden email]: Saving extension  
methods with MC is badly brokencement>> handlesAnnouncement:
http://code.google.com/p/pharo/issues/detail?id=7178

I checked and when I do not load this change and modify a simple class,  
scriptloader works as expected.
So I will continue to check the problem.


_______________________________________________
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 7178 in pharo: Saving extension methods with MC is badly brokencement>> handlesAnnouncement:

pharo

Comment #14 on issue 7178 by [hidden email]: Saving extension  
methods with MC is badly brokencement>> handlesAnnouncement:
http://code.google.com/p/pharo/issues/detail?id=7178

so now it is working.... may be I used a wrong image yesterday.


_______________________________________________
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 7178 in pharo: Saving extension methods with MC is badly brokencement>> handlesAnnouncement:

pharo
Updates:
        Status: Integrated

Comment #15 on issue 7178 by [hidden email]: Saving extension  
methods with MC is badly brokencement>> handlesAnnouncement:
http://code.google.com/p/pharo/issues/detail?id=7178

Integrated in 20460


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