Summary: | sci-electronics/pcb - add -9999 live git ebuild | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Joe M <joe9mail> |
Component: | Current packages | Assignee: | The Soldering-Iron Brotherhood <sci-electronics> |
Status: | RESOLVED WONTFIX | ||
Severity: | enhancement | CC: | alex_y_xu, joe9mail |
Priority: | Normal | Keywords: | EBUILD |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
pcb git ebuild
live ebuild live ebuild freecad-9999.ebuild freecad-9999.ebuild pcb-99999999.ebuild pcb-99999999.ebuild pcb-9999999.ebuild pcb-99999999.ebuild |
Description
Joe M
2013-03-09 05:41:46 UTC
Created attachment 341376 [details, diff]
pcb git ebuild
Created attachment 370250 [details]
live ebuild
IUSE="dbus debug doc gcode gif +gtk jpeg m4lib-png motif nelma opengl png test tk toporouter xrender nls" sort IUSE delete unused code xrender? ( motif ) " one line if !(use gtk || use motif); de morgan's laws if ! use png; then sed -i '/^hid_png/d' tests/tests.list || die fi use png || ... if (use png || use jpeg || use gif) ; then this is not C, you don't need () around if cond Created attachment 370264 [details]
live ebuild
(In reply to Alex Xu (Hello71) from comment #3) > IUSE="dbus debug doc gcode gif +gtk jpeg m4lib-png motif nelma opengl png > test tk toporouter xrender nls" > > sort IUSE done > > delete unused code > > xrender? ( motif ) > " > > one line done > > if !(use gtk || use motif); > > de morgan's laws "not (A or B)" is the same as "(not A) and (not B)" changed to if (! use gtk) && (! use motif); then Is that what is needed? > > if ! use png; then > sed -i '/^hid_png/d' tests/tests.list || die > fi > > use png || ... done > > if (use png || use jpeg || use gif) ; then > > this is not C, you don't need () around if cond done Thanks (In reply to Joe M from comment #5) > (In reply to Alex Xu (Hello71) from comment #3) > > > > if !(use gtk || use motif); > > > > de morgan's laws > "not (A or B)" is the same as "(not A) and (not B)" > changed to if (! use gtk) && (! use motif); then > Is that what is needed? > > Thanks this isn't C, you *do not need ()*. parens in shell mean "spawn a new process", which is to be avoided if possible. if ! use gtk && ! use motif Created attachment 370320 [details]
freecad-9999.ebuild
changed to
if ! use gtk && ! use motif; then
ewarn "${P} is being built without a GUI, make sure you know what you're doing! Otherwise please enable GTK or MOTIF use flags"
fi
(In reply to Alex Xu (Hello71) from comment #6) > (In reply to Joe M from comment #5) > > (In reply to Alex Xu (Hello71) from comment #3) > > > > > > if !(use gtk || use motif); > > > > > > de morgan's laws > > "not (A or B)" is the same as "(not A) and (not B)" > > changed to if (! use gtk) && (! use motif); then > > Is that what is needed? > > > > Thanks > > this isn't C, you *do not need ()*. parens in shell mean "spawn a new > process", which is to be avoided if possible. > > if ! use gtk && ! use motif Thanks for the explanation. Created attachment 370322 [details]
freecad-9999.ebuild
Created attachment 370344 [details]
pcb-99999999.ebuild
if use gtk ; then myconf="--with-gui=gtk $(use_enable dbus) $(use_enable opengl gl) --disable-xrender" elif use motif ; then myconf="--with-gui=lesstif $(use_enable dbus) $(use_enable xrender) --disable-gl" else myconf="--with-gui=batch --disable-xrender --disable-dbus --disable-gl" fi maybe warn when some USE flags are being ignored? e.g. use opengl && einfo "Ignoring opengl use flag, requires gtk" (In reply to Alex Xu (Hello71) from comment #11) > if use gtk ; then > myconf="--with-gui=gtk $(use_enable dbus) $(use_enable opengl gl) > --disable-xrender" > elif use motif ; then > myconf="--with-gui=lesstif $(use_enable dbus) $(use_enable xrender) > --disable-gl" > else > myconf="--with-gui=batch --disable-xrender --disable-dbus --disable-gl" > fi > > maybe warn when some USE flags are being ignored? > > e.g. use opengl && einfo "Ignoring opengl use flag, requires gtk" ok, added the change to the attached ebuild. Created attachment 371076 [details]
pcb-99999999.ebuild
Created attachment 371794 [details]
pcb-9999999.ebuild
changed IUSE from 2 lines to a single line.
Created attachment 371796 [details]
pcb-99999999.ebuild
Unless someone steps up to maintain this, it won't happen. |