Skip to content

Conversation

jbrichau
Copy link
Member

@jbrichau jbrichau commented Jun 28, 2020

For Gemstone, the following changes have occurred:

  • Monticello metadata removal
  • Added Symbol>>greaseAsMutator (as a Grease abstraction for Symbol>>asMutator)

jecisc and others added 26 commits June 3, 2019 22:31
update travis lineup for the new 3.5.0 release and update README
…ssumes-new-collection

95 Don't assume Interval >> #sorted returns a new object
add pharo8 and pharo9 to testing
@jbrichau jbrichau requested a review from dalehenrich June 28, 2020 09:41
@dalehenrich
Copy link
Member

@jbrichau I see that this PR is hitting the bug in 3.5.1/3.5.2 that appeared just recently (SeasideSt#105) ... since there have been earlier tests that have passed against 3.5.1/3.5.2 I'm not exactly sure where the change has been made that is causing the failure ... and I have not had the cycles to spend the time to track this problem down ...and it only seems to affect the Grease travis builds ... also it seems that the change was introduced between a month and 20 days ago ....

GsDevKit/Grease tests passed 2 months ago agains 3.5.1 and stated failing 19 days ago against 3.5.1.

Seaside tests failed a day ago with same error against 3.5.2 and passed 26 days ago against 3.5.0.

Stiil looking a bit ... do you have any ideas what may have changed?

@dalehenrich
Copy link
Member

here's a passing GsDevKit_home 3.5.1build that ran three days ago that did a build with GsDevKit/Grease master branch without a build failure.... which is now very odd ...

@dalehenrich
Copy link
Member

dalehenrich commented Jun 29, 2020

Turns out that the failure is actually occurring while SmalltalkCI is being loaded into the stone, which implies that this is a SmalltalkCI regression/bug (see SeasideSt#105 (comment)).

I think that there has been some recent work in SmalltalkCI related to improving Metacello for use with Squeak and perhaps, there was a change that blows up for GemStone 3.5.1 and beyond ...

@jbrichau
Copy link
Member Author

After fixing SeasideSt#105 all is well!

@dalehenrich dalehenrich merged commit 7e69439 into GsDevKit:master Jun 30, 2020
@jbrichau jbrichau deleted the mergegrease branch July 1, 2020 05:51
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.

5 participants