https://bugzilla.redhat.com/show_bug.cgi?id=854679 Jan Synacek <jsynacek@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(jskarvad@redhat.c | |om) --- Comment #2 from Jan Synacek <jsynacek@xxxxxxxxxx> --- Specfile: - use %global instead of %define - do not use BuildRoot: - do not use rm -rf %{buildroot} in install section (unless you intend to use the package for EPEL) - do not use %defattr (unless you intend to use the package for EPEL) - also, I believe there is a typo on line 36: + rm -f {buildroot}... should probably be rm -f %{buildroot}... Also, rpmlint complains a lot about incorrect fsf addresses: python-sudsds.noarch: E: incorrect-fsf-address /usr/lib/python2.7/site-packages/sudsds/transport/https.py python-sudsds.noarch: E: incorrect-fsf-address /usr/lib/python2.7/site-packages/sudsds/bindings/__init__.py python-sudsds.noarch: E: incorrect-fsf-address /usr/lib/python2.7/site-packages/sudsds/mx/typer.py That should probably be reported and corrected by upstream. Other than that, I don't see any more things that need correction. The package also does not conflict with 'python-suds' that is already present in Fedora. -- You are receiving this mail because: You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review