Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 168571 - Please review sci-electronics/gnucap-0.35.20070221.ebuild and mark ~amd64 if appropriate
Summary: Please review sci-electronics/gnucap-0.35.20070221.ebuild and mark ~amd64 if ...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: AMD64 Project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-02-27 12:51 UTC by Denis Dupeyron (RETIRED)
Modified: 2007-02-27 13:28 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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!