https://bugzilla.rpmfusion.org/show_bug.cgi?id=4647

--- Comment #18 from Dominik 'Rathann' Mierzejewski <[email protected]> ---
(In reply to Richard from comment #17)
> SPEC: https://www.dropbox.com/s/a3fy23xaw163ap5/unifi.spec
> SRPM: https://www.dropbox.com/s/s0lbqv7xxslf9g4/unifi-5.6.29-1.fc27.src.rpm
> 
> * Tue Jan 02 2018 Richard Shaw <[email protected]> - 5.6.29-1
> - Update to latest upstream release.
>  
> https://community.ubnt.com/t5/UniFi-Updates-Blog/UniFi-5-6-29-Stable-has-
> been-released/ba-p/2191996

SPEC and SRPM spec differ:

$ diff -u unifi.spec unifi\(1\).spec 
--- SRPM/unifi.spec     2018-01-03 00:53:54.000000000 +0100
+++ SPEC/unifi.spec     2018-01-17 11:52:07.684622573 +0100
@@ -263,6 +263,7 @@
 %changelog
 * Tue Jan 02 2018 Richard Shaw <[email protected]> - 5.6.29-1
 - Update to latest upstream release.
+ 
https://community.ubnt.com/t5/UniFi-Updates-Blog/UniFi-5-6-29-Stable-has-been-released/ba-p/2191996

 * Fri Dec 08 2017 Richard Shaw <[email protected]> - 5.6.26-1
 - Update to latest upstream release. For release notes see:

You unbundled the fontawesome font but didn't add Requires: for it:
Requires:       fontawesome-fonts
Requires:       fontawesome-fonts-web
Could you explain why only the ttf version was unbundled and why the above are
not added?

As far as I can tell they're referenced in webapps/ROOT/app-unifi/css/app.css,
which might need to be patched.

rpmlint complains about the License field.
unifi.x86_64: W: invalid-license Proprietary

URL: http://www.ubnt.com/unifi
returns 404 error now. I think it should be changed to:
https://www.ubnt.com/software/

As the current EULA (JULY 17, 2017) prohibits distribution without their
explicit permission, I think you must include the written permission with the
package. I guess including PDFs of
https://community.ubnt.com/t5/forums/forumtopicprintpage/board-id/UniFi/message-id/180009/print-single-message/false/page/1
and
https://community.ubnt.com/t5/forums/forumtopicprintpage/board-id/UniFi/message-id/194230/print-single-message/false/page/1
would be enough.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
rpmfusion-developers mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to