Re: guix lint questions

[Tobias Geerinckx-Rice]

mikadoZero,

mikadoZero wrote:
> I have three questions about the output of:
> `./pre-inst-env guix lint emacs-ace-link`
>
> 1)  What is the significance of:
>
> ;;; note: source file 
> /home/guix/u/guix/guix/gnu/packages/image-processing.scm
> ;;;       newer than compiled 
> /home/guix/u/guix/guix/gnu/packages/image-processing.g
> o                                                                                  
> ;;; note: source file 
> /home/guix/u/guix/guix/gnu/packages/image-processing.scm
> ;;;       newer than compiled 
> /run/current-system/profile/lib/guile/2.2/site-ccache/
> gnu/packages/image-processing.go                                                   
>
> This shows up many times for different scm files.

There is none.  These are mere warnings that the pre-built object 
files no longer match the source files you have checked out 
(i.e. the sources have changed).  They have nothing to do with 
your package.

You can use ‘guix environment guix -- make’ to make them go away 
if they really annoy you.

I just ignore them.

> 2)  How should this be addressed?
>
> gnu/packages/emacs-xyz.scm:1099:5: address@hidden: the 
> source URI should not b
> e an autogenerated tarball
>
> Line 1099 is referring to origin.

You're downloading a ‘github.com/…/achive/…’ tarball.  These 
tarballs aren't guaranteed to be stable over time (GitHub can 
regenerate them, changing the metadata and hence the hash, and has 
done so in the past).

They must not be used.

Instead, use (method git-fetch), (git-reference …) and friends to 
download the git repository directly.  Tonnes of examples of this 
in Guix already, unfortunately, since many projects can't be 
bothered to make proper releases…  GitHub culture?

See ‘progress’ in gnu/packages/admin.scm for an example.  Don't 
forget to add (file-name (git-file-name name version)) when doing 
this, by the way.

> 3)  How should this be addressed?
>
> /home/guix/u/guix/guix/gnu/packages/emacs-xyz.scm:1095:2: 
> address@hidden: line
>  1113 is way too long (417 characters)
>
> Line 1113 is referring to the text of the description.
>
> Looking at other packages in emacs-xyz.scm there are other 
> packages that
> have longer descriptions than the one here for emacs-ace-link.

Yes, but they're not all on one single line.  ;-)

Add a few well-placed newlines to wrap the text to eighty 
characters and the warning will go away.

Kind regards,

T G-R