Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 729914 - dev-lang/duktape-2.5.0: does not install header or pkgconfig files
Summary: dev-lang/duktape-2.5.0: does not install header or pkgconfig files
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Ian Whyman (thev00d00) (RETIRED)
URL:
Whiteboard:
Keywords:
: 729842 729936 729960 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-06-27 14:28 UTC by Brian Evans (RETIRED)
Modified: 2020-06-28 23:52 UTC (History)
5 users (show)

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


Attachments
ducktape-2.5.0.ebuild.patch (duktape-2.5.0.ebuild.patch,469 bytes, patch)
2020-06-27 15:16 UTC, Vladimir Smirnov
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brian Evans (RETIRED) gentoo-dev 2020-06-27 14:28:05 UTC
After upgrade to 2.5.0, only the following files are installed which look more like a binary than a link install as there are no header files.

~ # qlist duktape
/usr/lib64/libduktape.so.204
/usr/lib64/libduktape.so.204.20400
/usr/lib64/pkgconfig/duktape.pc
Comment 1 Brian Evans (RETIRED) gentoo-dev 2020-06-27 14:30:06 UTC
The previous version had the following:

~ # qlist duktape
/usr/lib64/libduktape.so
/usr/lib64/libduktape.so.204
/usr/lib64/pkgconfig/duktape.pc
/usr/lib64/libduktape.so.204.20400
/usr/lib64/libduktaped.so.204.20400
/usr/include/duktape.h
/usr/include/duk_config.h
/usr/lib64/libduktaped.so.204
/usr/lib64/libduktaped.so
Comment 2 Vladimir Smirnov 2020-06-27 15:16:13 UTC
Created attachment 646836 [details, diff]
ducktape-2.5.0.ebuild.patch

This fixes the way how it install files.

Problem is that because of ${D::-1} it removes last symbol from ${D} and install headers to `/var/tmp/portage/dev-lang/duktape-2.5.0/imag` which is not a valid folder.
Comment 3 Vladimir Smirnov 2020-06-27 15:33:22 UTC
This is due to EAPI=7 no longer have trailing slash in some variables, including D. So it's now harmful to remove last symbol before doing sed.

For original ebuild author correct way was to use ${D%/} instead of ${D::-1} currently in use in the ebuild. That way this issue would never surface.
Comment 4 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2020-06-27 19:55:26 UTC
*** Bug 729936 has been marked as a duplicate of this bug. ***
Comment 5 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2020-06-27 19:56:29 UTC
*** Bug 729842 has been marked as a duplicate of this bug. ***
Comment 6 tt_1 2020-06-27 20:04:02 UTC
the proposed fix seems to be working with cross compile
Comment 7 Larry the Git Cow gentoo-dev 2020-06-28 08:52:24 UTC
The bug has been closed via the following commit(s):

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

commit 10cf094cdd34df34e8085e3ab615a7a7988bbad8
Author:     Ian Whyman <thev00d00@gentoo.org>
AuthorDate: 2020-06-28 08:52:04 +0000
Commit:     Ian Whyman <thev00d00@gentoo.org>
CommitDate: 2020-06-28 08:52:04 +0000

    dev-lang/duktape: Fix header install
    
    Closes: https://bugs.gentoo.org/729914
    Closes: https://bugs.gentoo.org/729840
    Package-Manager: Portage-2.3.103, Repoman-2.3.23
    Signed-off-by: Ian Whyman <thev00d00@gentoo.org>

 dev-lang/duktape/{duktape-2.5.0.ebuild => duktape-2.5.0-r1.ebuild} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Comment 8 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2020-06-28 23:52:49 UTC
*** Bug 729960 has been marked as a duplicate of this bug. ***