[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 5/6] gnu: Add ceph.
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH 5/6] gnu: Add ceph. |
Date: |
Tue, 17 Jan 2017 23:47:54 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Marius Bakke <address@hidden> skribis:
> * gnu/packages/distributed-filesystems.scm: New file.
> * gnu/local.mk (GNU_SYSTEM_MODULES): Add it.
[...]
> + (substitute* "src/Makefile.in"
> + ;; By default, the PYTHONPATH is set to "src/pybind".
> + ;; Make sure our libraries are found too.
> + (("export PYTHONPATH=") (string-append "export PYTHONPATH="
> + (getenv "PYTHONPATH")
> + ":"))
I’d move (string-append …) on the next line (for the following
substitutions as well.)
> + ;; Ceph is licensed under LGPL2.1, but includes a number of components
> + ;; covered by other licenses. Consult COPYING for full information.
> + (license (list license:lgpl2.1
> + license:cc-by-sa3.0 ; documentation
> + license:bsd-2 ; xxHash
> + license:bsd-3
> + license:gpl3
> + license:gpl2
> + license:public-domain
> + license:boost1.0
> + license:expat))))
No “or any later version” clause for the GNU licenses here?
Then apart from the wrapping that you mentioned in the cover letter and
other issues, that looks good. Great work!
Ludo’.
- Re: [PATCH 2/6] gnu: Add crypto++., (continued)
- [PATCH 5/6] gnu: Add ceph., Marius Bakke, 2017/01/15
- Re: [PATCH 5/6] gnu: Add ceph.,
Ludovic Courtès <=
[PATCH 6/6] gnu: fio: Enable rbd support., Marius Bakke, 2017/01/15