Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 476734

Summary: app-text/acroread shouldn't use unpack on .TAR files
Product: Gentoo Linux Reporter: David Leverton <levertond>
Component: Current packagesAssignee: Printing Team <printing>
Status: RESOLVED FIXED    
Severity: normal CC: hypnos75
Priority: Normal Keywords: InVCS
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 476738    

Description David Leverton 2013-07-13 18:01:09 UTC
app-text/acroread is using the unpack function to extract .TAR files, however support for .TAR (upper-case) is a Portage-specific feature not in PMS and not supported by Pkgcore or Paludis, so it should be changed back to using tar xf.

I've filed bug 476730 to allow this in a future EAPI, so hopefully it'll be available in the not-hugely-distant future.
Comment 1 Hypnos 2013-07-13 18:13:49 UTC
Installation of acroread-9.5.5 fails for me under paludis, as unpack says it is "skipping" ILNXR.TAR and COMMON.TAR .  Changing the line in the ebuild to "tar xf ./ILNXR.TAR; tar xf ./COMMON.TAR" works around the problem.

However, what's strange is that acroread-9.5.4 had the identical ebuild, and it installed just fine under Paludis.
Comment 2 Zac Medico gentoo-dev 2013-07-13 19:07:42 UTC
This is fixed in cvs:

--- acroread-9.5.5.ebuild
+++ acroread-9.5.5.ebuild
@@ -87,7 +87,10 @@
 src_unpack() {
 	unpack ${A}
 	cd "${S}"
-	unpack ./ILINXR.TAR ./COMMON.TAR
+	# lowercase tar extension required for unpack, bug 476734
+	mv ./ILINXR.TAR ./ILINXR.tar || die
+	mv ./COMMON.TAR ./COMMON.tar || die
+	unpack ./ILINXR.tar ./COMMON.tar
 }
 
 src_prepare() {