Created attachment 145850 [details]
Some eclipse plugin settings need to be modified to include the "freebsd" OS for the default UI theme (GTK) and a default external web browser (Firefox).
Make the build quieter in the pre-patch script.
stupid diff viewer is eating half the patch.
This is a simple update from the maintainer, moving to patch-ready.
Unfortunately I cant take care of this PR - the patch fails to build for me on 9.3 due to some Maven out of memory error, what's strange, since my testing machine has 16GB of RAM and 4GB of swap available, and it was building fine for 11 and 10 amd64/i386 - numerous attempts to build on 9.3 failed for me.
I don't see anything in the patch that is platform specific, so if 9.3 has a problem it was there before and not caused by patch.
I'll sign off on this if you want to commit it.
I'd rather not, since I can't really get this thing built on 9.x and below. I will talk to Steve if he can try testing this on his setup, and if it goes well, I could take care of committing it. Until I can be sure it actually builds on all supported platforms, I'd rather not commit it.
well, can you build it on 9.x without any modification?
I suspect your poudriere just has a memory limit on tmpfs somewhere that's way too low.
I've got a high ram, dedicated poudriere machine.
poudriere testport logs for 10.0/amd64 and 9.3/amd64 at:
(and if you wait another hour or so, 8.4/amd64 will be there also)
No problems building 9.3 package here.
I think that's sufficient proof to go ahead and commit the PR as-is now.
A commit references this bug:
Date: Wed Aug 27 14:23:58 UTC 2014
New revision: 366329
java/eclipse: Set a default theme and external browser
Submitted by: Jimmy Kelley <firstname.lastname@example.org>
Reviewed by: swills (mentor), marino (mentor)
Approved by: mentors (implicit)
Committed, thanks for your work!