https://bugzilla.redhat.com/show_bug.cgi?id=1470436
Mattia Verga <mattia.ve...@email.it> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mattia.ve...@email.it
--- Comment #4 from Mattia Verga <mattia.ve...@email.it> ---
Thanks for your effort in continuing the process of importing astrometry.net
into Fedora.
I have no free time at the moment to follow a full review, but here are my 2
cents:
- I think would be better asking upstream to rename the conflicting executable
and also some other executables that have too common name and can become a
problem in future. Renaming the executable only here in Fedora can cause
confusion and I think this should be the last chance.
- Patches should be reported upstream
https://fedoraproject.org/wiki/Packaging:Guidelines#Patch_Guidelines and the
fact we are not the only distro using them should make the developer more open
to accept them upstream.
--
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
