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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxx>  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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]