* [gentoo-dev] bash-4.4 - call for testers @ 2016-09-20 11:41 Lars Wendler 2016-09-22 9:22 ` David Haller ` (2 more replies) 0 siblings, 3 replies; 33+ messages in thread From: Lars Wendler @ 2016-09-20 11:41 UTC (permalink / raw To: gentoo-dev@lists.gentoo.org [-- Attachment #1: Type: text/plain, Size: 1427 bytes --] Hi, bash-4.4/readline-7.0 have been released and are available in Gentoo already although they are still being package.masked. This is a call for testers. If you feel brave enough, please give bash-4.4 a shot and report all problems you might have with it. Add the following to you package.unmask file/directory: =app-shells/bash-4.4* =sys-libs/readline-7.0* and emerge both packages. Once readline-7.0 has been installed you most likely need to run emerge -v @preserved-rebuild in order to get all readline reverse deps being built against new readline. Reporting those packages in our bug tracker [1] would be helpful. In case you encounter a real bug in bash or readline please report them to our bug tracker [1] as well. If you just seem to have successfully installed bash and found no problems after a while of testing, I'd like to know about it here on the list. This would help me determining when this bash version can be removed from package.mask. Perhaps we even consider stabilizing bash-4.4 a bit sooner now that there's a security bug [2] that requires bash-4.4 for the fix. Thanks in advance. [1] https://bugs.gentoo.org [2] https://bugs.gentoo.org/594496 -- Lars Wendler Gentoo package maintainer GPG: 21CC CF02 4586 0A07 ED93 9F68 498F E765 960E 9B39 Attention! New gpg key! See https://www.gentoofan.org/blog/index.php?/archives/9-New-gpg-keys.html [-- Attachment #2: Digitale Signatur von OpenPGP --] [-- Type: application/pgp-signature, Size: 801 bytes --] ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-09-20 11:41 [gentoo-dev] bash-4.4 - call for testers Lars Wendler @ 2016-09-22 9:22 ` David Haller 2016-09-23 10:42 ` David Haller 2016-09-29 5:54 ` Dan Douglas 2016-10-01 21:48 ` Dan Douglas 2 siblings, 1 reply; 33+ messages in thread From: David Haller @ 2016-09-22 9:22 UTC (permalink / raw To: gentoo-dev Hello, On Tue, 20 Sep 2016, Lars Wendler wrote: >This is a call for testers. If you feel brave enough, please give >bash-4.4 a shot and report all problems you might have with it. > >Add the following to you package.unmask file/directory: > > =app-shells/bash-4.4* > =sys-libs/readline-7.0* with basically no testing so far: builds ok. >in order to get all readline reverse deps being built against new >readline. Reporting those packages in our bug tracker [1] would be >helpful. Creating a new bug or what? [2] seems inappropriate. >[2] https://bugs.gentoo.org/594496 I've found this that I guess has to do with the new readline: From the build.log of emerge -1 dev-lisp/clisp ==== [..] * Package: dev-lisp/clisp-2.49-r8 * Repository: gentoo * Maintainer: common-lisp@gentoo.org * USE: X abi_x86_64 amd64 berkdb elibc_glibc gdbm gtk kernel_linux pcre postgres readline threads unicode userland_GNU zlib * FEATURES: ccache compressdebug nostrip preserve-libs sandbox splitdebug userpriv usersandbox [..] ;; Loading file /var/tmp/portage/dev-lisp/clisp-2.49-r8/work/clisp-2.49/builddir/readline/readline.fas ... *** - FFI::FIND-FOREIGN-VARIABLE: foreign variable #<FOREIGN-VARIABLE "rl_readline_state" #x000000000088B460> does not have the required size or alignment ./clisp-link: failed in /var/tmp/portage/dev-lisp/clisp-2.49-r8/work/clisp-2.49/builddir Makefile:2621: recipe for target 'base' failed make: *** [base] Error 1 [..] ==== # equery uses -i =dev-lisp/clisp-2.49-r8 | xargs echo +X +berkdb -dbus -fastcgi +gdbm +gtk -hyperspec -pari +pcre +postgres +readline +threads +unicode +zlib So, something with 'rl_readline_state' and how clisp uses that has changed that I won't dig into right now. Clisp maintainers? That seems to be the only problem with readline-7.0/bash-4.4 on this install (so far). -dnh -- The use of COBOL cripples the mind; its teaching should, therefore, be regarded as a criminal offence. -- Edsger Wybe Dijkstra ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-09-22 9:22 ` David Haller @ 2016-09-23 10:42 ` David Haller 0 siblings, 0 replies; 33+ messages in thread From: David Haller @ 2016-09-23 10:42 UTC (permalink / raw To: gentoo-dev Hello, On Thu, 22 Sep 2016, David Haller wrote: [..] >I've found this that I guess has to do with the new readline: > >>From the build.log of > emerge -1 dev-lisp/clisp > >==== >[..] > * Package: dev-lisp/clisp-2.49-r8 > * Repository: gentoo > * Maintainer: common-lisp@gentoo.org > * USE: X abi_x86_64 amd64 berkdb elibc_glibc gdbm gtk > kernel_linux pcre postgres readline threads unicode userland_GNU > zlib > * FEATURES: ccache compressdebug nostrip preserve-libs sandbox > splitdebug userpriv usersandbox >[..] >;; Loading file /var/tmp/portage/dev-lisp/clisp-2.49-r8/work/clisp-2.49/builddir/readline/readline.fas ... >*** - FFI::FIND-FOREIGN-VARIABLE: foreign variable #<FOREIGN-VARIABLE "rl_readline_state" #x000000000088B460> does not have the > required size or alignment > >./clisp-link: failed in /var/tmp/portage/dev-lisp/clisp-2.49-r8/work/clisp-2.49/builddir >Makefile:2621: recipe for target 'base' failed >make: *** [base] Error 1 >[..] >==== [..] Got a patch for that: https://bugs.gentoo.org/show_bug.cgi?id=594552 https://bugs.gentoo.org/attachment.cgi?id=447492 -dnh -- Since attendees must wear their name tags, they must also wear shirts or blouses. Pants or skirts are also highly recommended. -- RFC 1391 ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-09-20 11:41 [gentoo-dev] bash-4.4 - call for testers Lars Wendler 2016-09-22 9:22 ` David Haller @ 2016-09-29 5:54 ` Dan Douglas 2016-09-29 21:56 ` Andy Mender 2016-09-30 17:49 ` konsolebox 2016-10-01 21:48 ` Dan Douglas 2 siblings, 2 replies; 33+ messages in thread From: Dan Douglas @ 2016-09-29 5:54 UTC (permalink / raw To: gentoo-dev The only reason I haven't been testing this for many months system-wide is it's in the same slot as 4.3, and it still is. Is it not possible to separate them? ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-09-29 5:54 ` Dan Douglas @ 2016-09-29 21:56 ` Andy Mender 2016-09-29 22:29 ` Kent Fredric 2016-09-30 17:53 ` konsolebox 2016-09-30 17:49 ` konsolebox 1 sibling, 2 replies; 33+ messages in thread From: Andy Mender @ 2016-09-29 21:56 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 777 bytes --] Yes, I also wouldn't want to sacrifice something as relevant as the default Shell interpreter. I quickly checked other GNU/Linux distributions and even on Arch Linux Bash 4.3 seems to be the newest stable version. Would it be OK to have the real stable version (now, Bash 4.3) as slot :0 and a single testing version (now, Bash 4.4) as slot:1 in addition to needing the "~amd64" flag? I believe the main problem comes from /bin/bash and potential symlinks that would need to be introduced as part of the slotting. Best regards, Andy On 29 September 2016 at 07:54, Dan Douglas <ormaaj@gmail.com> wrote: > The only reason I haven't been testing this for many months > system-wide is it's in the same slot as 4.3, and it still is. Is it > not possible to separate them? > > [-- Attachment #2: Type: text/html, Size: 1185 bytes --] ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-09-29 21:56 ` Andy Mender @ 2016-09-29 22:29 ` Kent Fredric 2016-09-30 13:38 ` William Hubbs 2016-10-01 7:47 ` Dan Douglas 2016-09-30 17:53 ` konsolebox 1 sibling, 2 replies; 33+ messages in thread From: Kent Fredric @ 2016-09-29 22:29 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 918 bytes --] On Thu, 29 Sep 2016 23:56:34 +0200 Andy Mender <andymenderunix@gmail.com> wrote: > I believe the main problem comes from /bin/bash and potential symlinks that > would need to be introduced as part of the slotting. In a pinch you could probably get away with calling :1 /usr/bin/bash-4.4 instead of /usr/bin/bash, and then offering no luxuries beyond that, leaving it up to the user to do the rest. Then you could test it in ~/ with PATH + Symlink in ~/bin/ ... maybe. There would just not be much point, because the real purpose of testing 4.4 is not for fear of it breaking user experience ( which is a problem, but not the primary motive ), but for making everything else that runs with bash runs OK. Maybe you could do some horrible QA Violation like USE=multislot which changes the slot from :0 and adds the -suffix at the same time. But I still don't think its a useful or good idea. [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 801 bytes --] ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-09-29 22:29 ` Kent Fredric @ 2016-09-30 13:38 ` William Hubbs 2016-09-30 17:09 ` Andy Mender 2016-10-01 6:59 ` Dan Douglas 2016-10-01 7:47 ` Dan Douglas 1 sibling, 2 replies; 33+ messages in thread From: William Hubbs @ 2016-09-30 13:38 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 1216 bytes --] On Fri, Sep 30, 2016 at 11:29:05AM +1300, Kent Fredric wrote: > On Thu, 29 Sep 2016 23:56:34 +0200 > Andy Mender <andymenderunix@gmail.com> wrote: > > > I believe the main problem comes from /bin/bash and potential symlinks that > > would need to be introduced as part of the slotting. > > In a pinch you could probably get away with > calling :1 /usr/bin/bash-4.4 instead of /usr/bin/bash, and then > offering no luxuries beyond that, leaving it up to the user to do the rest. > > Then you could test it in ~/ with PATH + Symlink in ~/bin/ ... maybe. > > There would just not be much point, because the real purpose of testing > 4.4 is not for fear of it breaking user experience ( which is a > problem, but not the primary motive ), but for making everything else > that runs with bash runs OK. > > Maybe you could do some horrible QA Violation like USE=multislot > which changes the slot from :0 and adds the -suffix at the same time. > > But I still don't think its a useful or good idea. I am against it as well. The purpose of this testing is to eventually move to bash-4.4 being stable and replacing bash-4.3, so slotting it would make that more complex later. William [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 181 bytes --] ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-09-30 13:38 ` William Hubbs @ 2016-09-30 17:09 ` Andy Mender 2016-10-01 6:59 ` Dan Douglas 1 sibling, 0 replies; 33+ messages in thread From: Andy Mender @ 2016-09-30 17:09 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 1440 bytes --] Alright then, I will give bash4.4 a try and if there are any problems, I'll file a proper bug report. Best regards, Andy On 30 September 2016 at 15:38, William Hubbs <williamh@gentoo.org> wrote: > On Fri, Sep 30, 2016 at 11:29:05AM +1300, Kent Fredric wrote: > > On Thu, 29 Sep 2016 23:56:34 +0200 > > Andy Mender <andymenderunix@gmail.com> wrote: > > > > > I believe the main problem comes from /bin/bash and potential symlinks > that > > > would need to be introduced as part of the slotting. > > > > In a pinch you could probably get away with > > calling :1 /usr/bin/bash-4.4 instead of /usr/bin/bash, and then > > offering no luxuries beyond that, leaving it up to the user to do the > rest. > > > > Then you could test it in ~/ with PATH + Symlink in ~/bin/ ... maybe. > > > > There would just not be much point, because the real purpose of testing > > 4.4 is not for fear of it breaking user experience ( which is a > > problem, but not the primary motive ), but for making everything else > > that runs with bash runs OK. > > > > Maybe you could do some horrible QA Violation like USE=multislot > > which changes the slot from :0 and adds the -suffix at the same time. > > > > But I still don't think its a useful or good idea. > > I am against it as well. The purpose of this testing is to eventually > move to bash-4.4 being stable and replacing bash-4.3, so slotting it > would make that more complex later. > > William > > [-- Attachment #2: Type: text/html, Size: 2089 bytes --] ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-09-30 13:38 ` William Hubbs 2016-09-30 17:09 ` Andy Mender @ 2016-10-01 6:59 ` Dan Douglas 1 sibling, 0 replies; 33+ messages in thread From: Dan Douglas @ 2016-10-01 6:59 UTC (permalink / raw To: gentoo-dev > I am against it as well. The purpose of this testing is to eventually > move to bash-4.4 being stable and replacing bash-4.3, so slotting it > would make that more complex later. Only true for users that have no interest in testing. Irrelevant for anyone that needs to test for backwards compatibility or cross-platform support in scripts. That's the only reason to support multiple slots of this in the first place. It doesn't imply one should substitute old versions for their system shell. ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-09-29 22:29 ` Kent Fredric 2016-09-30 13:38 ` William Hubbs @ 2016-10-01 7:47 ` Dan Douglas 1 sibling, 0 replies; 33+ messages in thread From: Dan Douglas @ 2016-10-01 7:47 UTC (permalink / raw To: gentoo-dev On Thu, Sep 29, 2016 at 5:29 PM, Kent Fredric <kentnl@gentoo.org> wrote: > Maybe you could do some horrible QA Violation like USE=multislot > which changes the slot from :0 and adds the -suffix at the same time. > > But I still don't think its a useful or good idea. Really? Why is this a problem? Some issue to do with the symlink involving chroots or network filesystems I'm not considering? Looks like every version ends up in /bin anyway so you can make /bin/bash another hardlink that's a synonym for /bin/bash-suffix. I can't think of a way to screw that up other than perhaps by doing something weird with overlayfs. ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-09-29 21:56 ` Andy Mender 2016-09-29 22:29 ` Kent Fredric @ 2016-09-30 17:53 ` konsolebox 1 sibling, 0 replies; 33+ messages in thread From: konsolebox @ 2016-09-30 17:53 UTC (permalink / raw To: gentoo-dev On Fri, Sep 30, 2016 at 5:56 AM, Andy Mender <andymenderunix@gmail.com> wrote: > Would it be OK to have the real stable version (now, Bash 4.3) as slot :0 > and a single testing version (now, Bash 4.4) as slot:1 in addition to > needing the "~amd64" flag? Or simply 4.3 and 4.4, not 0 and 1. And then 5.0 for the next development version, or just 9999. -- konsolebox ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-09-29 5:54 ` Dan Douglas 2016-09-29 21:56 ` Andy Mender @ 2016-09-30 17:49 ` konsolebox 2016-10-01 0:38 ` Kent Fredric 1 sibling, 1 reply; 33+ messages in thread From: konsolebox @ 2016-09-30 17:49 UTC (permalink / raw To: gentoo-dev On Thu, Sep 29, 2016 at 1:54 PM, Dan Douglas <ormaaj@gmail.com> wrote: > The only reason I haven't been testing this for many months > system-wide is it's in the same slot as 4.3, and it still is. Is it > not possible to separate them? That would also be helpful to me since I closely monitor the development of Bash. It would be nice to have some eselect command to easily switch from one version of Bash to another; probably something close to how it's done in dev-lang/ruby. -- konsolebox ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-09-30 17:49 ` konsolebox @ 2016-10-01 0:38 ` Kent Fredric 2016-10-01 14:38 ` konsolebox 0 siblings, 1 reply; 33+ messages in thread From: Kent Fredric @ 2016-10-01 0:38 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 353 bytes --] On Sat, 1 Oct 2016 01:49:56 +0800 konsolebox <konsolebox@gmail.com> wrote: > It would be nice to have some eselect command to > easily switch from one version of Bash to another; probably something > close to how it's done in dev-lang/ruby. Its just eselect itself is bash. So if bash is broken ..... switching out becomes impossible anyway. [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 801 bytes --] ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-01 0:38 ` Kent Fredric @ 2016-10-01 14:38 ` konsolebox 2016-10-01 15:40 ` konsolebox 0 siblings, 1 reply; 33+ messages in thread From: konsolebox @ 2016-10-01 14:38 UTC (permalink / raw To: gentoo-dev On Sat, Oct 1, 2016 at 8:38 AM, Kent Fredric <kentnl@gentoo.org> wrote: > On Sat, 1 Oct 2016 01:49:56 +0800 > konsolebox <konsolebox@gmail.com> wrote: > >> It would be nice to have some eselect command to >> easily switch from one version of Bash to another; probably something >> close to how it's done in dev-lang/ruby. > > Its just eselect itself is bash. So if bash is broken ..... switching out becomes > impossible anyway. Good point, but that only applies if bash can become as broken as how you imagine it. I had tested many development versions of bash and none of them became that unusable, especially if it only comes to simple scripts like eselect. It's even more unlikely to happen with incremental releases (beta, rc, etc.). -- konsolebox ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-01 14:38 ` konsolebox @ 2016-10-01 15:40 ` konsolebox 2016-10-01 16:34 ` Dan Douglas 0 siblings, 1 reply; 33+ messages in thread From: konsolebox @ 2016-10-01 15:40 UTC (permalink / raw To: gentoo-dev I should also add that testing newer versions of bash becomes more risky sometimes, since bash would sometimes need a newer version of readline (see https://lists.gnu.org/archive/html/bug-bash/2016-09/msg00020.html). Rebuilding or installing a newer version of readline would cause some other packages that depend on it to be rebuilt as well. If bash and readline become multi-slotted (or shared if there's a difference), it would be easier to test them. Other stuff would also not need to be rebuilt immediately. -- konsolebox ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-01 15:40 ` konsolebox @ 2016-10-01 16:34 ` Dan Douglas 2016-10-02 5:28 ` konsolebox 0 siblings, 1 reply; 33+ messages in thread From: Dan Douglas @ 2016-10-01 16:34 UTC (permalink / raw To: gentoo-dev On Sat, Oct 1, 2016 at 10:40 AM, konsolebox <konsolebox@gmail.com> wrote: > I should also add that testing newer versions of bash becomes more > risky sometimes, since bash would sometimes need a newer version of > readline (see https://lists.gnu.org/archive/html/bug-bash/2016-09/msg00020.html). > Rebuilding or installing a newer version of readline would cause some > other packages that depend on it to be rebuilt as well. > > If bash and readline become multi-slotted (or shared if there's a > difference), it would be easier to test them. Other stuff would also > not need to be rebuilt immediately. AFAIK upstream doesn't technically support unbundled readline anyway. If newer readline isn't tested with older bashes it would be miraculous for it to be backwards-compatible with every bash version in the tree. I'd be perfectly happy requiring bundled readline when USE="readline" for bash versions incompatible with the installed readline, or simply depending on USE="-readline" for those versions. I rarely if ever use interactive mode with anything other than my system default /bin/bash. ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-01 16:34 ` Dan Douglas @ 2016-10-02 5:28 ` konsolebox 2016-10-02 5:40 ` Kent Fredric 2016-10-02 6:29 ` konsolebox 0 siblings, 2 replies; 33+ messages in thread From: konsolebox @ 2016-10-02 5:28 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 1064 bytes --] On Sun, Oct 2, 2016 at 12:34 AM, Dan Douglas <ormaaj@gmail.com> wrote: > I'd be perfectly happy requiring bundled readline when USE="readline" > for bash versions incompatible with the installed readline, I guess that's another good way to solve the readline issue (when it comes to bash). But I'd prefer that it's not done automatically. Instead we should add a formal use flag like 'installed-readline'. We can add it to release versions of bash ([[ ${PV} != *_rc* ]] && IUSE+=' +installed-readline'), and enable it by default. Then we change all `[[ ${PV} != *_rc* ]]` condition checks to `[[ ${PV} != *_rc* ]] && use installed-readline`. `${PV} != *_rc*` probably should also be `${PV} != *_alpha* && ${PV} != *_beta* && ${PV} != *_rc*`. (See attached file for POC.) > or simply depending on USE="-readline" for those versions. > I rarely if ever use > interactive mode with anything other than my system default /bin/bash. I do, though. My application uses `read -e`. (That's not interactive mode I know, but it still uses readline.) -- konsolebox [-- Attachment #2: bash-4.4.ebuild --] [-- Type: application/octet-stream, Size: 7430 bytes --] # Copyright 1999-2016 Gentoo Foundation # Distributed under the terms of the GNU General Public License v2 # $Id$ EAPI="5" inherit eutils flag-o-matic toolchain-funcs multilib # Official patchlevel # See ftp://ftp.cwru.edu/pub/bash/bash-4.3-patches/ PLEVEL=${PV##*_p} MY_PV=${PV/_p*} MY_PV=${MY_PV/_/-} MY_P=${PN}-${MY_PV} [[ ${PV} != *_p* ]] && PLEVEL=0 patches() { local opt=$1 plevel=${2:-${PLEVEL}} pn=${3:-${PN}} pv=${4:-${MY_PV}} [[ ${plevel} -eq 0 ]] && return 1 eval set -- {1..${plevel}} set -- $(printf "${pn}${pv/\.}-%03d " "$@") if [[ ${opt} == -s ]] ; then echo "${@/#/${DISTDIR}/}" else local u for u in ftp://ftp.cwru.edu/pub/bash mirror://gnu/${pn} ; do printf "${u}/${pn}-${pv}-patches/%s " "$@" done fi } # The version of readline this bash normally ships with. READLINE_VER="7.0_rc2" IS_A_RELEASE=false [[ ${PV} != *_alpha* && ${PV} != *_beta* && ${PV} != *_rc* ]] && IS_A_RELEASE=true DESCRIPTION="The standard GNU Bourne again shell" HOMEPAGE="http://tiswww.case.edu/php/chet/bash/bashtop.html" if [[ ${IS_A_RELEASE} == true ]]; then SRC_URI="mirror://gnu/bash/${MY_P}.tar.gz $(patches)" else SRC_URI+=" ftp://ftp.cwru.edu/pub/bash/${MY_P}.tar.gz" fi LICENSE="GPL-3" SLOT="0" KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~hppa ~ia64 ~m68k ~mips ~ppc ~ppc64 ~s390 ~sh ~sparc ~x86 ~amd64-fbsd ~sparc-fbsd ~x86-fbsd" IUSE="afs bashlogger examples mem-scramble +net nls plugins +readline tinfo vanilla" [[ ${IS_A_RELEASE} == true ]] && IUSE+=' +installed_readline' DEPEND=">=sys-libs/ncurses-5.2-r2:0= readline? ( >=sys-libs/readline-${READLINE_VER}:0= ) tinfo? ( sys-libs/ncurses[tinfo] ) !tinfo? ( !sys-libs/ncurses[tinfo] ) nls? ( virtual/libintl )" RDEPEND="${DEPEND} !<sys-apps/portage-2.1.6.7_p1 !<sys-apps/paludis-0.26.0_alpha5" # we only need yacc when the .y files get patched (bash42-005) DEPEND+=" virtual/yacc" S=${WORKDIR}/${MY_P} pkg_setup() { if is-flag -malign-double ; then #7332 eerror "Detected bad CFLAGS '-malign-double'. Do not use this" eerror "as it breaks LFS (struct stat64) on x86." die "remove -malign-double from your CFLAGS mr ricer" fi if use bashlogger ; then ewarn "The logging patch should ONLY be used in restricted (i.e. honeypot) envs." ewarn "This will log ALL output you enter into the shell, you have been warned." fi use tinfo && export LDFLAGS="-ltinfo ${LDFLAGS}" LIBS="-ltinfo ${LIBS}" } src_unpack() { unpack ${MY_P}.tar.gz } src_prepare() { # Include official patches [[ ${PLEVEL} -gt 0 ]] && epatch $(patches -s) # Clean out local libs so we know we use system ones w/releases. if [[ ${IS_A_RELEASE} == true ]] && use installed_readline; then rm -rf lib/{readline,termcap}/* touch lib/{readline,termcap}/Makefile.in # for config.status sed -ri -e 's:\$[(](RL|HIST)_LIBSRC[)]/[[:alpha:]]*.h::g' Makefile.in || die fi # Avoid regenerating docs after patches #407985 sed -i -r '/^(HS|RL)USER/s:=.*:=:' doc/Makefile.in || die touch -r . doc/* epatch_user } src_configure() { local myconf=() # For descriptions of these, see config-top.h # bashrc/#26952 bash_logout/#90488 ssh/#24762 mktemp/#574426 append-cppflags \ -DDEFAULT_PATH_VALUE=\'\"${EPREFIX}/usr/local/sbin:${EPREFIX}/usr/local/bin:${EPREFIX}/usr/sbin:${EPREFIX}/usr/bin:${EPREFIX}/sbin:${EPREFIX}/bin\"\' \ -DSTANDARD_UTILS_PATH=\'\"${EPREFIX}/bin:${EPREFIX}/usr/bin:${EPREFIX}/sbin:${EPREFIX}/usr/sbin\"\' \ -DSYS_BASHRC=\'\"${EPREFIX}/etc/bash/bashrc\"\' \ -DSYS_BASH_LOGOUT=\'\"${EPREFIX}/etc/bash/bash_logout\"\' \ -DNON_INTERACTIVE_LOGIN_SHELLS \ -DSSH_SOURCE_BASHRC \ $(use bashlogger && echo -DSYSLOG_HISTORY) # Don't even think about building this statically without # reading Bug 7714 first. If you still build it statically, # don't come crying to us with bugs ;). #use static && export LDFLAGS="${LDFLAGS} -static" use nls || myconf+=( --disable-nls ) # Historically, we always used the builtin readline, but since # our handling of SONAME upgrades has gotten much more stable # in the PM (and the readline ebuild itself preserves the old # libs during upgrades), linking against the system copy should # be safe. # Exact cached version here doesn't really matter as long as it # is at least what's in the DEPEND up above. export ac_cv_rl_version=${READLINE_VER%%_*} # Force linking with system curses ... the bundled termcap lib # sucks bad compared to ncurses. For the most part, ncurses # is here because readline needs it. But bash itself calls # ncurses in one or two small places :(. if [[ ${IS_A_RELEASE} == true ]] && use installed_readline; then # Use system readline only with released versions. myconf+=( --with-installed-readline=. ) fi if use plugins; then append-ldflags -Wl,-rpath,/usr/$(get_libdir)/bash else # Disable the plugins logic by hand since bash doesn't # provide a way of doing it. export ac_cv_func_dl{close,open,sym}=no \ ac_cv_lib_dl_dlopen=no ac_cv_header_dlfcn_h=no sed -i \ -e '/LOCAL_LDFLAGS=/s:-rdynamic::' \ configure || die fi tc-export AR #444070 econf \ --docdir='$(datarootdir)'/doc/${PF} \ --htmldir='$(docdir)/html' \ --with-curses \ $(use_with afs) \ $(use_enable net net-redirections) \ --disable-profiling \ $(use_enable mem-scramble) \ $(use_with mem-scramble bash-malloc) \ $(use_enable readline) \ $(use_enable readline history) \ $(use_enable readline bang-history) \ "${myconf[@]}" } src_compile() { emake if use plugins ; then emake -C examples/loadables all others fi } src_install() { local d f default dodir /bin mv "${ED}"/usr/bin/bash "${ED}"/bin/ || die dosym bash /bin/rbash insinto /etc/bash doins "${FILESDIR}"/bash_logout doins "${FILESDIR}"/bashrc keepdir /etc/bash/bashrc.d insinto /etc/skel for f in bash{_logout,_profile,rc} ; do newins "${FILESDIR}"/dot-${f} .${f} done local sed_args=( -e "s:#${USERLAND}#@::" -e '/#@/d' ) if ! use readline ; then sed_args+=( #432338 -e '/^shopt -s histappend/s:^:#:' -e 's:use_color=true:use_color=false:' ) fi sed -i \ "${sed_args[@]}" \ "${ED}"/etc/skel/.bashrc \ "${ED}"/etc/bash/bashrc || die if use plugins ; then exeinto /usr/$(get_libdir)/bash doexe $(echo examples/loadables/*.o | sed 's:\.o::g') insinto /usr/include/bash-plugins doins *.h builtins/*.h include/*.h lib/{glob/glob.h,tilde/tilde.h} fi if use examples ; then for d in examples/{functions,misc,scripts,startup-files} ; do exeinto /usr/share/doc/${PF}/${d} insinto /usr/share/doc/${PF}/${d} for f in ${d}/* ; do if [[ ${f##*/} != PERMISSION ]] && [[ ${f##*/} != *README ]] ; then doexe ${f} else doins ${f} fi done done fi doman doc/*.1 newdoc CWRU/changelog ChangeLog dosym bash.info /usr/share/info/bashref.info } pkg_preinst() { if [[ -e ${EROOT}/etc/bashrc ]] && [[ ! -d ${EROOT}/etc/bash ]] ; then mkdir -p "${EROOT}"/etc/bash mv -f "${EROOT}"/etc/bashrc "${EROOT}"/etc/bash/ fi if [[ -L ${EROOT}/bin/sh ]] ; then # rewrite the symlink to ensure that its mtime changes. having /bin/sh # missing even temporarily causes a fatal error with paludis. local target=$(readlink "${EROOT}"/bin/sh) local tmp=$(emktemp "${EROOT}"/bin) ln -sf "${target}" "${tmp}" mv -f "${tmp}" "${EROOT}"/bin/sh fi } pkg_postinst() { # If /bin/sh does not exist, provide it if [[ ! -e ${EROOT}/bin/sh ]] ; then ln -sf bash "${EROOT}"/bin/sh fi } ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-02 5:28 ` konsolebox @ 2016-10-02 5:40 ` Kent Fredric 2016-10-02 8:03 ` konsolebox 2016-10-02 6:29 ` konsolebox 1 sibling, 1 reply; 33+ messages in thread From: Kent Fredric @ 2016-10-02 5:40 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 971 bytes --] On Sun, 2 Oct 2016 13:28:04 +0800 konsolebox <konsolebox@gmail.com> wrote: > I guess that's another good way to solve the readline issue (when it > comes to bash). But I'd prefer that it's not done automatically. > Instead we should add a formal use flag like 'installed-readline'. We > can add it to release versions of bash ([[ ${PV} != *_rc* ]] && > IUSE+=' +installed-readline'), and enable it by default. Then we > change all `[[ ${PV} != *_rc* ]]` condition checks to `[[ ${PV} != > *_rc* ]] && use installed-readline`. `${PV} != *_rc*` probably should > also be `${PV} != *_alpha* && ${PV} != *_beta* && ${PV} != *_rc*`. > (See attached file for POC.) Conventionally, it would be better to have IUSE=" +system-readline" Though I'd probably opt for IUSE=" system-readline" As the default for development versions. if [[ ${PV} != *_rc* ]]; then IUSE+=" system-readline" else; IUSE+=" +system-readline" fi [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 801 bytes --] ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-02 5:40 ` Kent Fredric @ 2016-10-02 8:03 ` konsolebox 2016-10-02 8:58 ` Kent Fredric 0 siblings, 1 reply; 33+ messages in thread From: konsolebox @ 2016-10-02 8:03 UTC (permalink / raw To: gentoo-dev On Sun, Oct 2, 2016 at 1:40 PM, Kent Fredric <kentnl@gentoo.org> wrote: > On Sun, 2 Oct 2016 13:28:04 +0800 > konsolebox <konsolebox@gmail.com> wrote: > >> I guess that's another good way to solve the readline issue (when it >> comes to bash). But I'd prefer that it's not done automatically. >> Instead we should add a formal use flag like 'installed-readline'. We >> can add it to release versions of bash ([[ ${PV} != *_rc* ]] && >> IUSE+=' +installed-readline'), and enable it by default. Then we >> change all `[[ ${PV} != *_rc* ]]` condition checks to `[[ ${PV} != >> *_rc* ]] && use installed-readline`. `${PV} != *_rc*` probably should >> also be `${PV} != *_alpha* && ${PV} != *_beta* && ${PV} != *_rc*`. >> (See attached file for POC.) > > Conventionally, it would be better to have IUSE=" +system-readline" > > Though I'd probably opt for > > IUSE=" system-readline" > > As the default for development versions. > > > if [[ ${PV} != *_rc* ]]; then > IUSE+=" system-readline" > else; > IUSE+=" +system-readline" > fi I actually don't like the idea of enabling or disabling "installed-readline" based on the `${PV} != *_rc*` condition. If a user would want to explicitly enable "installed-readline" globally, how would he make sure that it only affects the release version of bash? I suggest that we just don't entertain the flag (and not make it available) if the ebuild is not targetting a release version, and not use the readline installed by the system by default. No one would need a non-release version of bash compiled against a system readline anyway. I also thought using 'system-' prefix is confusing. Does that mean the system of the machine, or the system of the application? In firefox, I once thought that system-* means packages bundled within it. Perhaps I misread, or perhaps it was changed. At that time, descriptions of use flags were not widely provided yet so I could only guess it based on how I built the packages. But I don't really mind which one is used. I also just thought 'installed-readline' is better since it configures `--with-installed-readline`, so one can simply have `$(use_enable installed-readline)` if applicable. -- konsolebox ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-02 8:03 ` konsolebox @ 2016-10-02 8:58 ` Kent Fredric 2016-10-02 10:00 ` konsolebox 0 siblings, 1 reply; 33+ messages in thread From: Kent Fredric @ 2016-10-02 8:58 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 2748 bytes --] On Sun, 2 Oct 2016 16:03:11 +0800 konsolebox <konsolebox@gmail.com> wrote: > I actually don't like the idea of enabling or disabling > "installed-readline" based on the `${PV} != *_rc*` condition. If a > user would want to explicitly enable "installed-readline" globally, > how would he make sure that it only affects the release version of > bash? I suggest that we just don't entertain the flag (and not make > it available) if the ebuild is not targetting a release version, and > not use the readline installed by the system by default. No one would > need a non-release version of bash compiled against a system readline > anyway. That's not what that does. It doesn't enable or disable the mechanic, the code I offered only changes the default for _rc. That is, on _rc the default is "use bundled readline implementation", and on non_rc, the default is "use pre-existing readline implementation". if the user specifies an explicit USE="system-readline" in make.conf or /etc/portage/package.use, then this "default" gets overridden and the system readline implementation is always used. if the user speciifies an explicit USE="-system-readline" in make.conf or /etc/portage/package.use, then this "default" gets overriden and the inbuilt readline is awlays used. Explicitly defined configuration always trumps IUSE. The "+" prefix in IUSE only specifies that the USE flag defaults to on, as opposed to defaulting off. > > I also thought using 'system-' prefix is confusing. Does that mean > the system of the machine, or the system of the application? In > firefox, I once thought that system-* means packages bundled within > it. Perhaps I misread, or perhaps it was changed. At that time, > descriptions of use flags were not widely provided yet so I could only > guess it based on how I built the packages. The system of the machine. As in, "Operating System". Always. "system-foo" usually means "foo is bundled, and toggling this flag toggles between using the bundled foo, or the system foo. > > But I don't really mind which one is used. I also just thought > 'installed-readline' is better since it configures > `--with-installed-readline`, so one can simply have `$(use_enable > installed-readline)` if applicable. 'use_enable' takes a USE flag and a configure token anyway. So: $(use_enable system-readline installed-readline) is equivalent to: use system-readline && myconf+=" --enable-installed-readline" But you probably wanted "use_with" ( man 5 ebuild ) $(use_with system-readline installed-readline) Which is instead equivalent to use system-readline && myconf+=" --with-installed-readline" [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 801 bytes --] ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-02 8:58 ` Kent Fredric @ 2016-10-02 10:00 ` konsolebox 2016-10-02 10:18 ` konsolebox 0 siblings, 1 reply; 33+ messages in thread From: konsolebox @ 2016-10-02 10:00 UTC (permalink / raw To: gentoo-dev On Sun, Oct 2, 2016 at 4:58 PM, Kent Fredric <kentnl@gentoo.org> wrote: > On Sun, 2 Oct 2016 16:03:11 +0800 > konsolebox <konsolebox@gmail.com> wrote: > >> I actually don't like the idea of enabling or disabling >> "installed-readline" based on the `${PV} != *_rc*` condition. If a >> user would want to explicitly enable "installed-readline" globally, >> how would he make sure that it only affects the release version of >> bash? I suggest that we just don't entertain the flag (and not make >> it available) if the ebuild is not targetting a release version, and >> not use the readline installed by the system by default. No one would >> need a non-release version of bash compiled against a system readline >> anyway. > > That's not what that does. It doesn't enable or disable the mechanic, > the code I offered only changes the default for _rc. > > That is, on _rc the default is "use bundled readline implementation", > and on non_rc, the default is "use pre-existing readline > implementation". > > if the user specifies an explicit > > USE="system-readline" > > in make.conf or /etc/portage/package.use, then this "default" gets > overridden and the system readline implementation is always used. > > if the user speciifies an explicit > > USE="-system-readline" > > in make.conf or /etc/portage/package.use, then this "default" gets > overriden and the inbuilt readline is awlays used. > > Explicitly defined configuration always trumps IUSE. The "+" prefix in > IUSE only specifies that the USE flag defaults to on, as opposed to > defaulting off. I understand those elementary stuff, but my point was about whether we should allow the user to configure 'system-readline' on a non-release version of bash (i.e. make the flag available for them to configure). Your point is that instead of statically deciding whether we shouldn't use `--with-installed-readline` through `${PV} != *_rc*`, we instead base it on the flag, which means we allow the users to change the behavior of -non-release- versions of bash, which is, compiling against the bundled readline, to compiling against the system readline instead. How would a user be able to apply a distinguishing global configuration which would apply differently when the installing bash version is a release version, and when it's not? You might say that `system-readline` is enabled by default in a release version anyway, so specifying `system-readline` explicitly is meaningless and should only be done if you want to change the behavior of non-release versions of bash. But that's not the point. It's about the consistency. When we specify use flags in /etc/portage/make.conf, it shouldn't depend on the current default "enability" of a package's use flag. Also, do you think there could be a helpful case that one would install a non-release version of bash that compiles against the system readline? Perhaps if you're also brave enough to install an pre-release version of readline to the system, there is. Again, I say that it's better that the 'installed-readline' flag or 'system-readline' flag is only made available in the release versions of bash. And we also don't want to make it a dummy where it is just ignored and not excluded. >> I also thought using 'system-' prefix is confusing. Does that mean >> the system of the machine, or the system of the application? In >> firefox, I once thought that system-* means packages bundled within >> it. Perhaps I misread, or perhaps it was changed. At that time, >> descriptions of use flags were not widely provided yet so I could only >> guess it based on how I built the packages. > > The system of the machine. As in, "Operating System". Always. > "system-foo" usually means "foo is bundled, and toggling this flag > toggles between using the bundled foo, or the system foo. Yes, but I'm not talking about how it is now. But nevermind. >> But I don't really mind which one is used. I also just thought >> 'installed-readline' is better since it configures >> `--with-installed-readline`, so one can simply have `$(use_enable >> installed-readline)` if applicable. > > 'use_enable' takes a USE flag and a configure token anyway. > > So: > > $(use_enable system-readline installed-readline) > > is equivalent to: > > use system-readline && myconf+=" --enable-installed-readline" I know, but like I said, "simply". `installed-readline` is just more relative. But I don't really care whatever gets used. -- konsolebox ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-02 10:00 ` konsolebox @ 2016-10-02 10:18 ` konsolebox 2016-10-02 11:42 ` Kent Fredric 0 siblings, 1 reply; 33+ messages in thread From: konsolebox @ 2016-10-02 10:18 UTC (permalink / raw To: gentoo-dev On Sun, Oct 2, 2016 at 6:00 PM, konsolebox <konsolebox@gmail.com> wrote: > On Sun, Oct 2, 2016 at 4:58 PM, Kent Fredric <kentnl@gentoo.org> wrote: >> On Sun, 2 Oct 2016 16:03:11 +0800 >> konsolebox <konsolebox@gmail.com> wrote: >> >>> I actually don't like the idea of enabling or disabling >>> "installed-readline" based on the `${PV} != *_rc*` condition. If a >>> user would want to explicitly enable "installed-readline" globally, >>> how would he make sure that it only affects the release version of >>> bash? I suggest that we just don't entertain the flag (and not make >>> it available) if the ebuild is not targetting a release version, and >>> not use the readline installed by the system by default. No one would >>> need a non-release version of bash compiled against a system readline >>> anyway. >> >> That's not what that does. It doesn't enable or disable the mechanic, >> the code I offered only changes the default for _rc. >> >> That is, on _rc the default is "use bundled readline implementation", >> and on non_rc, the default is "use pre-existing readline >> implementation". >> >> if the user specifies an explicit >> >> USE="system-readline" >> >> in make.conf or /etc/portage/package.use, then this "default" gets >> overridden and the system readline implementation is always used. >> >> if the user speciifies an explicit >> >> USE="-system-readline" >> >> in make.conf or /etc/portage/package.use, then this "default" gets >> overriden and the inbuilt readline is awlays used. >> >> Explicitly defined configuration always trumps IUSE. The "+" prefix in >> IUSE only specifies that the USE flag defaults to on, as opposed to >> defaulting off. > > I understand those elementary stuff, but my point was about whether we > should allow the user to configure 'system-readline' on a non-release > version of bash (i.e. make the flag available for them to configure). > Your point is that instead of statically deciding whether we shouldn't > use `--with-installed-readline` through `${PV} != *_rc*`, we instead > base it on the flag, which means we allow the users to change the > behavior of -non-release- versions of bash, which is, compiling > against the bundled readline, to compiling against the system readline > instead. > > How would a user be able to apply a distinguishing global > configuration which would apply differently when the installing bash > version is a release version, and when it's not? You might say that > `system-readline` is enabled by default in a release version anyway, > so specifying `system-readline` explicitly is meaningless and should > only be done if you want to change the behavior of non-release > versions of bash. But that's not the point. It's about the > consistency. When we specify use flags in /etc/portage/make.conf, it > shouldn't depend on the current default "enability" of a package's use > flag. > > Also, do you think there could be a helpful case that one would > install a non-release version of bash that compiles against the system > readline? Perhaps if you're also brave enough to install an > pre-release version of readline to the system, there is. > > Again, I say that it's better that the 'installed-readline' flag or > 'system-readline' flag is only made available in the release versions > of bash. And we also don't want to make it a dummy where it is just > ignored and not excluded. I should also add that a dynamic "default" that varies depending on the version doesn't sound good to me. For one at least, it confuses the user. -- konsolebox ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-02 10:18 ` konsolebox @ 2016-10-02 11:42 ` Kent Fredric 2016-10-03 5:32 ` konsolebox 0 siblings, 1 reply; 33+ messages in thread From: Kent Fredric @ 2016-10-02 11:42 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 1706 bytes --] On Sun, 2 Oct 2016 18:18:17 +0800 konsolebox <konsolebox@gmail.com> wrote: > I should also add that a dynamic "default" that varies depending on > the version doesn't sound good to me. For one at least, it confuses > the user. I agree that its a bit unintuitive. However, the alternatives are: - A useflag that entirely goes away depending on the version - A useflag that is inoperative depending on the version Neither of those are improvements. And in both cases they're additionally messy as they require additional logic that changes what DEPEND is based on the version. > Also, do you think there could be a helpful case that one would > install a non-release version of bash that compiles against the system > readline? Perhaps if you're also brave enough to install an > pre-release version of readline to the system, there is. If this scenario was the expected scenario for non-rc releases, its only sensible that the development versions should be testing that usecase. If for example the development versions always only tested using their bundled readlines, and then the non-development versions always used dependencies, then testing is somewhat pointless. Because you're no longer testing for real world problems that would be possible due to using systemized dependenices.( ie: stipulating a new enough version, incompatibilities due to gentoo patching, etc ) "don't use external readline" would have to be the default of bash and everyone would have to be being encouraged to be using it that way in order for making the testing of that combination also a default. Otherwise you're testing a situation that will never be a reality. [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 801 bytes --] ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-02 11:42 ` Kent Fredric @ 2016-10-03 5:32 ` konsolebox 2016-10-03 5:37 ` konsolebox 2016-10-03 5:53 ` Kent Fredric 0 siblings, 2 replies; 33+ messages in thread From: konsolebox @ 2016-10-03 5:32 UTC (permalink / raw To: gentoo-dev On Sun, Oct 2, 2016 at 7:42 PM, Kent Fredric <kentnl@gentoo.org> wrote: > On Sun, 2 Oct 2016 18:18:17 +0800 > konsolebox <konsolebox@gmail.com> wrote: > >> I should also add that a dynamic "default" that varies depending on >> the version doesn't sound good to me. For one at least, it confuses >> the user. > > I agree that its a bit unintuitive. > > However, the alternatives are: > > - A useflag that entirely goes away depending on the version > - A useflag that is inoperative depending on the version > > Neither of those are improvements. "A useflag that entirely goes away depending on the version" (or a flag that is implemented in one ebuild but is not on another) is pretty common among packages and I see that as totally valid, and is way better than a solution that uses dynamic default. > And in both cases they're additionally messy as they require > additional logic that changes what DEPEND is based on the version. Doesn't look so messy to me. My solution is pretty clean and understandable. It would only depend on how it is perceived by the reading coder. The bash ebuild was also already hacky enough. Maybe you're just being conservative because it doesn't always happen in ebuilds. >> Also, do you think there could be a helpful case that one would >> install a non-release version of bash that compiles against the system >> readline? Perhaps if you're also brave enough to install an >> pre-release version of readline to the system, there is. > > If this scenario was the expected scenario for non-rc releases, its only > sensible that the development versions should be testing that usecase. > > If for example the development versions always only tested using their bundled > readlines, and then the non-development versions always used dependencies, > then testing is somewhat pointless. Pointless for bash, or applications that depend on readline as well? Because if it's only about bash, I see no difference. If someone would want to try a pre-release readline, nothing would stop him from doing it. > Because you're no longer testing for real world problems that would be possible > due to using systemized dependenices.( ie: stipulating a new enough version, > incompatibilities due to gentoo patching, etc ) If some maintainer would really want a pre-release bash tested against an external pre-release readline, he's free to modify the ebuild to work that way - just like how he uses to. We can even add an internal non-importing custom variable so this could be easily configured. But I don't think it's necessary to allow end-users to have that kind of feature as well. > "don't use external readline" would have to be the default of bash and > everyone would have to be being encouraged to be using it that way in order > for making the testing of that combination also a default. Ok I don't -really- see that as a bad alternative. The only question is, would everyone want that? And it still doesn't avoid the issue of users not being able to make 'system-readline' only apply to release versions of bash, once they enable it globally. > Otherwise you're testing a situation that will never be a reality. The difference in our view is whether we should allow users to test if a -pre-release- bash would link and run well against a pre-release readline, or not. That really isn't something to be concerned of, if you consider that release versions would be tested anyway. Getting those will-link-or-not-and-run-properly cases for pre-release versions tested by developers should be enough. -- konsolebox ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-03 5:32 ` konsolebox @ 2016-10-03 5:37 ` konsolebox 2016-10-03 5:47 ` konsolebox 2016-10-03 5:53 ` Kent Fredric 1 sibling, 1 reply; 33+ messages in thread From: konsolebox @ 2016-10-03 5:37 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 785 bytes --] I examined both RC versions of bash and readline, and it turns out that the KEYWORDS are disabled in those. If that is -always- the case, we could just have IUSE='+system-readline' in any version regardless if it's a release or not, since users would not be able to install it anyway, unless they apply 'app-shells/bash **' to /etc/portage/package.keywords. If they do, then that means they're prepared for the risk, and should know when to use the use flag. I created another example for this. This time, the feature can still be disabled, but only if there is no counter-package of readline for the targetted version of bash, which may happen in alpha or beta versions; not to mention 9999 (and 99999), which I believe should be easy to implement now after this. -- konsolebox [-- Attachment #2: bash-4.4-r1.ebuild --] [-- Type: application/octet-stream, Size: 7685 bytes --] # Copyright 1999-2016 Gentoo Foundation # Distributed under the terms of the GNU General Public License v2 # $Id$ EAPI="5" inherit eutils flag-o-matic toolchain-funcs multilib # Official patchlevel # See ftp://ftp.cwru.edu/pub/bash/bash-4.3-patches/ PLEVEL=${PV##*_p} MY_PV=${PV/_p*} MY_PV=${MY_PV/_/-} MY_P=${PN}-${MY_PV} [[ ${PV} != *_p* ]] && PLEVEL=0 patches() { local opt=$1 plevel=${2:-${PLEVEL}} pn=${3:-${PN}} pv=${4:-${MY_PV}} [[ ${plevel} -eq 0 ]] && return 1 eval set -- {1..${plevel}} set -- $(printf "${pn}${pv/\.}-%03d " "$@") if [[ ${opt} == -s ]] ; then echo "${@/#/${DISTDIR}/}" else local u for u in ftp://ftp.cwru.edu/pub/bash mirror://gnu/${pn} ; do printf "${u}/${pn}-${pv}-patches/%s " "$@" done fi } # The version of readline this bash normally ships with. # Set an empty value to it if there's none. READLINE_VER="7.0_rc2" IS_A_RELEASE=false [[ ${PV} != *_alpha* && ${PV} != *_beta* && ${PV} != *_rc* ]] && IS_A_RELEASE=true ALLOW_SYSTEM_READLINE=false [[ -n ${READLINE_VER} ]] && ALLOW_SYSTEM_READLINE=true DESCRIPTION="The standard GNU Bourne again shell" HOMEPAGE="http://tiswww.case.edu/php/chet/bash/bashtop.html" if [[ ${IS_A_RELEASE} == true ]]; then SRC_URI="mirror://gnu/bash/${MY_P}.tar.gz $(patches)" else SRC_URI+=" ftp://ftp.cwru.edu/pub/bash/${MY_P}.tar.gz" fi LICENSE="GPL-3" SLOT="0" [[ ${IS_A_RELEASE} == true ]] && KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~hppa ~ia64 ~m68k ~mips ~ppc ~ppc64 ~s390 ~sh ~sparc ~x86 ~amd64-fbsd ~sparc-fbsd ~x86-fbsd" IUSE="afs bashlogger examples mem-scramble +net nls plugins +readline tinfo vanilla" [[ ${ALLOW_SYSTEM_READLINE} == true ]] && IUSE+=' +system-readline' DEPEND=">=sys-libs/ncurses-5.2-r2:0= tinfo? ( sys-libs/ncurses:0=[tinfo] ) !tinfo? ( !sys-libs/ncurses:0=[tinfo] ) nls? ( virtual/libintl )" [[ ${ALLOW_SYSTEM_READLINE} == true ]] && DEPEND+=" readline? ( system-readline? ( >=sys-libs/readline-${READLINE_VER}:0= ) )" RDEPEND="${DEPEND} !<sys-apps/portage-2.1.6.7_p1 !<sys-apps/paludis-0.26.0_alpha5" # we only need yacc when the .y files get patched (bash42-005) DEPEND+=" virtual/yacc" S=${WORKDIR}/${MY_P} pkg_setup() { if is-flag -malign-double ; then #7332 eerror "Detected bad CFLAGS '-malign-double'. Do not use this" eerror "as it breaks LFS (struct stat64) on x86." die "remove -malign-double from your CFLAGS mr ricer" fi if use bashlogger ; then ewarn "The logging patch should ONLY be used in restricted (i.e. honeypot) envs." ewarn "This will log ALL output you enter into the shell, you have been warned." fi use tinfo && export LDFLAGS="-ltinfo ${LDFLAGS}" LIBS="-ltinfo ${LIBS}" } src_unpack() { unpack ${MY_P}.tar.gz } src_prepare() { # Include official patches [[ ${PLEVEL} -gt 0 ]] && epatch $(patches -s) # Clean out local libs so we know we use system ones w/releases. if [[ ${ALLOW_SYSTEM_READLINE} == true ]] && use system-readline; then rm -rf lib/{readline,termcap}/* touch lib/{readline,termcap}/Makefile.in # for config.status sed -ri -e 's:\$[(](RL|HIST)_LIBSRC[)]/[[:alpha:]]*.h::g' Makefile.in || die fi # Avoid regenerating docs after patches #407985 sed -i -r '/^(HS|RL)USER/s:=.*:=:' doc/Makefile.in || die touch -r . doc/* epatch_user } src_configure() { local myconf=() # For descriptions of these, see config-top.h # bashrc/#26952 bash_logout/#90488 ssh/#24762 mktemp/#574426 append-cppflags \ -DDEFAULT_PATH_VALUE=\'\"${EPREFIX}/usr/local/sbin:${EPREFIX}/usr/local/bin:${EPREFIX}/usr/sbin:${EPREFIX}/usr/bin:${EPREFIX}/sbin:${EPREFIX}/bin\"\' \ -DSTANDARD_UTILS_PATH=\'\"${EPREFIX}/bin:${EPREFIX}/usr/bin:${EPREFIX}/sbin:${EPREFIX}/usr/sbin\"\' \ -DSYS_BASHRC=\'\"${EPREFIX}/etc/bash/bashrc\"\' \ -DSYS_BASH_LOGOUT=\'\"${EPREFIX}/etc/bash/bash_logout\"\' \ -DNON_INTERACTIVE_LOGIN_SHELLS \ -DSSH_SOURCE_BASHRC \ $(use bashlogger && echo -DSYSLOG_HISTORY) # Don't even think about building this statically without # reading Bug 7714 first. If you still build it statically, # don't come crying to us with bugs ;). #use static && export LDFLAGS="${LDFLAGS} -static" use nls || myconf+=( --disable-nls ) # Historically, we always used the builtin readline, but since # our handling of SONAME upgrades has gotten much more stable # in the PM (and the readline ebuild itself preserves the old # libs during upgrades), linking against the system copy should # be safe. # Exact cached version here doesn't really matter as long as it # is at least what's in the DEPEND up above. export ac_cv_rl_version=${READLINE_VER%%_*} # Force linking with system curses ... the bundled termcap lib # sucks bad compared to ncurses. For the most part, ncurses # is here because readline needs it. But bash itself calls # ncurses in one or two small places :(. if [[ ${ALLOW_SYSTEM_READLINE} == true ]] && use system-readline; then # Use system readline only with released versions. myconf+=( --with-system-readline=. ) fi if use plugins; then append-ldflags -Wl,-rpath,/usr/$(get_libdir)/bash else # Disable the plugins logic by hand since bash doesn't # provide a way of doing it. export ac_cv_func_dl{close,open,sym}=no \ ac_cv_lib_dl_dlopen=no ac_cv_header_dlfcn_h=no sed -i \ -e '/LOCAL_LDFLAGS=/s:-rdynamic::' \ configure || die fi tc-export AR #444070 econf \ --docdir='$(datarootdir)'/doc/${PF} \ --htmldir='$(docdir)/html' \ --with-curses \ $(use_with afs) \ $(use_enable net net-redirections) \ --disable-profiling \ $(use_enable mem-scramble) \ $(use_with mem-scramble bash-malloc) \ $(use_enable readline) \ $(use_enable readline history) \ $(use_enable readline bang-history) \ "${myconf[@]}" } src_compile() { emake if use plugins ; then emake -C examples/loadables all others fi } src_install() { local d f default dodir /bin mv "${ED}"/usr/bin/bash "${ED}"/bin/ || die dosym bash /bin/rbash insinto /etc/bash doins "${FILESDIR}"/bash_logout doins "${FILESDIR}"/bashrc keepdir /etc/bash/bashrc.d insinto /etc/skel for f in bash{_logout,_profile,rc} ; do newins "${FILESDIR}"/dot-${f} .${f} done local sed_args=( -e "s:#${USERLAND}#@::" -e '/#@/d' ) if ! use readline ; then sed_args+=( #432338 -e '/^shopt -s histappend/s:^:#:' -e 's:use_color=true:use_color=false:' ) fi sed -i \ "${sed_args[@]}" \ "${ED}"/etc/skel/.bashrc \ "${ED}"/etc/bash/bashrc || die if use plugins ; then exeinto /usr/$(get_libdir)/bash doexe $(echo examples/loadables/*.o | sed 's:\.o::g') insinto /usr/include/bash-plugins doins *.h builtins/*.h include/*.h lib/{glob/glob.h,tilde/tilde.h} fi if use examples ; then for d in examples/{functions,misc,scripts,startup-files} ; do exeinto /usr/share/doc/${PF}/${d} insinto /usr/share/doc/${PF}/${d} for f in ${d}/* ; do if [[ ${f##*/} != PERMISSION ]] && [[ ${f##*/} != *README ]] ; then doexe ${f} else doins ${f} fi done done fi doman doc/*.1 newdoc CWRU/changelog ChangeLog dosym bash.info /usr/share/info/bashref.info } pkg_preinst() { if [[ -e ${EROOT}/etc/bashrc ]] && [[ ! -d ${EROOT}/etc/bash ]] ; then mkdir -p "${EROOT}"/etc/bash mv -f "${EROOT}"/etc/bashrc "${EROOT}"/etc/bash/ fi if [[ -L ${EROOT}/bin/sh ]] ; then # rewrite the symlink to ensure that its mtime changes. having /bin/sh # missing even temporarily causes a fatal error with paludis. local target=$(readlink "${EROOT}"/bin/sh) local tmp=$(emktemp "${EROOT}"/bin) ln -sf "${target}" "${tmp}" mv -f "${tmp}" "${EROOT}"/bin/sh fi } pkg_postinst() { # If /bin/sh does not exist, provide it if [[ ! -e ${EROOT}/bin/sh ]] ; then ln -sf bash "${EROOT}"/bin/sh fi } ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-03 5:37 ` konsolebox @ 2016-10-03 5:47 ` konsolebox 0 siblings, 0 replies; 33+ messages in thread From: konsolebox @ 2016-10-03 5:47 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 240 bytes --] On Mon, Oct 3, 2016 at 1:37 PM, konsolebox <konsolebox@gmail.com> wrote: > I created another example for this. I mistakenly renamed --with-installed-readline to --with-system-readline there, sorry. Here's the correct one. -- konsolebox [-- Attachment #2: bash-4.4-r1.ebuild --] [-- Type: application/octet-stream, Size: 7688 bytes --] # Copyright 1999-2016 Gentoo Foundation # Distributed under the terms of the GNU General Public License v2 # $Id$ EAPI="5" inherit eutils flag-o-matic toolchain-funcs multilib # Official patchlevel # See ftp://ftp.cwru.edu/pub/bash/bash-4.3-patches/ PLEVEL=${PV##*_p} MY_PV=${PV/_p*} MY_PV=${MY_PV/_/-} MY_P=${PN}-${MY_PV} [[ ${PV} != *_p* ]] && PLEVEL=0 patches() { local opt=$1 plevel=${2:-${PLEVEL}} pn=${3:-${PN}} pv=${4:-${MY_PV}} [[ ${plevel} -eq 0 ]] && return 1 eval set -- {1..${plevel}} set -- $(printf "${pn}${pv/\.}-%03d " "$@") if [[ ${opt} == -s ]] ; then echo "${@/#/${DISTDIR}/}" else local u for u in ftp://ftp.cwru.edu/pub/bash mirror://gnu/${pn} ; do printf "${u}/${pn}-${pv}-patches/%s " "$@" done fi } # The version of readline this bash normally ships with. # Set an empty value to it if there's none. READLINE_VER="7.0_rc2" IS_A_RELEASE=false [[ ${PV} != *_alpha* && ${PV} != *_beta* && ${PV} != *_rc* ]] && IS_A_RELEASE=true ALLOW_SYSTEM_READLINE=false [[ -n ${READLINE_VER} ]] && ALLOW_SYSTEM_READLINE=true DESCRIPTION="The standard GNU Bourne again shell" HOMEPAGE="http://tiswww.case.edu/php/chet/bash/bashtop.html" if [[ ${IS_A_RELEASE} == true ]]; then SRC_URI="mirror://gnu/bash/${MY_P}.tar.gz $(patches)" else SRC_URI+=" ftp://ftp.cwru.edu/pub/bash/${MY_P}.tar.gz" fi LICENSE="GPL-3" SLOT="0" [[ ${IS_A_RELEASE} == true ]] && KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~hppa ~ia64 ~m68k ~mips ~ppc ~ppc64 ~s390 ~sh ~sparc ~x86 ~amd64-fbsd ~sparc-fbsd ~x86-fbsd" IUSE="afs bashlogger examples mem-scramble +net nls plugins +readline tinfo vanilla" [[ ${ALLOW_SYSTEM_READLINE} == true ]] && IUSE+=' +system-readline' DEPEND=">=sys-libs/ncurses-5.2-r2:0= tinfo? ( sys-libs/ncurses:0=[tinfo] ) !tinfo? ( !sys-libs/ncurses:0=[tinfo] ) nls? ( virtual/libintl )" [[ ${ALLOW_SYSTEM_READLINE} == true ]] && DEPEND+=" readline? ( system-readline? ( >=sys-libs/readline-${READLINE_VER}:0= ) )" RDEPEND="${DEPEND} !<sys-apps/portage-2.1.6.7_p1 !<sys-apps/paludis-0.26.0_alpha5" # we only need yacc when the .y files get patched (bash42-005) DEPEND+=" virtual/yacc" S=${WORKDIR}/${MY_P} pkg_setup() { if is-flag -malign-double ; then #7332 eerror "Detected bad CFLAGS '-malign-double'. Do not use this" eerror "as it breaks LFS (struct stat64) on x86." die "remove -malign-double from your CFLAGS mr ricer" fi if use bashlogger ; then ewarn "The logging patch should ONLY be used in restricted (i.e. honeypot) envs." ewarn "This will log ALL output you enter into the shell, you have been warned." fi use tinfo && export LDFLAGS="-ltinfo ${LDFLAGS}" LIBS="-ltinfo ${LIBS}" } src_unpack() { unpack ${MY_P}.tar.gz } src_prepare() { # Include official patches [[ ${PLEVEL} -gt 0 ]] && epatch $(patches -s) # Clean out local libs so we know we use system ones w/releases. if [[ ${ALLOW_SYSTEM_READLINE} == true ]] && use system-readline; then rm -rf lib/{readline,termcap}/* touch lib/{readline,termcap}/Makefile.in # for config.status sed -ri -e 's:\$[(](RL|HIST)_LIBSRC[)]/[[:alpha:]]*.h::g' Makefile.in || die fi # Avoid regenerating docs after patches #407985 sed -i -r '/^(HS|RL)USER/s:=.*:=:' doc/Makefile.in || die touch -r . doc/* epatch_user } src_configure() { local myconf=() # For descriptions of these, see config-top.h # bashrc/#26952 bash_logout/#90488 ssh/#24762 mktemp/#574426 append-cppflags \ -DDEFAULT_PATH_VALUE=\'\"${EPREFIX}/usr/local/sbin:${EPREFIX}/usr/local/bin:${EPREFIX}/usr/sbin:${EPREFIX}/usr/bin:${EPREFIX}/sbin:${EPREFIX}/bin\"\' \ -DSTANDARD_UTILS_PATH=\'\"${EPREFIX}/bin:${EPREFIX}/usr/bin:${EPREFIX}/sbin:${EPREFIX}/usr/sbin\"\' \ -DSYS_BASHRC=\'\"${EPREFIX}/etc/bash/bashrc\"\' \ -DSYS_BASH_LOGOUT=\'\"${EPREFIX}/etc/bash/bash_logout\"\' \ -DNON_INTERACTIVE_LOGIN_SHELLS \ -DSSH_SOURCE_BASHRC \ $(use bashlogger && echo -DSYSLOG_HISTORY) # Don't even think about building this statically without # reading Bug 7714 first. If you still build it statically, # don't come crying to us with bugs ;). #use static && export LDFLAGS="${LDFLAGS} -static" use nls || myconf+=( --disable-nls ) # Historically, we always used the builtin readline, but since # our handling of SONAME upgrades has gotten much more stable # in the PM (and the readline ebuild itself preserves the old # libs during upgrades), linking against the system copy should # be safe. # Exact cached version here doesn't really matter as long as it # is at least what's in the DEPEND up above. export ac_cv_rl_version=${READLINE_VER%%_*} # Force linking with system curses ... the bundled termcap lib # sucks bad compared to ncurses. For the most part, ncurses # is here because readline needs it. But bash itself calls # ncurses in one or two small places :(. if [[ ${ALLOW_SYSTEM_READLINE} == true ]] && use system-readline; then # Use system readline only with released versions. myconf+=( --with-installed-readline=. ) fi if use plugins; then append-ldflags -Wl,-rpath,/usr/$(get_libdir)/bash else # Disable the plugins logic by hand since bash doesn't # provide a way of doing it. export ac_cv_func_dl{close,open,sym}=no \ ac_cv_lib_dl_dlopen=no ac_cv_header_dlfcn_h=no sed -i \ -e '/LOCAL_LDFLAGS=/s:-rdynamic::' \ configure || die fi tc-export AR #444070 econf \ --docdir='$(datarootdir)'/doc/${PF} \ --htmldir='$(docdir)/html' \ --with-curses \ $(use_with afs) \ $(use_enable net net-redirections) \ --disable-profiling \ $(use_enable mem-scramble) \ $(use_with mem-scramble bash-malloc) \ $(use_enable readline) \ $(use_enable readline history) \ $(use_enable readline bang-history) \ "${myconf[@]}" } src_compile() { emake if use plugins ; then emake -C examples/loadables all others fi } src_install() { local d f default dodir /bin mv "${ED}"/usr/bin/bash "${ED}"/bin/ || die dosym bash /bin/rbash insinto /etc/bash doins "${FILESDIR}"/bash_logout doins "${FILESDIR}"/bashrc keepdir /etc/bash/bashrc.d insinto /etc/skel for f in bash{_logout,_profile,rc} ; do newins "${FILESDIR}"/dot-${f} .${f} done local sed_args=( -e "s:#${USERLAND}#@::" -e '/#@/d' ) if ! use readline ; then sed_args+=( #432338 -e '/^shopt -s histappend/s:^:#:' -e 's:use_color=true:use_color=false:' ) fi sed -i \ "${sed_args[@]}" \ "${ED}"/etc/skel/.bashrc \ "${ED}"/etc/bash/bashrc || die if use plugins ; then exeinto /usr/$(get_libdir)/bash doexe $(echo examples/loadables/*.o | sed 's:\.o::g') insinto /usr/include/bash-plugins doins *.h builtins/*.h include/*.h lib/{glob/glob.h,tilde/tilde.h} fi if use examples ; then for d in examples/{functions,misc,scripts,startup-files} ; do exeinto /usr/share/doc/${PF}/${d} insinto /usr/share/doc/${PF}/${d} for f in ${d}/* ; do if [[ ${f##*/} != PERMISSION ]] && [[ ${f##*/} != *README ]] ; then doexe ${f} else doins ${f} fi done done fi doman doc/*.1 newdoc CWRU/changelog ChangeLog dosym bash.info /usr/share/info/bashref.info } pkg_preinst() { if [[ -e ${EROOT}/etc/bashrc ]] && [[ ! -d ${EROOT}/etc/bash ]] ; then mkdir -p "${EROOT}"/etc/bash mv -f "${EROOT}"/etc/bashrc "${EROOT}"/etc/bash/ fi if [[ -L ${EROOT}/bin/sh ]] ; then # rewrite the symlink to ensure that its mtime changes. having /bin/sh # missing even temporarily causes a fatal error with paludis. local target=$(readlink "${EROOT}"/bin/sh) local tmp=$(emktemp "${EROOT}"/bin) ln -sf "${target}" "${tmp}" mv -f "${tmp}" "${EROOT}"/bin/sh fi } pkg_postinst() { # If /bin/sh does not exist, provide it if [[ ! -e ${EROOT}/bin/sh ]] ; then ln -sf bash "${EROOT}"/bin/sh fi } ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-03 5:32 ` konsolebox 2016-10-03 5:37 ` konsolebox @ 2016-10-03 5:53 ` Kent Fredric 2016-10-03 6:37 ` konsolebox 1 sibling, 1 reply; 33+ messages in thread From: Kent Fredric @ 2016-10-03 5:53 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 1833 bytes --] On Mon, 3 Oct 2016 13:32:34 +0800 konsolebox <konsolebox@gmail.com> wrote: > "A useflag that entirely goes away depending on the version" (or a > flag that is implemented in one ebuild but is not on another) is > pretty common among packages and I see that as totally valid, and is > way better than a solution that uses dynamic default. I think the main assumption I'm trying to challenge here is that users would desire different USE and BUILD configurations based on _rc status, and I don't think that's much the case. People IME tend to prefer to set something on a package level, and then apply consistent enforcement for all versions, or perhaps apply USE flags to ranges of versions ( ie: everything larger than X, everything smaller than X ), or on a per-slot basis. Mostly because its consistent and minimises the amount of effort. Thus, for basically any argument you can make for having the USE flag at all, maps to an argument that applies to both _rc and non _rc versions. And I get the impression that the desire you have to have different behaviour for non-rc versions is a bit niche. ( I can appreciate it in a -9999 version, but that produces a situation which falls under "all versions larger than 9" and thus gives you version-range based consistency. ) Anything else seems to me to devolve into "being too fiddly", and may discourage testing. But I also see it that people who are testing _rc + KEYWORDS="" versions to be people who's express intent is to optimise for maximum breakage, in order to find the most possible problems before they happen, and problems like "readline version is too new or too old" are exactly the kinds of problems I'd want to make sure we didn't have before we shipped bash in that configuration to everyone ( even to ~arch ) [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 801 bytes --] ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-03 5:53 ` Kent Fredric @ 2016-10-03 6:37 ` konsolebox 2016-10-03 7:09 ` Kent Fredric 0 siblings, 1 reply; 33+ messages in thread From: konsolebox @ 2016-10-03 6:37 UTC (permalink / raw To: gentoo-dev On Mon, Oct 3, 2016 at 1:53 PM, Kent Fredric <kentnl@gentoo.org> wrote: > And I get the impression that the desire you have to have different > behaviour for non-rc versions is a bit niche. Not quite. I just started by siding with the current implementation which is to disable system readline in rc versions of bash. You're actually the one challenging a new implementation which would allow it. But anyway, what do you think about just enabling IUSE="+system-readline" by default to any version, and just rely on KEYWORDS="" to prevent the user from misusing it with a pre-release version of bash? I believe we can both agree on that idea. -- konsolebox ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-03 6:37 ` konsolebox @ 2016-10-03 7:09 ` Kent Fredric 2016-10-03 12:02 ` konsolebox 0 siblings, 1 reply; 33+ messages in thread From: Kent Fredric @ 2016-10-03 7:09 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 543 bytes --] On Mon, 3 Oct 2016 14:37:15 +0800 konsolebox <konsolebox@gmail.com> wrote: > But anyway, what do you think about just enabling > IUSE="+system-readline" by default to any version, and just rely on > KEYWORDS="" to prevent the user from misusing it with a pre-release > version of bash? I believe we can both agree on that idea. Yeah. And at least that way if you need to prevent its use/force its use for any arch target you can use the USE forces. In general "less magical behaviour that tracks its version" is a good thing. [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 801 bytes --] ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-03 7:09 ` Kent Fredric @ 2016-10-03 12:02 ` konsolebox 0 siblings, 0 replies; 33+ messages in thread From: konsolebox @ 2016-10-03 12:02 UTC (permalink / raw To: gentoo-dev On Mon, Oct 3, 2016 at 3:09 PM, Kent Fredric <kentnl@gentoo.org> wrote: > On Mon, 3 Oct 2016 14:37:15 +0800 > konsolebox <konsolebox@gmail.com> wrote: > >> But anyway, what do you think about just enabling >> IUSE="+system-readline" by default to any version, and just rely on >> KEYWORDS="" to prevent the user from misusing it with a pre-release >> version of bash? I believe we can both agree on that idea. > > Yeah. And at least that way if you need to prevent its use/force > its use for any arch target you can use the USE forces. > > In general "less magical behaviour that tracks its version" is a good > thing. I created a formal request for this in https://bugs.gentoo.org/show_bug.cgi?id=596028. -- konsolebox ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-02 5:28 ` konsolebox 2016-10-02 5:40 ` Kent Fredric @ 2016-10-02 6:29 ` konsolebox 1 sibling, 0 replies; 33+ messages in thread From: konsolebox @ 2016-10-02 6:29 UTC (permalink / raw To: gentoo-dev [-- Attachment #1: Type: text/plain, Size: 1355 bytes --] On Sun, Oct 2, 2016 at 1:28 PM, konsolebox <konsolebox@gmail.com> wrote: > On Sun, Oct 2, 2016 at 12:34 AM, Dan Douglas <ormaaj@gmail.com> wrote: >> I'd be perfectly happy requiring bundled readline when USE="readline" >> for bash versions incompatible with the installed readline, > > I guess that's another good way to solve the readline issue (when it > comes to bash). But I'd prefer that it's not done automatically. > Instead we should add a formal use flag like 'installed-readline'. We > can add it to release versions of bash ([[ ${PV} != *_rc* ]] && > IUSE+=' +installed-readline'), and enable it by default. Then we > change all `[[ ${PV} != *_rc* ]]` condition checks to `[[ ${PV} != > *_rc* ]] && use installed-readline`. `${PV} != *_rc*` probably should > also be `${PV} != *_alpha* && ${PV} != *_beta* && ${PV} != *_rc*`. > (See attached file for POC.) I missed disabling the dependency to readline when 'installed-readline' is disabled. Here's another example. I also fixed a repoman warning related to tinfo and renamed 'installed_readline' to 'installed-readline'. It installs and runs well with `USE='-installed-readline tinfo' emerge -v 'bash::local'`. The modifications related to tinfo by the way is related to this: https://bugs.gentoo.org/show_bug.cgi?id=588486. I can't install bash properly without it. -- konsolebox [-- Attachment #2: bash-4.4.ebuild --] [-- Type: application/octet-stream, Size: 7503 bytes --] # Copyright 1999-2016 Gentoo Foundation # Distributed under the terms of the GNU General Public License v2 # $Id$ EAPI="5" inherit eutils flag-o-matic toolchain-funcs multilib # Official patchlevel # See ftp://ftp.cwru.edu/pub/bash/bash-4.3-patches/ PLEVEL=${PV##*_p} MY_PV=${PV/_p*} MY_PV=${MY_PV/_/-} MY_P=${PN}-${MY_PV} [[ ${PV} != *_p* ]] && PLEVEL=0 patches() { local opt=$1 plevel=${2:-${PLEVEL}} pn=${3:-${PN}} pv=${4:-${MY_PV}} [[ ${plevel} -eq 0 ]] && return 1 eval set -- {1..${plevel}} set -- $(printf "${pn}${pv/\.}-%03d " "$@") if [[ ${opt} == -s ]] ; then echo "${@/#/${DISTDIR}/}" else local u for u in ftp://ftp.cwru.edu/pub/bash mirror://gnu/${pn} ; do printf "${u}/${pn}-${pv}-patches/%s " "$@" done fi } # The version of readline this bash normally ships with. READLINE_VER="7.0_rc2" IS_A_RELEASE=false [[ ${PV} != *_alpha* && ${PV} != *_beta* && ${PV} != *_rc* ]] && IS_A_RELEASE=true DESCRIPTION="The standard GNU Bourne again shell" HOMEPAGE="http://tiswww.case.edu/php/chet/bash/bashtop.html" if [[ ${IS_A_RELEASE} == true ]]; then SRC_URI="mirror://gnu/bash/${MY_P}.tar.gz $(patches)" else SRC_URI+=" ftp://ftp.cwru.edu/pub/bash/${MY_P}.tar.gz" fi LICENSE="GPL-3" SLOT="0" KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~hppa ~ia64 ~m68k ~mips ~ppc ~ppc64 ~s390 ~sh ~sparc ~x86 ~amd64-fbsd ~sparc-fbsd ~x86-fbsd" IUSE="afs bashlogger examples mem-scramble +net nls plugins +readline tinfo vanilla" [[ ${IS_A_RELEASE} == true ]] && IUSE+=' +installed-readline' DEPEND=">=sys-libs/ncurses-5.2-r2:0= tinfo? ( sys-libs/ncurses:0=[tinfo] ) !tinfo? ( !sys-libs/ncurses:0=[tinfo] ) nls? ( virtual/libintl )" [[ ${IS_A_RELEASE} == true ]] && DEPEND+=" readline? ( installed-readline? ( >=sys-libs/readline-${READLINE_VER}:0= ) )" RDEPEND="${DEPEND} !<sys-apps/portage-2.1.6.7_p1 !<sys-apps/paludis-0.26.0_alpha5" # we only need yacc when the .y files get patched (bash42-005) DEPEND+=" virtual/yacc" S=${WORKDIR}/${MY_P} pkg_setup() { if is-flag -malign-double ; then #7332 eerror "Detected bad CFLAGS '-malign-double'. Do not use this" eerror "as it breaks LFS (struct stat64) on x86." die "remove -malign-double from your CFLAGS mr ricer" fi if use bashlogger ; then ewarn "The logging patch should ONLY be used in restricted (i.e. honeypot) envs." ewarn "This will log ALL output you enter into the shell, you have been warned." fi use tinfo && export LDFLAGS="-ltinfo ${LDFLAGS}" LIBS="-ltinfo ${LIBS}" } src_unpack() { unpack ${MY_P}.tar.gz } src_prepare() { # Include official patches [[ ${PLEVEL} -gt 0 ]] && epatch $(patches -s) # Clean out local libs so we know we use system ones w/releases. if [[ ${IS_A_RELEASE} == true ]] && use installed-readline; then rm -rf lib/{readline,termcap}/* touch lib/{readline,termcap}/Makefile.in # for config.status sed -ri -e 's:\$[(](RL|HIST)_LIBSRC[)]/[[:alpha:]]*.h::g' Makefile.in || die fi # Avoid regenerating docs after patches #407985 sed -i -r '/^(HS|RL)USER/s:=.*:=:' doc/Makefile.in || die touch -r . doc/* epatch_user } src_configure() { local myconf=() # For descriptions of these, see config-top.h # bashrc/#26952 bash_logout/#90488 ssh/#24762 mktemp/#574426 append-cppflags \ -DDEFAULT_PATH_VALUE=\'\"${EPREFIX}/usr/local/sbin:${EPREFIX}/usr/local/bin:${EPREFIX}/usr/sbin:${EPREFIX}/usr/bin:${EPREFIX}/sbin:${EPREFIX}/bin\"\' \ -DSTANDARD_UTILS_PATH=\'\"${EPREFIX}/bin:${EPREFIX}/usr/bin:${EPREFIX}/sbin:${EPREFIX}/usr/sbin\"\' \ -DSYS_BASHRC=\'\"${EPREFIX}/etc/bash/bashrc\"\' \ -DSYS_BASH_LOGOUT=\'\"${EPREFIX}/etc/bash/bash_logout\"\' \ -DNON_INTERACTIVE_LOGIN_SHELLS \ -DSSH_SOURCE_BASHRC \ $(use bashlogger && echo -DSYSLOG_HISTORY) # Don't even think about building this statically without # reading Bug 7714 first. If you still build it statically, # don't come crying to us with bugs ;). #use static && export LDFLAGS="${LDFLAGS} -static" use nls || myconf+=( --disable-nls ) # Historically, we always used the builtin readline, but since # our handling of SONAME upgrades has gotten much more stable # in the PM (and the readline ebuild itself preserves the old # libs during upgrades), linking against the system copy should # be safe. # Exact cached version here doesn't really matter as long as it # is at least what's in the DEPEND up above. export ac_cv_rl_version=${READLINE_VER%%_*} # Force linking with system curses ... the bundled termcap lib # sucks bad compared to ncurses. For the most part, ncurses # is here because readline needs it. But bash itself calls # ncurses in one or two small places :(. if [[ ${IS_A_RELEASE} == true ]] && use installed-readline; then # Use system readline only with released versions. myconf+=( --with-installed-readline=. ) fi if use plugins; then append-ldflags -Wl,-rpath,/usr/$(get_libdir)/bash else # Disable the plugins logic by hand since bash doesn't # provide a way of doing it. export ac_cv_func_dl{close,open,sym}=no \ ac_cv_lib_dl_dlopen=no ac_cv_header_dlfcn_h=no sed -i \ -e '/LOCAL_LDFLAGS=/s:-rdynamic::' \ configure || die fi tc-export AR #444070 econf \ --docdir='$(datarootdir)'/doc/${PF} \ --htmldir='$(docdir)/html' \ --with-curses \ $(use_with afs) \ $(use_enable net net-redirections) \ --disable-profiling \ $(use_enable mem-scramble) \ $(use_with mem-scramble bash-malloc) \ $(use_enable readline) \ $(use_enable readline history) \ $(use_enable readline bang-history) \ "${myconf[@]}" } src_compile() { emake if use plugins ; then emake -C examples/loadables all others fi } src_install() { local d f default dodir /bin mv "${ED}"/usr/bin/bash "${ED}"/bin/ || die dosym bash /bin/rbash insinto /etc/bash doins "${FILESDIR}"/bash_logout doins "${FILESDIR}"/bashrc keepdir /etc/bash/bashrc.d insinto /etc/skel for f in bash{_logout,_profile,rc} ; do newins "${FILESDIR}"/dot-${f} .${f} done local sed_args=( -e "s:#${USERLAND}#@::" -e '/#@/d' ) if ! use readline ; then sed_args+=( #432338 -e '/^shopt -s histappend/s:^:#:' -e 's:use_color=true:use_color=false:' ) fi sed -i \ "${sed_args[@]}" \ "${ED}"/etc/skel/.bashrc \ "${ED}"/etc/bash/bashrc || die if use plugins ; then exeinto /usr/$(get_libdir)/bash doexe $(echo examples/loadables/*.o | sed 's:\.o::g') insinto /usr/include/bash-plugins doins *.h builtins/*.h include/*.h lib/{glob/glob.h,tilde/tilde.h} fi if use examples ; then for d in examples/{functions,misc,scripts,startup-files} ; do exeinto /usr/share/doc/${PF}/${d} insinto /usr/share/doc/${PF}/${d} for f in ${d}/* ; do if [[ ${f##*/} != PERMISSION ]] && [[ ${f##*/} != *README ]] ; then doexe ${f} else doins ${f} fi done done fi doman doc/*.1 newdoc CWRU/changelog ChangeLog dosym bash.info /usr/share/info/bashref.info } pkg_preinst() { if [[ -e ${EROOT}/etc/bashrc ]] && [[ ! -d ${EROOT}/etc/bash ]] ; then mkdir -p "${EROOT}"/etc/bash mv -f "${EROOT}"/etc/bashrc "${EROOT}"/etc/bash/ fi if [[ -L ${EROOT}/bin/sh ]] ; then # rewrite the symlink to ensure that its mtime changes. having /bin/sh # missing even temporarily causes a fatal error with paludis. local target=$(readlink "${EROOT}"/bin/sh) local tmp=$(emktemp "${EROOT}"/bin) ln -sf "${target}" "${tmp}" mv -f "${tmp}" "${EROOT}"/bin/sh fi } pkg_postinst() { # If /bin/sh does not exist, provide it if [[ ! -e ${EROOT}/bin/sh ]] ; then ln -sf bash "${EROOT}"/bin/sh fi } ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-09-20 11:41 [gentoo-dev] bash-4.4 - call for testers Lars Wendler 2016-09-22 9:22 ` David Haller 2016-09-29 5:54 ` Dan Douglas @ 2016-10-01 21:48 ` Dan Douglas 2016-10-01 22:40 ` Dan Douglas 2 siblings, 1 reply; 33+ messages in thread From: Dan Douglas @ 2016-10-01 21:48 UTC (permalink / raw To: gentoo-dev One note - it may be useful to install `BASH_LOADABLES_PATH=/usr/lib/bash` to the global env.d environment when USE=plugins, now that bash has this feature. It doesn't enable anything by default but just informs bash that it can look for loadables in that location. (ins)ormaaj@smorgbox ~ $ BASH_LOADABLES_PATH=/usr/lib/bash bash (ins)ormaaj@smorgbox ~ $ enable -f hello hello hello builtin loaded (ins)ormaaj@smorgbox ~ $ hello hello world ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [gentoo-dev] bash-4.4 - call for testers 2016-10-01 21:48 ` Dan Douglas @ 2016-10-01 22:40 ` Dan Douglas 0 siblings, 0 replies; 33+ messages in thread From: Dan Douglas @ 2016-10-01 22:40 UTC (permalink / raw To: gentoo-dev On Sat, Oct 1, 2016 at 4:48 PM, Dan Douglas <ormaaj@gmail.com> wrote: > One note - it may be useful to install `BASH_LOADABLES_PATH=/usr/lib/bash` > to the global env.d environment when USE=plugins, now that bash has > this feature. It doesn't enable anything by default but just informs bash that > it can look for loadables in that location. > > (ins)ormaaj@smorgbox ~ $ BASH_LOADABLES_PATH=/usr/lib/bash bash > (ins)ormaaj@smorgbox ~ $ enable -f hello hello > hello builtin loaded > (ins)ormaaj@smorgbox ~ $ hello > hello world Hm on looking again it seems to look there anyway? I guess dlopen finds them via -rpath. Didn't know that was sufficient in this case. Sorry about that. ^ permalink raw reply [flat|nested] 33+ messages in thread
end of thread, other threads:[~2016-10-03 12:03 UTC | newest] Thread overview: 33+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-09-20 11:41 [gentoo-dev] bash-4.4 - call for testers Lars Wendler 2016-09-22 9:22 ` David Haller 2016-09-23 10:42 ` David Haller 2016-09-29 5:54 ` Dan Douglas 2016-09-29 21:56 ` Andy Mender 2016-09-29 22:29 ` Kent Fredric 2016-09-30 13:38 ` William Hubbs 2016-09-30 17:09 ` Andy Mender 2016-10-01 6:59 ` Dan Douglas 2016-10-01 7:47 ` Dan Douglas 2016-09-30 17:53 ` konsolebox 2016-09-30 17:49 ` konsolebox 2016-10-01 0:38 ` Kent Fredric 2016-10-01 14:38 ` konsolebox 2016-10-01 15:40 ` konsolebox 2016-10-01 16:34 ` Dan Douglas 2016-10-02 5:28 ` konsolebox 2016-10-02 5:40 ` Kent Fredric 2016-10-02 8:03 ` konsolebox 2016-10-02 8:58 ` Kent Fredric 2016-10-02 10:00 ` konsolebox 2016-10-02 10:18 ` konsolebox 2016-10-02 11:42 ` Kent Fredric 2016-10-03 5:32 ` konsolebox 2016-10-03 5:37 ` konsolebox 2016-10-03 5:47 ` konsolebox 2016-10-03 5:53 ` Kent Fredric 2016-10-03 6:37 ` konsolebox 2016-10-03 7:09 ` Kent Fredric 2016-10-03 12:02 ` konsolebox 2016-10-02 6:29 ` konsolebox 2016-10-01 21:48 ` Dan Douglas 2016-10-01 22:40 ` Dan Douglas
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox