pharo-graphics / Roassal

The Roassal Visualization Engine
MIT License
14 stars 8 forks source link

Fix usage of removed dependentPackages #63

Closed pavel-krivanek closed 2 months ago

pavel-krivanek commented 2 months ago

God knows why, this PR does a lot of format changes too.. For actual changes, look for string dependentPackagesOf: in the PR files preview.

astares commented 2 months ago

@pavel-krivanek The format changes are due to Tonel adoptions in latest release - so it is dependends if you use Iceberg in Pharo 11 (or 10 and before) or using Pharo 12 (or higher)

It is my understanding that we are back to Strings instead of Symbols due to compatibility with other Smalltalks using Tonel and now use Tonel version 3.0

See TonelWriter(class)>>defaultVersion

So it is best to create a PR in a Pharo version that you intend the change to be integrated in.