Bugzilla Bug 198247

When the minimal flag is set, dropbear's scp will not be build:

        use minimal \
                                        && progs="dropbear dbclient
dropbearkey" \
                                        || progs="dropbear dbclient dropbearkey
dropbearconvert scp"

Yet further on, during install, it is always installed, regardless of
the minimal use flag or the value of progs:

        mv "${D}"/usr/bin/{,db}scp || die

In the previous version of the ebuild (0.49) the "|| die" part was not
there, so this silently failed, still allowing the ebuild to install.
Now, dropbear always fails when the minimal use flag is set.

This would probably be fixed by adding a guarding if around the offending mv
command:

        if [ -e "${D}/usr/bin/scp" ]; then
            mv "${D}"/usr/bin/{,db}scp || die
        fi

This suggestion is untested, since I do not yet possess enough portage
voodoo to circumvent the ebuild checksums to try this. But, AFAICS, this
would fix the situation, also when the multicall use flag is set (which
is why I used -e instead of -f).


Reproducible: Always

Steps to Reproduce:
1. Unmask dropbear-0.50
2. USE=minimal emerge dropbear
Actual Results:  
cannot stat `/var/tmp/portage/net-misc/dropbear-0.50/image//usr/bin/scp': No
such file or directory
 *
 * ERROR: net-misc/dropbear-0.50 failed.
 * Call stack:
 *              ebuild.sh, line 1695:  Called dyn_install
 *              ebuild.sh, line 1132:  Called qa_call 'src_install'
 *              ebuild.sh, line   44:  Called src_install
 *   dropbear-0.50.ebuild, line   78:  Called die
 * The specific snippet of code:
 *      mv "${D}"/usr/bin/{,db}scp || die
 *  The die message:
 *   (no error message)
 *


Portage 2.1.3.16 (uclibc/x86, gcc-4.1.2, uclibc-0.9.28.3-r0,
2.6.23-gdb26fd2a-dirty i686)
=================================================================
System uname: 2.6.23-gdb26fd2a-dirty i686 AMD Turion(tm) 64 Mobile Technology
MT-30
Timestamp of tree: Mon, 05 Nov 2007 17:30:01 +0000
app-shells/bash:     3.2_p17
dev-lang/python:     2.4.4-r6
dev-python/pycrypto: 2.0.1-r6
sys-apps/baselayout: 1.12.9-r2
sys-apps/sandbox:    1.2.18.1-r2
sys-devel/autoconf:  2.61-r1
sys-devel/automake:  1.10
sys-devel/binutils:  2.18-r1
sys-devel/gcc-config: 1.3.16
sys-devel/libtool:   1.5.24
virtual/os-headers:  2.6.22-r2
ACCEPT_KEYWORDS="x86"
CBUILD="i386-gentoo-linux-uclibc"
CFLAGS="-Os -pipe -march=k6 -mmmx"
CHOST="i386-gentoo-linux-uclibc"
CONFIG_PROTECT="/etc"
CONFIG_PROTECT_MASK="/etc/env.d /etc/gconf /etc/terminfo /etc/udev/rules.d"
CXXFLAGS="-Os -pipe"
DISTDIR="/usr/portage/distfiles"
FEATURES="autoconfig buildpkg distlocks metadata-transfer nodoc noinfo noman
sandbox sfperms strict unmerge-orphans userfetch"
GENTOO_MIRRORS="http://distfiles.gentoo.org
http://distro.ibiblio.org/pub/linux/distributions/gentoo"
LDFLAGS="-Wl,-z,relro"
MAKEOPTS="-j2"
PKGDIR="/usr/packages"
PORTAGE_RSYNC_OPTS="--recursive --links --safe-links --perms --times --compress
--force --whole-file --delete --delete-after --stats --timeout=180
--exclude=/distfiles --exclude=/local --exclude=/packages
--filter=H_**/files/digest-*"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
SYNC="rsync://rsync.gentoo.org/gentoo-portage"
USE="bitmap-fonts bsdpty cli cracklib dri midi minimal mmx mudflap ncurses
openmp pcre readline reflection session spl truetype-fonts type1-fonts uclibc
x86 xorg zlib" ALSA_PCM_PLUGINS="adpcm alaw asym copy dmix dshare dsnoop empty
extplug file hooks iec958 ioplug ladspa lfloat linear meter mulaw multi null
plug rate route share shm softvol" ELIBC="uclibc" INPUT_DEVICES="keyboard mouse
evdev" KERNEL="linux" LCD_DEVICES="bayrad cfontz cfontz633 glk hd44780 lb216
lcdm001 mtxorb ncurses text" USERLAND="GNU" VIDEO_CARDS="dummy fbdev v4l"
Unset:  CPPFLAGS, CTARGET, EMERGE_DEFAULT_OPTS, INSTALL_MASK, LANG, LC_ALL,
LINGUAS, PORTAGE_COMPRESS, PORTAGE_COMPRESS_FLAGS, PORTAGE_RSYNC_EXTRA_OPTS,
PORTDIR_OVERLAY

Created an attachment (id=139572) [edit]
simple fix

Same problem here, I think a appropriate fix is to only do the mv when the
minimal use-flag is unset.

I can confirm this.

fixed in cvs