[Bug 1120771] Review Request: uronode - Alternative packet radio system for Linux
bugzilla at redhat.com
bugzilla at redhat.com
Fri Nov 28 14:17:07 UTC 2014
https://bugzilla.redhat.com/show_bug.cgi?id=1120771
Jaroslav Škarvada <jskarvad at redhat.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flags|needinfo?(jskarvad at redhat.c |
|om) |
--- Comment #5 from Jaroslav Škarvada <jskarvad at redhat.com> ---
(In reply to Jan Synacek from comment #4)
Thanks for the review.
> [ ]: Package is licensed with an open-source compatible license and meets
> other legal requirements as defined in the legal section of Packaging
> Guidelines.
> [ ]: License field in the package spec file matches the actual license.
> Note: Checking patched sources after %prep for licenses. Licenses found:
> "GPL (v2 or later) (with incorrect FSF address)", "Unknown or
> generated".
> 22 files have unknown license. Detailed output of licensecheck in
> /home/jsynacek/work/reviews/uronode/1120771-uronode/licensecheck.txt
>
>
> It looks like the legal question hasn't been sorted out yet.
>
I will sort it out, somehow :)
> [ ]: Package must own all directories that it creates.
> Note: Directories without known owners: /etc/ax25
>
>
> This directory is missing in %files.
>
Bug 1168929.
>
>
> [ ]: Spec file is legible and written in American English.
>
>
> Almost. In the description, s/colours/colors/.
>
Oxford Dictionary spelling. No problem to switch it to American.
> [ ]: Package contains systemd file(s) if in need.
>
>
> Yes. There is also a xinetd file, which could be easily substituted with a
> systemd service/socket pair. It's not a problem, but IMHO a preferred
> solution.
>
NP, I will add it.
> [ ]: Useful -debuginfo package or justification otherwise.
>
>
> Debuginfo is empty...
>
Thanks for catching, there were hidden strips :)
> uronode.x86_64: E: incorrect-fsf-address /usr/share/doc/uronode/LICENSE
>
>
> Please, notify upstream about this.
>
Upstream was already notified.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
More information about the package-review
mailing list