the configure script of sci-physics is an ad-hoc, home made one, which is hard wired to check /usr/lib, etc. This would make the result software depending on libraries from outside prefix. Reproducible: Always Steps to Reproduce: emerge root Actual Results: configure outputs dependencies outside prefix. What is the recommended way to fix this? patch the configure? Contact upstream? Reproducible: Always Steps to Reproduce:
*** Bug 346931 has been marked as a duplicate of this bug. ***
Created attachment 255939 [details, diff] prefixify patch
Here I submitted a patch to prefixify ROOT. Another problem is that the libraries ROOT ships are in ${EPREFIX}/usr/lib/root. Which cannot be found by "root.exe" (called by "root") executable. Upstream suggests[1] setting LD_LIBRARY_PATH explicitly, which is considered bad packaging practice. The upstream is very unfriendly with distributions. I guess we need to maintain patches. To resolve this. a. just put all the libraries of ROOT in ${EPREFIX}/usr/lib b. tune LDFLAGS to let root executables look into ${EPREFIX}/usr/lib/root. Which one is more acceptable for prefix? 1. e.g. http://root.cern.ch/drupal/content/how-use-use-python-pyroot-interpreter
depending on how root is packaged today on Gentoo Linux, either of your options would be ok. If Gentoo Linux uses the root dir, then we need to make sure the search path is extended to them.
Created attachment 256011 [details, diff] final
Hi, (In reply to comment #4) > If Gentoo Linux uses the root dir, then we need to make sure the > search path is extended to them. With ABCD's advice, I have made a new patch to fix it. Please review my patch. if it is okay to be included, this bug could be closed. Yours, Benda
newly bumped root-5.28.00b should be prefix ready. please test. thanks.
Dear Sébastien, I have tested root-5.28.00b, it works. Thanks a lot! There is one small issue remains: rpath in prefix environment, ${EPREFIX}/usr/lib/root can not be searched at runtime by host or prefix default. We need to add rpath option when compiling. Fortunately, root provide such a mechanism via ./configure --enable-rpath. Therefore I prepose the attached patch. (adding ~amd64-linux also) (In reply to comment #7) > newly bumped root-5.28.00b should be prefix ready. please test. thanks.
Created attachment 271433 [details, diff] enable rpath in prefix env, add ~amd64-linux keyword
Hello, I merged your changes into root-5.30.00 ebuild from the science overlay. Please test it out, thanks.
(In reply to comment #10) > I merged your changes into root-5.30.00 ebuild from the science overlay. > Please test it out, thanks. Hi, Andrew, I have tested =root-5.32* from science overlay on x86-linux. I works like a charm. Please consider keywording it.
Hello, (In reply to comment #11) > Hi, Andrew, I have tested =root-5.32* from science overlay on x86-linux. I > works like a charm. I'm glad to hear this :) > Please consider keywording it. Hmm, it already has ~amd64-linux keyword, or do you mean something else?
(In reply to comment #12) > > Please consider keywording it. > > Hmm, it already has ~amd64-linux keyword, or do you mean something else? I meant I have tested ~x86-linux, please keyword that also. Good job and thank you! I think we can close this bug when >=root-5.32* migrates to gentoo-x86 tree.
I added ~x86-linux keyword in science overlay without version bump.
(In reply to comment #14) > I added ~x86-linux keyword in science overlay without version bump. Pardon me, but when would you bump an ebuild for a keyword addition?
Hello, (In reply to comment #15) > Pardon me, but when would you bump an ebuild for a keyword addition? If it is just for a keyword addition — never. As far as I understand Gentoo ebuild policy, simple keyword addition is not a subject for a revision bump: http://www.gentoo.org/proj/en/devrel/handbook/handbook.xml?part=3&chap=1#doc_chap3 See "Versioning and revision bumps" paragraph. There are no changes in the installed files or dependencies. Program behaviour and user experience will not be affected. The only affected are people using ~x86-linux prefix, they need to resync their tree anyway.
root-5.32 prefix clean now in tree. thanks!
