Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 168571

Summary: Please review sci-electronics/gnucap-0.35.20070221.ebuild and mark ~amd64 if appropriate
Product: Gentoo Linux Reporter: Denis Dupeyron (RETIRED) <calchan>
Component: New packagesAssignee: AMD64 Project <amd64>
Status: RESOLVED FIXED    
Severity: normal CC: sci-electronics
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description Denis Dupeyron (RETIRED) gentoo-dev 2007-02-27 12:51:33 UTC
I have just committed an ebuild for a new snapshot of sci-electronics/gnucap. The major difference with previous releases is support for external plugins in the form of .so libraries. As I do not have access to an amd64 machine and I have never used one, I have no idea if all this is amd64/multilib clean or not, so I temporarily dropped the ~amd64 keyword.

So, please review gnucap-0.35.20070221.ebuild and mark ~amd64 if appropriate. If not, fell free to make any modification you feel is necessary. I would appreciate, though, that you contact me before you make any major change. Note that the libraries are installed using insinto/newins because newlib.so does not allow to install in a subdirectory of /usr/lib. This is necessary because of the many duplicate plugin names (right now it's in /usr/lib/gnucap/bsim, /usr/lib/gnucap/ngspice17 and /usr/lib/gnucap/spice3f5).

Thanks in advance,
Denis.
Comment 1 Simon Stelling (RETIRED) gentoo-dev 2007-02-27 13:28:06 UTC
Well guessed, it's not multilib-strict safe. However, it didn't take more then

replacing 
  insinto /usr/lib/gnucap/${PLUGIN_DIR}
with
  insinto /usr/$(get_libdir)/gnucap/${PLUGIN_DIR}

So it's fixed and the keyword added.
Thanks!