Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 621046 - dev-util/bloaty-0_p20170321 fails to build
Summary: dev-util/bloaty-0_p20170321 fails to build
Status: RESOLVED TEST-REQUEST
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Manuel Rüger (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-06-06 15:56 UTC by darkdimius
Modified: 2017-06-06 22:59 UTC (History)
3 users (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 darkdimius 2017-06-06 15:56:21 UTC
emerge bloaty fails to build:

>>> Preparing source in /var/tmp/portage/dev-util/bloaty-0_p20170321/work/bloaty-0_p20170321 ...
sed: -e expression #2, char 12: unknown option to `s'
 * ERROR: dev-util/bloaty-0_p20170321::gentoo failed (prepare phase):
 *   (no error message)
 *
 * Call stack:
 *     ebuild.sh, line 115:  Called src_prepare
 *   environment, line 905:  Called die
 * The specific snippet of code:
 *       sed -i -e "s#\$(RE2_[AH])##" -e "s/	ar /	$(tc-getAR) /" -e "s#-lpthread#-lre2 -lpthread#" -e "/^CXXFLAGS/ s#-I third_party/re2##" Makefile || die
Comment 1 David Hallas 2017-06-06 18:02:37 UTC
I just emerged dev-util/bloaty-0_p20170321 and it worked just fine.

Could you run some of the sed command below to narrow down which substitution is failing?
Comment 2 darkdimius 2017-06-06 18:29:04 UTC
@David Hallas.
It fails because I have AR set to `/usr/bin/gcc-ar` in make.conf.
This sed line assumes that there are no slashes in AR location.
Comment 3 David Hallas 2017-06-06 19:15:32 UTC
So, simply changing the -e "s/	ar /	$(tc-getAR) /" expression to -e "s#	ar #	$(tc-getAR) #" should fix the problem?
Comment 4 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2017-06-06 21:07:21 UTC
Or just putting '$(AR)' make variable, and passing AR to make instead of sweeping issues under the carpet just to force bad ideas. Also, using patches instead of unreliable sed expressions.
Comment 5 Manuel Rüger (RETIRED) gentoo-dev 2017-06-06 22:10:08 UTC
commit 77c8552e1c1d23172123080b8c9331cd81106c55
Author: Manuel Rüger <mrueg@gentoo.org>
Date:   Wed Jun 7 00:08:52 2017 +0200

    dev-util/bloaty: Update snapshot
    
    Gentoo-Bug: #621046
    Package-Manager: Portage-2.3.6, Repoman-2.3.2

Please test the new snapshot.
Comment 6 darkdimius 2017-06-06 22:59:06 UTC
Fixed now! Thanks for a swift resolution!