2010/1/9 Miguel Enrique Cobá Martinez <[email protected]>

> Hi Mariano,
>
> good job. Some feedback.
>
> The installScript has unused variables and your proxy settings exposed.
> Maybe this should be deleted from the final build.
>
>
hahahahah I knew I was going to forget about that :(
Thanks. Some bash guru that help me to automatize this? I would like
something that automatically removes that from the file if it is found.
Can you help me Miguel ?


> Shouldn't be this a RC 2? because there is a new build process and has
> nothing to do with the RC1?
>
>
I thought about putting it RC2, actually, I was going to do that. But as the
PharoCore was named rc1, I din't want to name it rc2 because maybe it was
confusing.
I am not sure, thus. What do you think ?


> Also, the changes files, shouldn't be compressed for each RC and for the
> final release?
>

Excellent point. Do you rather to Smalltalk >> condenseChanges ?  It can be
perfectly added. Right now, we are doing all this:


    MCFileBasedRepository flushAllCaches.
    MCDefinition clearInstances.
    Smalltalk removeEmptyMessageCategories.
    Utilities cleanseOtherworldlySteppers.
    Undeclared removeUnreferencedKeys.
    Categorizer sortAllCategories.
    Symbol compactSymbolTable.
    Smalltalk at: #ReleaseBuilderDeveloper ifPresent: [:builder | builder
new fixObsoleteReferences].
    RequiredSelectors initialize.
    ProvidedSelectors initialize.
    LocalSends initialize.
    Smalltalk organization removeEmptyCategories.
    SystemBrowser removeObsolete.
    TheWorldMenu removeObsolete.
    AppRegistry removeObsolete.
    Set rehashAllSets.
    Smalltalk cleanOutUndeclared.
    Smalltalk garbageCollect.


Do you think something else should be addded?   Maybe we can reuse
ScriptLoader cleanings ?



> Also, the packages.txt file talks about PharoCore1.0 (#10505). Has
> someone thought about the name shown in System->About:
>
> PharoCore1.0rc1
> Latest update: #10505
>
> Because this is not a PharoCore image. Shouldn't this read something
> like
>
> Pharo 1.0 Dev image.
> Based on PharoCore1.0rc1
> Latest update: #10505
>
>
Yes, that's true. I agree. At least it was not "my falt" as it was already
like that. But I am agree we should change that. However, I don't know how
to do it.
Maybe something related to SystemVersion? Any help ?



> What about the disabling of software updates in the Pharo dev/web
> images? Will this be implemented or a careful update stream will be made
> in order to not break a dev image with overrides (real difficult I
> think).
>
>
Maybe. But in addition, sometimes there are bugs that updating just fix
them. I am not sure if I want to disable. I rather warn people about it.
Do you know if we break some overrrides doing an update?


> Maybe all this setup can be put in a post-script in the
> ConfigurationOfPharo class.
>

It is. We have pre and post do its. So, no problem :)


>
> That's all for now. I'm going to use this image to work the following
> days.
>

Cheers and keep the good work.
>
>
Thanks Miguel for the excellent feedback. Let's hope to build better images!

Mariano



> --
> Miguel Cobá
> http://miguel.leugim.com.mx
>
>
> _______________________________________________
> Pharo-project mailing list
> [email protected]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Reply via email to