X Tutup
Skip to content

Make loadable in Pharo 7#57

Closed
StephanEggermont wants to merge 1 commit intoSeasideSt:masterfrom
StephanEggermont:patch-1
Closed

Make loadable in Pharo 7#57
StephanEggermont wants to merge 1 commit intoSeasideSt:masterfrom
StephanEggermont:patch-1

Conversation

@StephanEggermont
Copy link

No description provided.

@StephanEggermont
Copy link
Author

Build failure looks unrelated

@marschall
Copy link
Contributor

Since we're now metadataless and #14 as been fixed I wonder whether it would not be better to copy and paste Grease-Pharo60-Core to Grease-Pharo70-Core or Grease-Pharo7-Core.

@StephanEggermont
Copy link
Author

I noticed kernel class traits have been flattened in Pharo 7. In Magritte that is an issue. Here too?

@jbrichau
Copy link
Member

jbrichau commented Nov 9, 2017

I agree with @marschall , we have adopted the strategy to copy/paste the packages per Pharo version. If we load the same packages in the different Pharo versions, there is more work to support and understand where somethings needs to be fixed than fixing a bug across all supported platforms.

@jbrichau
Copy link
Member

Added Pharo70 package in aa9e29d

@jbrichau
Copy link
Member

Fixed this in #60

@jbrichau jbrichau closed this Nov 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

X Tutup