On 28 Jan 2014, at 00:18, Sven Van Caekenberghe <sven@stfx.eu> wrote:

OK, I will try and trust you and Sean.

Well, it is in 30 726, but there are 2 regressions found:

ReleaseTests.ReleaseTest.testObsoleteClasses
ReleaseTests.ReleaseTest.testUnknownProcesses

on all 3 platforms, these might be transient, but with all the renaming something could have gone wrong....

On 28 Jan 2014, at 00:13, Benjamin <benjamin.vanryseghem.pharo@gmail.com> wrote:

On 27 Jan 2014, at 20:08, Sven Van Caekenberghe <sven@stfx.eu> wrote:

Are the compatible ?
Does one have to come before the other ?
Are they both needed ?

I think we should drop

https://pharo.fogbugz.com/f/cases/12717/ReactiveVariable-NewValueHolder

and integrate

https://pharo.fogbugz.com/f/cases/12684/NewValueHolder-Enhancements

For at least two reasons:
- I added hooks for Spec to continue working with the slice I proposed (otherwise no more Spec fix until the release)
- I do not have to review it :P

Ok maybe the second point is not that relevant LOL
Just that I am sure that the fix I proposed works well with Spec.
And I an pretty sure the other will break it :)

I was not aware of another case related to this.
It is not flagged Spec nor my name, so there is no way for me to guess it exists.

Ben