[Bug 1268716] Review Request: cjdns - IP6 VPN with crypto address allocation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1268716



--- Comment #33 from Stuart D Gathman <stuart@xxxxxxxxxxx> ---
The nacl library does not need to be bundled.  I made a flag whether to use
Fedora or embedded nacl in rel 5.  I added the Provides for the embedded libuv.
 The %{_libdir}/cjdns directory is not used by the main package.  It is only
for the tools packages.  I suppose I should treat the tools package as a nodejs
package, and the python tools as a python package.  I may need to add proper
python packaging for them.  I can just delete the python tools in the meantime.

make *is* needed to build the embedded libuv library.

I am redoing the %install section to use /bin/install instead of cp so as to
force desired permissions.  Using %defattr was too cumbersome with the mix of
files that are executable or not.

Changed license string and changed all $RPM_BUILD_ROOT to %{buildroot}

I'll have to investigate the bencode.py license - or just delete the python
tools.

-- 
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
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]