<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "http://bugs.gentoo.org/bugzilla.dtd">

<bugzilla version="2.22.7"
          urlbase="http://bugs.gentoo.org/"
          maintainer="bugzilla@gentoo.org"
>

    <bug>
          <bug_id>117125</bug_id>
          
          <creation_ts>2005-12-29 14:18 0000</creation_ts>
          <short_desc>games-fps/quake1-data fails with my original Quake CD-Rom</short_desc>
          <delta_ts>2006-01-04 05:56:23 0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>Gentoo Linux</product>
          <component>Games</component>
          <version>unspecified</version>
          <rep_platform>All</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>henrique.rodrigues@ist.utl.pt</reporter>
          <assigned_to>games@gentoo.org</assigned_to>
          

      

      
          <long_desc isprivate="0">
            <who>henrique.rodrigues@ist.utl.pt</who>
            <bug_when>2005-12-29 14:18:24 0000</bug_when>
            <thetext>I&apos;ve been trying to emerge quake1-data with no success. Here&apos;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!
&gt;&gt;&gt; emerge (1 of 2) games-fps/quake1-data-2.40 to /
&gt;&gt;&gt; md5 files   ;-) quake1-data-2.40.ebuild
&gt;&gt;&gt; md5 files   ;-) files/digest-quake1-data-2.40
 * Found CD #1 root at /media/cdrecorder
&gt;&gt;&gt; Unpacking source...
&gt;&gt;&gt; Unpacking q101_int to /var/tmp/portage/quake1-data-2.40/work
&gt;&gt;&gt; Source unpacked.
&gt;&gt;&gt; Test phase [not enabled]: games-fps/quake1-data-2.40

&gt;&gt;&gt; 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&apos;: No such file or directory
install: cannot stat `/var/tmp/portage/quake1-data-2.40/temp/pak0.pak&apos;: No such file or directory


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

cat &quot;${CDROM_ROOT}&quot;/q101_int.1 &quot;${CDROM_ROOT}&quot;/q101_int.2 &gt; q101_int
lha xqf &quot;${CDROM_ROOT}&quot;/q101_int || die &quot;failure unpacking q101_int&quot;
rm -f q101_int

The second line doesn&apos;t appear to use the q101_int file created on the first line.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>wolf31o2@gentoo.org</who>
            <bug_when>2006-01-03 06:56:28 0000</bug_when>
            <thetext>Should be fixed in CVS...</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>henrique.rodrigues@ist.utl.pt</who>
            <bug_when>2006-01-03 16:24:18 0000</bug_when>
            <thetext>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!
&gt;&gt;&gt; emerge (1 of 2) games-fps/quake1-data-2.40 to /
&gt;&gt;&gt; md5 files   ;-) quake1-data-2.40.ebuild
&gt;&gt;&gt; md5 files   ;-) files/digest-quake1-data-2.40
 * Found CD #1 root at /media/cdrecorder
&gt;&gt;&gt; Unpacking source...
&gt;&gt;&gt; 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&apos;ve been playing around with the ebuild and found out that lha cannot uncompress a self-extractable file unless it&apos;s name ends in &quot;.exe&quot;. So, if we change the filename q101_int to q101_int.exe, it works allright.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>henrique.rodrigues@ist.utl.pt</who>
            <bug_when>2006-01-03 16:26:38 0000</bug_when>
            <thetext>Created an attachment (id=76119)
Patch to make lha unpack correctly

Here&apos;s the patch to make the temporary filename lha friendly, so that it unpacks cleanly.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>wolf31o2@gentoo.org</who>
            <bug_when>2006-01-04 05:56:23 0000</bug_when>
            <thetext>Fixed... thanks for doing the research to get this resolved...</thetext>
          </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>76119</attachid>
            <date>2006-01-03 16:26 0000</date>
            <desc>Patch to make lha unpack correctly</desc>
            <filename>lha_unpack_patch</filename>
            <type>text/plain</type>
            <data encoding="base64">LS0tIHF1YWtlMS1kYXRhLTIuNDAuZWJ1aWxkCTIwMDYtMDEtMDQgMDA6MTM6MzAuMDAwMDAwMDAw
ICswMDAwCisrKyBxdWFrZTEtZGF0YS0yLjQwLXIxLmVidWlsZAkyMDA2LTAxLTA0IDAwOjEzOjUx
LjAwMDAwMDAwMCArMDAwMApAQCAtMjUsMTIgKzI1LDEyIEBACiB9CiAKIHNyY191bnBhY2soKSB7
Ci0JZWNobyAiPj4+IFVucGFja2luZyBxMTAxX2ludCB0byAke1BXRH0iCisJZWNobyAiPj4+IFVu
cGFja2luZyBxMTAxX2ludC5leGUgdG8gJHtQV0R9IgogCWlmIFtbICR7Q0RST01fU0VUfSA9PSAi
MSIgXV0gOyB0aGVuCiAJCWNhdCAiJHtDRFJPTV9ST09UfSIvcTEwMV9pbnQuMSAiJHtDRFJPTV9S
T09UfSIvcTEwMV9pbnQuMiA+IFwKLQkJCSIke1N9Ii9xMTAxX2ludAotCQlsaGEgeHFmICIke1N9
Ii9xMTAxX2ludCB8fCBkaWUgImZhaWx1cmUgdW5wYWNraW5nIHExMDFfaW50IgotCQlybSAtZiBx
MTAxX2ludAorCQkJIiR7U30iL3ExMDFfaW50LmV4ZQorCQlsaGEgeHFmICIke1N9Ii9xMTAxX2lu
dC5leGUgfHwgZGllICJmYWlsdXJlIHVucGFja2luZyBxMTAxX2ludC5leGUiCisJCXJtIC1mIHEx
MDFfaW50LmV4ZQogCWZpCiB9CiAK
</data>        

          </attachment>
    </bug>

</bugzilla>