guix-patches
[Top][All Lists]
Advanced

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

[bug#40862] gnu: Add xarchiver. (v2)


From: Raghav Gururajan
Subject: [bug#40862] gnu: Add xarchiver. (v2)
Date: Sun, 26 Apr 2020 05:53:05 -0400

Hello Mathieu!

> Thanks for this patch. When running 'guix lint' on this patch, I have
> the following warnings:
> 
> --8<---------------cut here---------------start------------->8---
> /home/mathieu/guix-master/gnu/packages/compression.scm:87:5:
> xarchiver@0.5.4.14: the source file name should contain the package name
> /home/mathieu/guix-master/gnu/packages/compression.scm:87:5:
> xarchiver@0.5.4.14: the source URI should not be an autogenerated tarball
> fetching CVE database for 2020...... fetching CVE database for 2019...
> fetching CVE database for 2018... fetching CVE database for 2017...
> /home/mathieu/guix-master/gnu/packages/compression.scm:85:13:
> xarchiver@0.5.4.14: can be upgraded to 0.5b2 --8<---------------cut
> here---------------end--------------->8---
> 
> >  
> > +(define-public xarchiver
> > +  (package
> > +    (name "xarchiver")
> > +    (version "0.5.4.14")
> > +    (source
> > +     (origin
> > +       (method url-fetch)
> > +       (uri
> > +        (string-append "https://github.com/ib/"; name "/archive/"
> > +                       version ".tar.gz"))  
> 
> You should use "git-fetch" method for github repositories (see linter
> warning above).
> 
> > +       (list
> > +        "--disable-plugin")))           ; Not required  
> 
> Why is it not required?
> 
> > +    (synopsis "Graphical Front-end for Archive Operations")  
> 
> No camel case notation.
> 
> > +    (description "Xarchiver is the ultimate solution for handling
> > archives.  It +is a desktop-environment independent archiver front end.")  
> 
> "Ultimate" is maybe exaggerated! Also you could elaborate a bit on the
> functionalities of this tool.

Please find the revised patch attached with this email.

Regards,
RG.

Attachment: 0001-gnu-Add-xarchiver.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]