On Fri, 2019-11-15 at 08:27 -0800, Erik Joelsson wrote: > On 2019-11-15 04:23, Severin Gehwolf wrote: > > Hi Erik, Magnus, > > > > On Fri, 2019-11-08 at 08:57 -0800, Erik Joelsson wrote: > > > Hello Severin, > > > > > > On 2019-11-08 07:48, Severin Gehwolf wrote: > > > > Right. I believe webrev 03 does this? > > > Yes, that was mostly a reply to Magnus. > > > > Thanks! Tried it, which doesn't seem to work[1]: > > > > http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8233712/04/webrev/ > > > Looking at your job, it's failing already in configure on our Linux > > > hosts. At least on Oracle Linux, it seems ulimit is a bash builtin and > > > not found in path. Configure needs to fallback on that. > > Thanks for the info. This updated webrev passes jdk/submit now: > > http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8233712/05/webrev/ > > > > Thoughts? > > That is an interesting approach. I think it's good to go. Magnus may > want to tweak it, but as he is on vacation, it may take a while to get > feedback from him.
I see. > Perhaps we could just put the builtin logic in the default > BASIC_REQUIRE_PROG, but we could also wait and do that later. No need to > stall this fix more I think. Thanks, Erik. I'll get this pushed on Monday if I hear no objections by then. Cheers, Severin > /Erik > > > Thanks, > > Severin > > > > > > Is there some way to reproduce locally? > > > > > > > > Thanks, > > > > Severin > > > > > > > > [1] [Mach5] mach5-one-sgehwolf-JDK-8233712-4-20191108-1454-6533841: > > > > FAILED > > > > > > > > > /Erik > > > > > > > > > > > Thanks, > > > > > > Severin > > > > > >
