search

Cuis-Smalltalk / Morphic

MIT License
7 stars 3 forks source link

Loading AllPackages from the package installer fails #4

Open Kirtai opened 1 year ago

Kirtai commented 1 year ago

When I try to use the PackageInstaller to install AllPackages it failes with an error on Dice.

Installing [Dice] requires base system updated to #4430
But this system is updated to #5494
Please install Cuis base system updates and retry.

Which is kind of awkward :)

I see that Dice is deprecated but the Cuis package scan still finds it and still adds it to the AllPackages list.

Maybe it should be moved or renamed to avoid this?

Or maybe the PackageInstaller shouldn't show packages that don't load because they require an image older than the current one?

The same probably applies to other deprecated packages.

Kirtai commented 1 year ago

It also breaks the MorphicBook example in Erudite.

thiagoslino commented 1 year ago

Hi @Kirtai. Can you test again? This has been fixed. (see merge request https://github.com/Cuis-Smalltalk/Morphic/pull/3)

Kirtai commented 1 year ago

Yes, Dice loads but AllPackages now fails on Morphic-Misc1

thiagoslino commented 1 year ago

Fixed in merge request https://github.com/Cuis-Smalltalk/Morphic/pull/5 (not merged yet)

Thanks.

thiagoslino commented 1 year ago

@Kirtai fixed

Kirtai commented 1 year ago

Morphic-Misc1 now asks if it can replace a base method #Morph>>changeColor.

It fails if you say No.

KenDickey commented 1 year ago

The README.md file in the Morph repo notes that the Morphic-Misc1 package is deprecated and has been superseded by packages in the Cuis-Smalltalk-UI repo.

This package and other deprecated packages should be removed from AllPackages.