Summary: | sys-apps/sed[forced-sandbox]: sed can't re-build itself | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Nils Freydank <holgersson> |
Component: | Current packages | Assignee: | Gentoo's Team for Core System packages <base-system> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | build.log |
Description
Nils Freydank
2018-03-09 19:54:23 UTC
Created attachment 523190 [details]
build.log
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=1088728d5541b66d6e572a431ac15d1013f8fc7d commit 1088728d5541b66d6e572a431ac15d1013f8fc7d Author: Mike Frysinger <vapier@gentoo.org> AuthorDate: 2018-03-11 22:30:04 +0000 Commit: Mike Frysinger <vapier@gentoo.org> CommitDate: 2018-03-11 22:36:28 +0000 sys-apps/sed: use bootstrap logic with USE=forced-sandbox #650052 The build system runs sed scripts that use r (read) commands, so if the system version had sandbox enabled, we won't be able to rebuild sed. Use the existing bootstrap logic so people can enable/disable this flag on the fly w/out wedging their systems. Bug: https://bugs.gentoo.org/650052 sys-apps/sed/metadata.xml | 6 +++++- sys-apps/sed/sed-4.4-r1.ebuild | 4 ++-- 2 files changed, 7 insertions(+), 3 deletions(-) https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=a64fc681cba91934c0ebb96fc7c2312fbcb5cef4 commit a64fc681cba91934c0ebb96fc7c2312fbcb5cef4 Author: Mike Frysinger <vapier@gentoo.org> AuthorDate: 2018-03-11 22:24:58 +0000 Commit: Mike Frysinger <vapier@gentoo.org> CommitDate: 2018-03-11 22:36:27 +0000 sys-apps/sed: update bootstrap logic #650052 The bootstrap.sh logic was deleted upstream a while back. Switch to using busybox's sed as a fallback as needed. This should be good enough for most cases, and we can figure out something more if the need comes up. Bug: https://bugs.gentoo.org/650052 sys-apps/sed/sed-4.2.2.ebuild | 10 ++++------ sys-apps/sed/sed-4.3-r1.ebuild | 10 ++++------ sys-apps/sed/sed-4.4-r1.ebuild | 10 ++++------ sys-apps/sed/sed-4.4.ebuild | 10 ++++------ 4 files changed, 16 insertions(+), 24 deletions(-)} sed can rebuild itself now, but be aware that you're going to run into similar build failures in other packages that use the r command, so you prob don't want to deploy this on your dev system where you compile packages. if ! type -p sed > /dev/null || has_version 'sys-apps/sed[forced-sandbox]' ; then Mike, are you sure this will really help? I mean, even if you have installed sed with forced-sandbox enabled, "type -p sed" will still return 0, no? Shouldn't the has_version check be first here? Nevermind, too low caffeine level this morning. Thanks for your fix Mike ;) Thanks from my side aswell, works fine now. |