On Mar 21, 2012, at 6:23 AM, Steve Ebersole wrote: > GRADLE-2169 (or at least tests hanging on your system) was the main reason to > switch to m9. If that's not addressed by m9 then IMO we should not switch. > Like I said on IRC all the reports I saw (JIRA and forum) said that that > particular thing is still an issue. Curious it only fails on some systems. > The next release is supposed to be the first RC, I'd vote we wait for that if > possible. One thought I have been having... Is Gradle using its daemon now > by default? If so, I wonder if our memory settings, when set > environmentally, are not passed along to the daemon. One of the forums posts > suggest the underlying cause of these hangs is situations where the "worker > process" encounters certain types of errors such as OOM. That would > certainly happen trying to run the testsuites.
no, the daemon is not started by default, it just be considered matured enough. you still need "--daemon" explicitly to enable it GRADLE-2169 maybe not the one i'm getting, at least our tests do not use any security stuff I believe. but since my change is ready to push, so, I would suggest you give it a try, if it doesn't break anything (though it doesn't fix my issue either) I would vote we accept it, since with this change, it would make the next upgrade to the CR release easier > > As for name-spacing the manual, the doctype declaration for manual now all > use SYSTEM instead of PUBLIC. So even if they cannot be resolved locally, it > should never try to resolve them remotely. So not sure why this causes extra > time. I would think either it does not load the DTD or it fails. Of course > if you already have the work done to convert it to be name-spaced, I'd say > push it. don't know why, but from my local testing, the dev guide / quickstart docs are built much faster than manual when using dtd > > On Tue 20 Mar 2012 01:11:53 PM CDT, Strong Liu wrote: >> I have pushed my changes [1] >> >> some notes here: >> >> * this upgrade does not resolve the GRADLE-2169 issue, my build still hangs >> (but with "-i" option, it disappears, needs to dig more ) >> * the gradle-jdocbook change doesn't compatible with previous gradle release >> (there is a gradle class renamed) >> * I have to change manual's xml to use namespace instead of the old dtd, >> jdocbook plugin can't resolve those locally, that's why it takes so long to >> finish [2] >> * I haven't created pull request yet, just want to hear feedback first >> >> >> 1. https://github.com/stliu/gradle-jdocbook/tree/upgrade-to-m9 >> https://github.com/stliu/hibernate-orm/tree/upgrade-to-m9 >> >> 2. >> time gradle :documentation:renderDocBook_manual_en-US_html >> ----------------------------------------------------------- (mac + m9) >> Total time: 1 mins 43.315 secs >> >> real1m43.546s >> user1m14.794s >> sys0m2.919s >> ----------------------------------------------------------- (fedora + m8a) >> Total time: 2 mins 33.44 secs >> >> real2m33.527s >> user1m1.912s >> sys0m1.590s >> ----------------------------------------------------------- (mac + m8a) >> Total time: 3 mins 53.628 secs >> >> real3m53.798s >> user1m12.152s >> sys0m3.037s >> ------------------------- >> Best Regards, >> >> Strong Liu <stliu at hibernate.org <http://hibernate.org/>> >> http://about.me/stliu/bio >> > > -- > st...@hibernate.org > http://hibernate.org _______________________________________________ hibernate-dev mailing list hibernate-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/hibernate-dev
