Bug 704348 – sci-libs/parmetis-4.0.3 and sci-libs/metis-5.1.0-r3 install headers in the wrong directory

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[Description kevinmbecause 2019-12-31 04:54:55 UTC]

Both packages install metis.h into /usr/include/ while creating a pkg-config file which indicates they are installed in /usr/include/metis/. parmetis installs /usr/include/parmetis.h while the pkg-config file indicates it should be in /usr/include/parmetis/

Reproducible: Always

Steps to Reproduce:
1. Install metis or parmetis
2. pkg-config --cflags metis or parmetis
3. ls /usr/include/metis
Actual Results:  
/usr/include/metis and /usr/include/parmetis do not exist

Expected Results:  
/usr/include/metis and /usr/include/parmetis should contain metis.h and parmetis.h

[Comment 1 kevinmbecause 2019-12-31 04:56:21 UTC]

Created attachment 601982 [details]
Revised parmetis ebuild

[Comment 2 kevinmbecause 2019-12-31 04:56:53 UTC]

Created attachment 601984 [details]
Revised metis ebuild

[Comment 3 CaptainBlood 2020-02-14 12:35:09 UTC]

(In reply to kevinmbecause from comment #1)
> Created attachment 601982 [details]
> Revised parmetis ebuild

Expected directory not created here:

Installed here as parmetis-4.0.3-r3 from local repo:

diff parmetis-4.0.3-r1.ebuild parmetis-4.0.3-r2.ebuild
1c1
< # Copyright 1999-2020 Gentoo Authors
---
> # Copyright 1999-2019 Gentoo Authors
19d18
< LICENSE="all-rights-reserved"
21a21
> LICENSE="free-noncomm"
23d22
< RESTRICT="mirror bindist"
60d58
< 			-e '/add_subdirectory(include/d' \
63a62,64
> 	sed -i \
> 		-e '/add_subdirectory(include/d' \
> 		CMakeLists.txt || die
104c105
< 	insinto /usr/include
---
> 	insinto /usr/include/metis
138a140,141
> 		insinto /usr/include/${PN}
> 		doins include/parmetis.h
-----------------------------------------------------------
eix parmetis 
[I] sci-libs/parmetis
     Available versions:  4.0.3-r1^md 4.0.3-r1^md[1] 4.0.3-r2[1] {doc double-precision examples int64 mpi openmp pcre static-libs}
     Installed versions:  4.0.3-r2[1](15:19:06 14/02/2020)(double-precision int64 mpi openmp -doc -examples -pcre -static-libs)
     Homepage:            http://www-users.cs.umn.edu/~karypis/metis/parmetis/
     Description:         Parallel (MPI) unstructured graph partitioning library

[1] "local-repo" /var/db/repos/local-repo
amd64 /var/db/repos/local-repo/sci-libs/parmetis #  ls /usr/include/parmeti*
parmetis.h
--------------------------------------------------------
Thks 4 ur attention.

[Comment 4 Larry the Git Cow 2023-05-03 08:49:27 UTC]

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=bf207af9976559a6e11ba760fc967f1f6cf0808a

commit bf207af9976559a6e11ba760fc967f1f6cf0808a
Author:     Andrew Ammerlaan <[e-mail]>
AuthorDate: 2023-05-03 08:48:12 +0000
Commit:     Andrew Ammerlaan <[e-mail]>
CommitDate: 2023-05-03 08:49:15 +0000

    sci-libs/parmetis: fixup pkgconfig file
    
    Newer versions of parmetis no longer have gklib bundled.
    For backwards compatibility add gklib to the pkgconfig
    file for software that still assumes that gklib is
    part of parmetis.
    
    Bug: https://bugs.gentoo.org/905632
    Closes: https://bugs.gentoo.org/704348
    Signed-off-by: Andrew Ammerlaan <[e-mail]>

 ...etis-4.0.3_p20230326.ebuild => parmetis-4.0.3_p20230326-r1.ebuild} | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0df978301da3f85b322ea621500dca812c6c8a0f

commit 0df978301da3f85b322ea621500dca812c6c8a0f
Author:     Andrew Ammerlaan <[e-mail]>
AuthorDate: 2023-05-03 08:44:04 +0000
Commit:     Andrew Ammerlaan <[e-mail]>
CommitDate: 2023-05-03 08:49:14 +0000

    sci-libs/metis: fixup pkgconfig file
    
    Newer versions of metis no longer have gklib bundled.
    For backwards compatibility add gklib to the pkgconfig
    file for software that still assumes that gklib is
    part of metis.
    
    Closes: https://bugs.gentoo.org/905632
    Bug: https://bugs.gentoo.org/704348
    Signed-off-by: Andrew Ammerlaan <[e-mail]>

 sci-libs/metis/{metis-5.2.1.ebuild => metis-5.2.1-r1.ebuild} | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

[Comment 5 Larry the Git Cow 2023-05-03 09:25:21 UTC]

The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=4c2662713534dc1b7abf9f546d53af0ea6ea8ecb

commit 4c2662713534dc1b7abf9f546d53af0ea6ea8ecb
Author:     Sam James <[e-mail]>
AuthorDate: 2023-05-03 09:24:37 +0000
Commit:     Sam James <[e-mail]>
CommitDate: 2023-05-03 09:24:37 +0000

    profiles: mask gone (para)metis versions to force upgrades
    
    Bug: https://bugs.gentoo.org/704348
    Bug: https://bugs.gentoo.org/905632
    Bug: https://bugs.gentoo.org/905641
    Signed-off-by: Sam James <[e-mail]>

 profiles/package.mask | 6 ++++++
 1 file changed, 6 insertions(+)