[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add ustr.
From: |
Ricardo Wurmus |
Subject: |
Re: [PATCH] Add ustr. |
Date: |
Mon, 21 Mar 2016 15:50:44 +0100 |
Leo Famulari <address@hidden> writes:
> On Thu, Mar 17, 2016 at 01:13:33PM +0100, Ricardo Wurmus wrote:
>> * gnu/packages/textutils.scm (ustr): New variable.
>
> [...]
>
>> + (uri (string-append "http://www.and.org/ustr/" version
>> + "/ustr-" version ".tar.bz2"))
[...]
> It looks like this directory only contains the current version of the
> code, but I can't find an archive of old releases. How about mentioning
> this in a comment, and (maybe) asking and.org to provide an archive?
Actually, there are older versions as well. Version 1.0.2, for example,
is at
http://www.and.org/ustr/1.0.2/ustr-1.0.2.tar.bz2
>> + (build-system gnu-build-system)
>> + (arguments
>> + `(#:make-flags
>> + (list "CC=gcc"
>> + "HIDE="
>
> I read the Makefile's comment about this. ;)
I guess we don’t qualify as “all sane people” :)
>> + "LDCONFIG=echo"
>
> Can you explain this one in a comment?
Will do.
>> + (string-append "prefix=" (assoc-ref %outputs "out"))
>> + "all-shared")
>
> [...]
>
>> + ;; Quoted from the home page: "The License for the code is MIT, new-BSD,
>> + ;; LGPL, etc. ... if you need another license to help compatibility,
>> just
>> + ;; ask for it. It's basically public domain, without all the legal
>> + ;; problems for everyone that trying to make something public domain
>> + ;; entails."
>> + (license license:public-domain)))
>
> Sounds good to me. Hopefully there are no objections from the legal
> experts!
Thanks for taking the time to review! I’ll push this after adding the
comment.
~~ Ricardo