Bugzilla Bug 117125

I've been trying to emerge quake1-data with no success. Here's the contents of
my Quake CD-Rom:

$ ls -R /media/cdrecorder/
/media/cdrecorder/:
deice.exe  install.bat  q101_int.1  q101_int.2  q101_int.dat  readintr.txt  sw

/media/cdrecorder/sw:
deice.exe    qsw101.1  qsw101.3  qsw101.5  qsw101.7    readints.txt
install.bat  qsw101.2  qsw101.4  qsw101.6  qsw101.dat


Looking at the src_unpack() function in the quake1-data ebuild, it seems to be
a supported Quake CD-ROM, because of the q101_int.x files. This is the error
message:

# CD_ROOT=/media/cdrecorder emerge -v quakeforge
Calculating dependencies   ...done!
>>> emerge (1 of 2) games-fps/quake1-data-2.40 to /
>>> md5 files   ;-) quake1-data-2.40.ebuild
>>> md5 files   ;-) files/digest-quake1-data-2.40
 * Found CD #1 root at /media/cdrecorder
>>> Unpacking source...
>>> Unpacking q101_int to /var/tmp/portage/quake1-data-2.40/work
>>> Source unpacked.
>>> Test phase [not enabled]: games-fps/quake1-data-2.40

>>> Install quake1-data-2.40 into /var/tmp/portage/quake1-data-2.40/image/ category games-fps
cp: cannot stat `/media/cdrecorder/Setup/ID1/PAK0.PAK': No such file or
directory
install: cannot stat `/var/tmp/portage/quake1-data-2.40/temp/pak0.pak': No such
file or directory


Looking again at the ebuild, I believe quake1-data doesn't detect correcly the
type of my Quake CD-Rom. By the way, is this snipet of the ebuild's 
src_unpack() correct?

cat "${CDROM_ROOT}"/q101_int.1 "${CDROM_ROOT}"/q101_int.2 > q101_int
lha xqf "${CDROM_ROOT}"/q101_int || die "failure unpacking q101_int"
rm -f q101_int

The second line doesn't appear to use the q101_int file created on the first
line.

Should be fixed in CVS...

Thank you, but it seems the fix applied is not sufficient. The error now shown
is this:


# CD_ROOT=/media/cdrecorder emerge -v quakeforge
Calculating dependencies   ...done!
>>> emerge (1 of 2) games-fps/quake1-data-2.40 to /
>>> md5 files   ;-) quake1-data-2.40.ebuild
>>> md5 files   ;-) files/digest-quake1-data-2.40
 * Found CD #1 root at /media/cdrecorder
>>> Unpacking source...
>>> Unpacking q101_int to /var/tmp/portage/quake1-data-2.40/work
LHa: Warning: Checksum error (LHarc file?)
LHa: Error: Unknown method skiped ...
LHa: Fatal error: Unknown level header

!!! ERROR: games-fps/quake1-data-2.40 failed.
!!! Function src_unpack, Line 32, Exitcode 1
!!! failure unpacking q101_int
!!! If you need support, post the topmost build error, NOT this status message.


I've been playing around with the ebuild and found out that lha cannot
uncompress a self-extractable file unless it's name ends in ".exe". So, if we
change the filename q101_int to q101_int.exe, it works allright.

Created an attachment (id=76119) [edit]
Patch to make lha unpack correctly

Here's the patch to make the temporary filename lha friendly, so that it
unpacks cleanly.

Fixed... thanks for doing the research to get this resolved...