Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 65729 - pingus doesn't merge properly with bash-3.0-r6 [with fix]
Summary: pingus doesn't merge properly with bash-3.0-r6 [with fix]
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Gentoo Games
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-09-28 14:42 UTC by Colin Macdonald
Modified: 2004-09-30 04:38 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
bash-3.0-braceexpansion-fix.patch (bash-3.0-braceexpansion-fix.patch,298 bytes, text/plain)
2004-09-29 18:49 UTC, Lars Wendler (Polynomial-C) (RETIRED)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Colin Macdonald 2004-09-28 14:42:18 UTC
After building, emerge pingus dies with the following error:
make[1]: Leaving directory `/var/tmp/portage/pingus-0.6.0-r1/work/pingus-0.6.0'
mv: missing file argument
Try `mv --help' for more information.
nls
man:
gzipping man page: pingus.6
.
.
.
(but processes finishes "successfully")

An I ended up with a "/usr/games/games" dir with the pingus executable in their instead of in "/usr/games/bin" with some other games I have installed.

This is related to the "POS install process" :)

src_install() {
        make install DESTDIR=${D} || die "make install failed"
        # pos install process ... FIXME
        mv "${D}/usr/games/"{games,bin}
        cd "${D}/usr/share/games"
        use nls && mv locale ../
        mv games/pingus .
        rm -rf games
        # end pos install process
        prepgamesdirs
}

The mv line expands correctly on my desktop system but not on my desktop:

LAPTOP:
nilok@belladonna ~ $ echo "${D}/usr/games/"{games,bin}
temp/usr/games/{games,bin}
nilok@belladonna ~ $ bash --version
GNU bash, version 3.00.13(1)-release (i686-pc-linux-gnu)
Copyright (C) 2004 Free Software Foundation, Inc.

DESKTOP:
nilok@aconite ~ $ echo "${D}/usr/games/"{games,bin}
temp/usr/games/games temp/usr/games/bin
nilok@aconite ~ $ bash --version
GNU bash, version 3.00.0(1)-release (i686-pc-linux-gnu)
Copyright (C) 2004 Free Software Foundation, Inc.
nilok@aconite ~ $

So I guess its a bash version thing.  Anyway, long story short, change the mv line from:
mv "${D}/usr/games/"{games,bin}
to
mv ${D}/usr/games/games ${D}/usr/games/bin
and stop trying to use fancy bash script that I don't understand ;-)

Also, should mv commands and stuff like that have a "| die" after them.  This bug wouldn't been a lot more obivious if they did...
Comment 1 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2004-09-29 18:49:47 UTC
Created attachment 40743 [details]
bash-3.0-braceexpansion-fix.patch

Hi,

Aron Griffis already reported this bug to Chet Ramey (bash maintainer). And
Chet already provided a fix for this. It's short but not yet reported here, so
I do this :)

Poly
Comment 2 Mr. Bones. (RETIRED) gentoo-dev 2004-09-30 04:38:29 UTC
straight-forward is better.  I just changed it to bash code that'll work with both versions.