159100 2006-12-25 15:23 0000 lha-114i-r6 cannot unpack quake1-demodata 2007-07-27 23:11:23 0000 1 1 1 Unclassified Gentoo Linux Games unspecified All Linux RESOLVED FIXED P2 normal --- 1 jkt@gentoo.org usata@gentoo.org astralsin@gmail.com brad@oatmail.org brebs@sent.com games@gentoo.org ikelos@gentoo.org john_r_graham@mindspring.com vapier@gentoo.org voyageur@gentoo.org jkt@gentoo.org 2006-12-25 15:23:06 0000 velbloud ~ # emerge -av quake1-demodata These are the packages that would be merged, in order: Calculating dependencies... done! [ebuild N ] games-fps/quake1-demodata-1.06 USE="-symlink" 0 kB Total size of downloads: 0 kB Would you like to merge these packages? [Yes/No] >>> Emerging (1 of 1) games-fps/quake1-demodata-1.06 to / * quake106.zip MD5 ;-) ... [ ok ] * quake106.zip RMD160 ;-) ... [ ok ] * quake106.zip SHA1 ;-) ... [ ok ] * quake106.zip SHA256 ;-) ... [ ok ] * quake106.zip size ;-) ... [ ok ] * checking ebuild checksums ;-) ... [ ok ] * checking auxfile checksums ;-) ... [ ok ] * checking miscfile checksums ;-) ... [ ok ] * checking quake106.zip ;-) ... [ ok ] >>> Unpacking source... >>> Unpacking quake106.zip to /var/tmp/portage/quake1-demodata-1.06/work LHa: Error: Checksum error (LHarc file?) !!! ERROR: games-fps/quake1-demodata-1.06 failed. Call stack: ebuild.sh, line 1546: Called dyn_unpack ebuild.sh, line 708: Called src_unpack quake1-demodata-1.06.ebuild, line 45: Called die !!! lha failed !!! If you need support, post the topmost build error, and the call stack if relevant. velbloud ~ # emerge -pv lha These are the packages that would be merged, in order: Calculating dependencies... done! [ebuild R ] app-arch/lha-114i-r6 0 kB Total size of downloads: 0 kB brebs@sent.com 2006-12-26 02:10:56 0000 lha-114i-r6 is broken. The same command works in Ubuntu's Edgy, but not in Gentoo: lha e resource.1 http://packages.ubuntu.com/edgy/utils/lha voyageur@gentoo.org 2006-12-29 06:04:17 0000 Thanks for the link, I built a lha binary from ubuntu's source+diff, and indeed emerge quake1-demodata worked fine brebs@sent.com 2006-12-29 19:58:58 0000 Created an attachment (id=104968) lha-114i-r6.ebuild Compiles Debian's lha. Works with quake1-demodata. brebs@sent.com 2007-01-01 23:22:32 0000 Shouldn't this bug be reassigned? It's a problem with lha. Seems to break quake1-data also: http://forums.gentoo.org/viewtopic-t-528421.html nyhm@gentoo.org 2007-02-28 21:18:28 0000 *** Bug 168674 has been marked as a duplicate of this bug. *** vapier@gentoo.org 2007-07-09 22:09:06 0000 *** Bug 184774 has been marked as a duplicate of this bug. *** john_r_graham@mindspring.com 2007-07-22 18:10:51 0000 Well, there's good news and bad news. lha-114i-r5 *can* unpack that file. It's no longer in the Portage tree, but, perhaps it should be reinstated (and lha-114i-r6 masked) until the issues are sorted out. lha-114i-r6 is based on a different upstream tarball that's pretty significantly different. lha-114i-r5 did not use autotools and had a pretty significant list of patches. lha-114i-r6 uses autotools, incorporates all of the patches, uses getopt to parse the command line, and really includes a lot of cleanup which generally looks good. Too bad it doesn't work. Just as an aside, the brand new lha-114i-r7 that I just got put into the tree with the help of the Sunrise folks *does not* address this issue. It *does* however address *another* issue introduced with -r6. See bug #184911. - John john_r_graham@mindspring.com 2007-07-22 18:13:10 0000 Incidentally, lha-114i-r5 uses the same upstream tarball that Debian is using right now, which explains why the Debian version is working. - John john_r_graham@mindspring.com 2007-07-23 03:28:35 0000 Okay, now I am in a quandary. lha-114i-r5 worked because it was broken. lha-114i-r6 doesn't work because it's not broken. Read on. lha contains a function called archive_is_msdos_sfx1() that determines whether or not the archive file is an MS-DOS self-extracting executable archive. It does so in a very simplistic way, by comparing the last few bytes of the file name against a list of known executable file types (under DOS!). In lha-114i-r5, this function was broken and almost always returned "true" but the code to skip the embedded DOS self extraction code was smart enough to not skip a non-existent executable blob. In lha-114i-r6, the archive_is_msdos_sfx1() function was fixed and the skip code was simplified. The end result is that the code will no longer correctly process self extracting archives whose file name doesn't end in ".com", ".exe", or ".x". So the quandary is that lha-114i-r6 is behaving as designed and the file that quake1-demodata cannot extract has been erroneously renamed from something.exe to resource.1. Technically, the issue is with the quake1-demodata ebuild. But, I'm unhappy with the archive_is_msdos_sfx1() function. It's really easy to look into the contents of a file to tell if it's an MS-DOS executable. I'm working on a change to the archive_is_msdos_sfx1() function so that it depends on the contents as opposed to the name. Advice? Opinions? - John john_r_graham@mindspring.com 2007-07-23 03:44:18 0000 Created an attachment (id=125719) New quake1-demodata ebuild that works with the existing lha (114i-r6). brebs@sent.com 2007-07-23 04:07:32 0000 Created an attachment (id=125721) quake1-demodata-1.06.diff Works with Debian's lha, lha-114i-r6 and lha-114i-r7 wolf31o2@gentoo.org 2007-07-24 20:06:22 0000 *** Bug 166583 has been marked as a duplicate of this bug. *** wolf31o2@gentoo.org 2007-07-27 23:11:23 0000 OK. This is FIXED now using the patch from Paul. 104968 2006-12-29 19:58 0000 lha-114i-r6.ebuild lha-114i-r6.ebuild text/plain IyBDb3B5cmlnaHQgMTk5OS0yMDA2IEdlbnRvbyBGb3VuZGF0aW9uCiMgRGlzdHJpYnV0ZWQgdW5k ZXIgdGhlIHRlcm1zIG9mIHRoZSBHTlUgR2VuZXJhbCBQdWJsaWMgTGljZW5zZSB2MgojICRIZWFk ZXI6ICQKCmluaGVyaXQgZXV0aWxzCgpNWV9QVj0ke1BWOjA6MX0uJHtQVjoxfQpNWV9QX09SSUc9 IiR7UE59XyR7TVlfUFZ9IgpNWV9QPSIke01ZX1BfT1JJR30tMTAuMSIKREVCSUFOPSJtaXJyb3I6 Ly9kZWJpYW4vcG9vbC9ub24tZnJlZS9sIgoKREVTQ1JJUFRJT049IlV0aWxpdHkgZm9yIGNyZWF0 aW5nIGFuZCBvcGVuaW5nIGx6aCBhcmNoaXZlcyIKSE9NRVBBR0U9Imh0dHA6Ly9saGEuc291cmNl Zm9yZ2UuanAvIgpTUkNfVVJJPSIke0RFQklBTn0vJHtQTn0vJHtNWV9QX09SSUd9Lm9yaWcudGFy Lmd6Cgkke0RFQklBTn0vJHtQTn0vJHtNWV9QfS5kaWZmLmd6IgoKTElDRU5TRT0ibGhhIgpTTE9U PSIwIgpLRVlXT1JEUz0iYWxwaGEgYW1kNjQgYXJtIGhwcGEgaWE2NCBtNjhrIHBwYyBwcGMtbWFj b3MgcHBjNjQgczM5MCBzaCBzcGFyYyB4ODYgfng4Ni1mYnNkIgpJVVNFPSIiCgpSREVQRU5EPSIi CkRFUEVORD0iIgoKUz0ke1dPUktESVJ9LyR7UE59LSR7TVlfUFZ9Lm9yaWcKCnNyY191bnBhY2so KSB7Cgl1bnBhY2sgJHtBfQoKCWVwYXRjaCAuLyR7TVlfUH0uZGlmZgp9CgpzcmNfaW5zdGFsbCgp IHsKCWRvYmluIHNyYy8ke1BOfSB8fCBkaWUKCgluZXdtYW4gbWFuLyR7UE59Lm1hbiAke1BOfS5u IHx8IGRpZQoJZG9tYW4gZGViaWFuLyR7UE59LjEgfHwgZGllCgoJZG9kb2MgKi57ZXVjLHR4dH0K fQo= 125719 2007-07-23 03:44 0000 New quake1-demodata ebuild that works with the existing lha (114i-r6). quake1-demodata-1.06-r1.ebuild text/plain IyBDb3B5cmlnaHQgMTk5OS0yMDA3IEdlbnRvbyBGb3VuZGF0aW9uCiMgRGlzdHJpYnV0ZWQgdW5k ZXIgdGhlIHRlcm1zIG9mIHRoZSBHTlUgR2VuZXJhbCBQdWJsaWMgTGljZW5zZSB2MgojICRIZWFk ZXI6IC92YXIvY3Zzcm9vdC9nZW50b28teDg2L2dhbWVzLWZwcy9xdWFrZTEtZGVtb2RhdGEvcXVh a2UxLWRlbW9kYXRhLTEuMDYuZWJ1aWxkLHYgMS41IDIwMDcvMDMvMTIgMTQ6NDc6MTkgZ2Vub25l IEV4cCAkCgppbmhlcml0IGV1dGlscyB2ZXJzaW9uYXRvciBnYW1lcwoKTVlfUFY9JChkZWxldGVf YWxsX3ZlcnNpb25fc2VwYXJhdG9ycykKTVlfUE49InF1YWtlIgoKREVTQ1JJUFRJT049IkRlbW8g ZGF0YSBmb3IgUXVha2UgMSIKSE9NRVBBR0U9Imh0dHA6Ly9lbi53aWtpcGVkaWEub3JnL3dpa2kv UXVha2VfSSIKU1JDX1VSST0ibWlycm9yOi8vaWRzb2Z0d2FyZS8ke01ZX1BOfS8ke01ZX1BOfSR7 TVlfUFZ9LnppcCIKCiMgU2VlIGxpY2luZm8udHh0CkxJQ0VOU0U9InF1YWtlMS1kZW1vZGF0YSIK U0xPVD0iMCIKS0VZV09SRFM9In5hbWQ2NCB+eDg2IgpJVVNFPSJzeW1saW5rIgoKREVQRU5EPSJh cHAtYXJjaC9saGEKCWFwcC1hcmNoL3VuemlwIgoKUz0ke1dPUktESVJ9CmRpcj0ke0dBTUVTX0RB VEFESVJ9LyR7TVlfUE59MQoKcGtnX3NldHVwKCkgewoJZ2FtZXNfcGtnX3NldHVwCgoJaWYgaGFz X3ZlcnNpb24gImdhbWVzLWZwcy9xdWFrZTEtZGF0YSIgOyB0aGVuCgkJZXdhcm4gImdhbWVzLWZw cy9xdWFrZTEtZGF0YSBhbHJlYWR5IGluY2x1ZGVzIHRoZSBkZW1vIGRhdGEsIgoJCWV3YXJuICJz byB0aGlzIGluc3RhbGxhdGlvbiBpcyBub3QgdmVyeSB1c2VmdWwuIgoJCWVjaG8KCQlpZiB1c2Ug c3ltbGluayA7IHRoZW4KCQkJZWVycm9yICJUaGUgc3ltbGluayBmb3IgdGhlIGRlbW8gZGF0YSBj b25mbGljdHMgd2l0aCB0aGUgY2RpbnN0YWxsIGRhdGEiCgkJCWRpZSAiUmVtb3ZlIHRoZSAnc3lt bGluaycgVVNFIGZsYWcgZm9yIHRoaXMgcGFja2FnZSIKCQlmaQoJCWViZWVwCgkJZXBhdXNlCglm aQp9CgpzcmNfdW5wYWNrKCkgewoJdW5wYWNrICR7QX0KCgltdiByZXNvdXJjZS4xIHJlc291cmNl LmV4ZQoJbGhhIGVxIHJlc291cmNlLmV4ZSB8fCBkaWUgImxoYSBmYWlsZWQiCgkjIERvbid0IHdh bnQgdG8gY29uZmxpY3Qgd2l0aCB0aGUgY2RpbnN0YWxsIGZpbGVzCgltdiBpZDEgZGVtbwp9Cgpz cmNfaW5zdGFsbCgpIHsKCWluc2ludG8gIiR7ZGlyfSIKCWRvaW5zIC1yIGRlbW8gfHwgZGllICJk b2lucyAtciBmYWlsZWQiCgoJZG9kb2MgKi50eHQKCglpZiB1c2Ugc3ltbGluayA7IHRoZW4KCQkj IE1ha2UgdGhlIGRlbW8gdGhlIGRlZmF1bHQsIHNvIHRoYXQgcGVvcGxlIGNhbiBqdXN0IHJ1biBp dCwKCQkjIHdpdGhvdXQgaGF2aW5nIHRvIG1lc3Mgd2l0aCBjb21tYW5kLWxpbmUgb3B0aW9ucy4K CQljZCAiJHtEfS8ke2Rpcn0iICYmIGxuIC1zZm4gZGVtbyBpZDEKCWZpCgoJcHJlcGdhbWVzZGly cwp9Cgpwa2dfcG9zdGluc3QoKSB7CglnYW1lc19wa2dfcG9zdGluc3QKCgllbG9nICJUaGlzIGlz IGp1c3QgdGhlIGRlbW8gZGF0YS4iCgllbG9nICJZb3Ugd2lsbCBzdGlsbCBuZWVkIGEgUXVha2Ug MSBjbGllbnQsIHRvIHBsYXksIHN1Y2ggYXMgZGFya3BsYWNlcy4iCgllY2hvCgoJaWYgdXNlIHN5 bWxpbmsgOyB0aGVuCgkJZWxvZyAiaWQxIGhhcyBiZWVuIHN5bWxpbmtlZCB0byBkZW1vLCBmb3Ig Y29udmVuaWVuY2UsIHdpdGhpbjoiCgkJZWxvZyAiJHtkaXJ9IgoJCWVjaG8KCWZpCn0K 125721 2007-07-23 04:07 0000 quake1-demodata-1.06.diff quake1-demodata-1.06.diff text/plain LS0tIC91c3IvcG9ydGFnZS9nYW1lcy1mcHMvcXVha2UxLWRlbW9kYXRhL3F1YWtlMS1kZW1vZGF0 YS0xLjA2LmVidWlsZAkyMDA3LTAzLTEyIDE1OjA1OjQwLjAwMDAwMDAwMCArMDAwMAorKysgcXVh a2UxLWRlbW9kYXRhLTEuMDYuZWJ1aWxkCTIwMDctMDctMjMgMDQ6NTc6MzMuMDAwMDAwMDAwICsw MTAwCkBAIC00Miw3ICs0MiwxMCBAQAogc3JjX3VucGFjaygpIHsKIAl1bnBhY2sgJHtBfQogCi0J bGhhIGVxIHJlc291cmNlLjEgfHwgZGllICJsaGEgZmFpbGVkIgorCSMgRmlsZSByZW5hbWUgZm9y IGJ1ZyAjMTU5MTAwCisJbXYgcmVzb3VyY2UuezEseH0KKworCWxoYSB4ZnEgcmVzb3VyY2UueCB8 fCBkaWUgImxoYSBmYWlsZWQiCiAJIyBEb24ndCB3YW50IHRvIGNvbmZsaWN0IHdpdGggdGhlIGNk aW5zdGFsbCBmaWxlcwogCW12IGlkMSBkZW1vCiB9Cg==