Mariano,

The progress-bar problem is solvable.

The progress bar problem appears because the dev image doits reinitalize the 
progress bar class. The first question is: "Must the progress bar code be 
initialized?" if no, then remove the initialization code and the problem should 
be solved:) If yes, then I will dig into the problem and see what I can come up 
with ... there are a couple of routes I can take, but I'd like to know that we 
can't bypass the initialization before getting my hands that dirty:)

Dale

----- "Mariano Martinez Peck" <[email protected]> wrote:

| 2010/1/24 Miguel Enrique Cobá Martinez <[email protected]>
| 
| >
| > Now to the point. I understand that the separation of concerns is
| good,
| > because ImageForDevelopers is already a separated and tested
| package
| > that shouldn't be inside ConfigurationOfPharo.
| >
| 
| I will tell you a bit the background: First, when I took the
| installScript
| from Damien, there where some stuff done in the script (like settings
| the
| preferences and the fonts) done in the installScript. It didn't have
| sense
| for me (of course I asked Damien) and I put that in ImageForDevelopers
| (and
| this is cool because it is versiones with monticello). Actually in
| DEVImageCreator >> install.  I need also to clean that class as it has
| a lot
| of old stuff. But not time for the moment.
| 
| When I create rc1, ImageForDevelopers was a dependency of
| ConfigurationOfPharo as you said. There was a problem installing it
| with the
| progress bar and Metacello. The image just frezze because of a
| sempahore and
| a delay....see issue:
| http://code.google.com/p/metacello/issues/detail?id=39
| But I can even reproduce the problem....So, that quick solution (as I
| needed
| to relase the Pharo image) was to put in the installScript.
| 
| And after a while I discovered that I even like more that approach.
| Someone
| may want just the packages of a dev image, but not to set
| preferences,
| fonts, cleanups, etc. And I don't have the Metacello bug.
| The problem I found also putting it as a dependency is that it has to
| be the
| LATEST (I think that you cannot even said that XX package has to be
| the last
| one to be loaded) package to be loaded. I can put it as postDoIt for
| the
| project, but this cannot be done because it is ONLY required for
| certain
| groups
| 
| 
| > - Shouldn't be necessary to create a
| ConfigurationOfImageForDevelopers
| > and in this configuration reference ConfigurationOfPharo and then
| the
| > installScript for the dev image use *only*
| > ConfigurationOfImageForDevelopers?
| >
| >
| It can work, but actually for me we are having a problem (maybe we
| need the
| support the post/pre doit for groupos...but dale said it was
| complicated). I
| would like to just do (ConfigurationOfPharo project version: 'XXX')
| load:'ImageForDeveloper'
| 


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

Reply via email to