Re: [ros-dev] [ros-diffs] [cgutman] 56043: [NEWCC] - Turn NewCC on temporarily for some testing - Patchbot is broken so I had to do it this way :(‏

2012-03-05 Thread Pierre Schweitzer
Nice try! However it failed on KVM: http://build.reactos.org/builders/Linux_AMD64_1% 20KVM-CMake-Test/builds/255/steps/test/logs/stdio Regards, -- Pierre Schweitzer pie...@reactos.org Systems Administrator ReactOS Foundation smime.p7s Description: S/MIME cryptographic signature

Re: [ros-dev] [ros-diffs] [cgutman] 56043: [NEWCC] - Turn NewCC on temporarily for some testing - Patchbot is broken so I had to do it this way :(‏

2012-03-05 Thread Cameron Gutman
On Mar 5, 2012, at 3:09 PM, Pierre Schweitzer wrote: Nice try! However it failed on KVM: http://build.reactos.org/builders/Linux_AMD64_1% 20KVM-CMake-Test/builds/255/steps/test/logs/stdio Seems like it may be a race between the balancer thread and CC shutting down. Definitely a case I've

Re: [ros-dev] [ros-diffs] [cgutman] 56043: [NEWCC] - Turn NewCC on temporarily for some testing - Patchbot is broken so I had to do it this way :(‏

2012-03-05 Thread caemyr
Patchbot is not broken, it simply needs revision number On Mon, Mar 5, 2012, at 03:14 PM, Cameron Gutman wrote: On Mar 5, 2012, at 3:09 PM, Pierre Schweitzer wrote: Nice try! However it failed on KVM: http://build.reactos.org/builders/Linux_AMD64_1%

[ros-dev] Problems with patchbot/newcc/rosbe

2012-03-05 Thread caemyr
After a quick look into your problems with patchbot it seems that vbox scripts were missing some errorlevel checks, which should bail out the test run at prepare stage when ISO was not present (which is the situation when you dont pass the proper revision/id number. Same thing was with newcc1

Re: [ros-dev] Problems with patchbot/newcc/rosbe

2012-03-05 Thread Zachary Gorden
Would also be nice if we could turn off the whole sound alert thing. Some of us find it a bit irritating. On Mon, Mar 5, 2012 at 6:11 PM, cae...@myopera.com wrote: After a quick look into your problems with patchbot it seems that vbox scripts were missing some errorlevel checks, which should