[Bug 1452649] Review Request: python-exabgp - Package review request for ExaBGP spec

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

 



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



--- Comment #9 from Luke Hinds <lhinds@xxxxxxxxxx> ---
Hi Richard,

I have some updates:

- %license COPYRIGHT

done

- The exabgp package depends on  /usr/bin/perl

This is from two perl scripts which are examples of what can be used for
monitoring routes. Neither require modules outside of the standard set included
in perl. The main library will work fine without perl in place, as you rightly
noted around python being the main lang. So no Requires or BuildRequires
needed.

-  exabgp.noarch: E: non-executable-script
/etc/exabgp/examples/api-announcement.run 644 /usr/bin/env python

I spoke with a developer on IRC and they said that these do not need to be
executionable, as they are only there as examples for people to reference of
use if it suits their needs.

- exabgp.noarch: W: non-conffile-in-etc /etc/exabgp/examples/api-api.conf

Same again as the above, examples for users to reference..the recommendation
would be to copy them out and then amend, but most folks write their own.

- wrong-script-interpreter /etc/exabgp/examples/api-multisession.run
/usr/bin/env python

Upstream prefers not to change it to a fixed path, and instead use env for
python virtualenv use (I figure you already knew this from being a python man,
so that's why you omitted mentioning).

So the only others now are:

- spurious-executable-perm /usr/share/man/man1/exabgp.1.gz
- manual-page-warning /usr/share/man/man1/exabgp.1.gz 3: warning: macro `OS'
not defined
- exabgp.noarch: W: no-manual-page-for-binary exabgp-healthcheck

I guess the above can be waived as they are warnings and not errors?

Original links have been refreshed. Latest review.txt here:
https://paste.fedoraproject.org/paste/mV-hAsjKX8pNkdvL-lYNe15M1UNdIGYhyRLivL9gydE= 

p.s. thanks for time walking this through with me.

-- 
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
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux