summaryrefslogtreecommitdiffstats
path: root/lib/libalpm/util.c
diff options
context:
space:
mode:
authorXavier Chantry <shiningxc@gmail.com>2009-03-15 12:43:39 +0100
committerDan McGee <dan@archlinux.org>2009-03-15 17:41:37 +0100
commitde97282fbd78755cde71c1d064ec00b806efcca5 (patch)
treedc1a066d0db68e23e39d1f13b0be248f1465d38a /lib/libalpm/util.c
parent36863b968f562840461e60be4ed23abf635d6008 (diff)
downloadpacman-de97282fbd78755cde71c1d064ec00b806efcca5.tar.gz
pacman-de97282fbd78755cde71c1d064ec00b806efcca5.tar.xz
Revert "popen does NOT require /bin/sh in a subchroot"
This reverts commit 9558639d8009483fbf422b138d020745986f82f1. This change was wrong, popen does require /bin/sh in a subchroot. 1) pacman -S lilo -r root Notice no error 2) rm root/bin/sh ; pacman -S lilo -r root Notice an error : error: scriptlet failed to execute correctly Actually, we already get an explicit error here, when popen is run, so there is no need to check for bin/sh explicitely. Besides this check was problematic in some cases. For example, bash itself has a scriptlet, but only post_install and post_upgrade, no pre_install and pre_upgrade. However, since bash has a scriptlet, runscriptlet will also be called before bash is installed. It won't do anything since the scriptlet has no pre_install function. But if we keep the check, we will still get "error : no /bin/sh". Conflicts: lib/libalpm/trans.c Signed-off-by: Xavier Chantry <shiningxc@gmail.com> Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'lib/libalpm/util.c')
0 files changed, 0 insertions, 0 deletions