Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 117125 - games-fps/quake1-data fails with my original Quake CD-Rom
Summary: games-fps/quake1-data fails with my original Quake CD-Rom
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Games (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Games
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-29 14:18 UTC by Henrique Rodrigues
Modified: 2006-01-04 05:56 UTC (History)
0 users

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


Attachments
Patch to make lha unpack correctly (lha_unpack_patch,591 bytes, patch)
2006-01-03 16:26 UTC, Henrique Rodrigues
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Henrique Rodrigues 2005-12-29 14:18:24 UTC
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.
Comment 1 Chris Gianelloni (RETIRED) gentoo-dev 2006-01-03 06:56:28 UTC
Should be fixed in CVS...
Comment 2 Henrique Rodrigues 2006-01-03 16:24:18 UTC
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.
Comment 3 Henrique Rodrigues 2006-01-03 16:26:38 UTC
Created attachment 76119 [details, diff]
Patch to make lha unpack correctly

Here's the patch to make the temporary filename lha friendly, so that it unpacks cleanly.
Comment 4 Chris Gianelloni (RETIRED) gentoo-dev 2006-01-04 05:56:23 UTC
Fixed... thanks for doing the research to get this resolved...