Marius Bakke writes: >> + (modify-phases %standard-phases >> + ;; Remove faulty python-requests version check. This has been fixed >> + ;; upstream, and can be removed in version 9.4. >> + (add-after 'unpack 'remove-python-requests-version >> + (lambda _ >> + (substitute* "linkcheck/__init__.py" >> + (("requests.__version__ <= '2.2.0'") "False"))))))) > > Please end this phase on a #t since (substitute* ...) has an unspecified > return value. Done. >> + (license (list l:gpl2+ >> + l:bsd-2 ; linkcheck/better_exchook2.py >> + l:bsd-3 ; linkcheck/colorama.py >> + l:psfl ; linkcheck/gzip2.py >> + l:expat ; linkcheck/mem.py >> + l:isc ; third_party/dnspython >> + l:asl2.0)))) ; third_party/miniboa > > Is it possible to remove these third party libraries and use packaged > copies instead? At least 'dnspython' is already in Guix. linkchecker is using a modified version of dnspython and miniboa. So, I don't think it will be easy. For a list of modifications, look at https://github.com/wummel/linkchecker/blob/master/third_party/dnspython/changelog.txt and https://github.com/wummel/linkchecker/blob/master/third_party/miniboa-r42/CHANGES.txt > Also, since GPL2+ is stricter than the other licenses, the program as a > whole is effectively GPL2+, so we don't really need to mention those > one-off source files in the license field (but comments are great). But, we generally don't hide this information from the user, right? Many Guix packages have licenses of even single files mentioned in the `license' field. Do you still think I should remove them? > Not sure about the 3rd party bundles though, since they probably don't > depend on anything from the GPL2+ distribution. If they are installed, > I guess we should mention them with a FIXME. I've added a FIXME. I'm attaching a new patch with the discussed changes. Do let me know if everything looks good.