2015-02-10 15:27 GMT+01:00 Pablo Herrero <pablodherrero@gmail.com>:
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 <btc@openinworld.com>:
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 <pablodherrero@gmail.com> 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...
.
Thanks, Pablo