https://bugzilla.redhat.com/show_bug.cgi?id=1470436 Mattia Verga <mattia.verga@xxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mattia.verga@xxxxxxxx --- Comment #4 from Mattia Verga <mattia.verga@xxxxxxxx> --- 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@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
