Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 476734 - app-text/acroread shouldn't use unpack on .TAR files
Summary: app-text/acroread shouldn't use unpack on .TAR files
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Printing Team
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks: 476738
  Show dependency tree
 
Reported: 2013-07-13 18:01 UTC by David Leverton
Modified: 2013-07-13 19:07 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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() {