[Bug 469134] Review Request: deco-archive - Extraction scripts for various archive formats for use of deco

bugzilla at redhat.com bugzilla at redhat.com
Mon Dec 1 15:01:54 UTC 2008


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=469134





--- Comment #6 from Ville Skyttä <ville.skytta at iki.fi>  2008-12-01 10:01:53 EDT ---
Created an attachment (id=325238)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=325238)
Simplify specfile

Here's a patch that simplifies the specfile somewhat, works for me on F-9.  I
don't think it's necessary to change anything else but [ and \ to ?.

Regarding the default dependencies: I can see the point for gzipped tarballs
and friends, the nested logic would indeed be ugly and possibly fragile.  But
I'd still trim at least gzip and bzip2 from the list (ditto maybe cpio; I
suppose deco has no support for compressed cpio files) - the logic for handling
those is already there in for example 7zip and lzma cases as long as we can
assume tar is around.  But I won't consider this a blocker if you don't agree.

But if you decide to keep the defaults, I suppose there's no need to do the
symlinking for the defaults - their dirs could be simply directly installed to
/var/lib/deco instead, no?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the Fedora-package-review mailing list