2015-02-10 15:27 GMT+01:00 Pablo Herrero <[email protected]>: > I think I tried that before but it didn't work, maybe I did something > wrong then. >
I changed the compiler to the old one. load your slice changed the compiler back to opal. - But now, runing OC* tests half of them fail. - Executing OpalCompiler recompileAll throws some errors and the images is really slow, constantly writing errors to the debugfile: [31mMessageNotUnderstood: RBArgumentNode>>start: [0mRBArgumentNode(Object)>>doesNotUnderstand: #start: RBArgumentNode(RBVariableNode)>>named:start: RBArgumentNode class(RBVariableNode class)>>named:start: RBArgumentNode class(RBVariableNode class)>>identifierNamed:at: RBExplicitVariableParser>>parsePrimitiveIdentifier RBExplicitVariableParser(RBParser)>>parseVariableNode RBExplicitVariableParser(RBParser)>>parseKeywordPattern RBExplicitVariableParser(RBParser)>>parseMessagePattern > > 2015-02-10 11:23 GMT-03:00 Ben Coman <[email protected]>: > > Hi Pablo, > > > > I presume you have not noticed when you <Edit> the Fogbugz issue, there > are > > two header fields "Preload" & "Postload" , which may satisfy your need. > Add > > your code just like you would type it into a Workspace. > > > > cheers -ben > > > > On Tue, Feb 10, 2015 at 9:19 PM, Pablo Herrero <[email protected]> > > wrote: > >> > >> Hello, > >> > >> I had this patch (for issue 11992) for a while now at the bug system. > >> The problem I'm having is that in order to to run the test suite > >> properly I need to run some code before merging the slice and then > >> just before running the test suite. And as a consequence, I'm not > >> notified whenever some other patch that gets integrated breaks my > >> code. > >> > >> I have read that you can use metacello to configure post and preLoad > >> hooks for a project. I'm not sure if this could work for the fogbugz > >> CI, also since you need to add a class in order to use metacello it > >> would probably get integrated into the image which is not what I > >> intend. > >> > >> Is there then some easy way so run some code before the CI merges the > >> patch and just before the test suite runs?. > >> > >> PS1: Sorry is the answer for this turns out being pretty obvious, I'm > >> not that familiar with metacello. > >> PS2: The issue haven't receive a lot of attention probably because > >> it's been at 'Work Needed' state since forever, maybe it would help if > >> the test suite passes. > >> PS3: Here's a link to the issue > >> > >> > https://pharo.fogbugz.com/f/cases/11992/Remove-tokens-from-the-AST-Core-Node-classes > >> . > >> > >> Thanks, > >> Pablo > >> > > > >
