guix-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] gnu: Add lmms


From: Rodger Fox
Subject: Re: [PATCH] gnu: Add lmms
Date: Thu, 23 Feb 2017 10:13:54 -0800
User-agent: Roundcube Webmail/1.0.6

 It's a good idea to run "./pre-inst-env guix lint PACKAGENAME" which
 will pick up long lines for example.
 I have a ruler/colored line at character 83 in vim, but I'm not yet
 ready with the setup to write correct code again in vim. There's
 a similar thing for emacs.

 I would replace the third or second "It" with @code{LMMS} or simply
 LMMS.

Thanks for pointing that out. I missed that pre-inst-env tool in
the manual until you just mentioned it here. That helps a lot.
My corrected patch is attached. It passes the linter this time.

-Rodger Fox

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


reply via email to

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