[Bug 1369213] python-django-netjsongraph - Reusable django app for collecting and visualizing network topology

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

 



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



--- Comment #14 from gil cattaneo <puntogil@xxxxxxxxx> ---
(In reply to Germano Massullo from comment #10)
> (In reply to gil cattaneo from comment #9)
> > 
> > (In reply to Germano Massullo from comment #8)
> > > (In reply to gil cattaneo from comment #7)
> > > > Please, remove "%if 0%{?with_python2}" and
> > > > replace "%if 0%{?with_python3}" with "%if 0%{?el7}"
> > > 
> > > Why? The package either installs Python 2 version or Python3 version
> > > depending on which Python interpreter version is shipped with the system
> > 
> > Because of Comment#6 and is not safe use your approch in the spec file
> > i suggested of revrite your spec file using the template mentioned above.
> 
> Please define "not safe"

1. You put all "Requires: " under the %description tag, and they become part of
this tag:

python-django-netjsongraph.noarch: W: tag-in-description C Requires:
python-django-netjsongraph.noarch: W: tag-in-description C Requires:
python-django-netjsongraph.noarch: W: tag-in-description C Requires:
python-django-netjsongraph.src: E: description-line-too-long C Requires:      
python3-django-rest-framework >= 3.3, python3-django-rest-framework < 3.4

2. some of the "django packages" do not use both Python interpreters
   for me it means only that they are not compatible with either of these two.
   perhaps this package does not fall into this category ... ?

-- 
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://lists.fedoraproject.org/admin/lists/package-review@xxxxxxxxxxxxxxxxxxxxxxx




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