[Bug 1346038] Review Request: python-blivet1 - python2/ blivet-1.x compatibility package

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

 



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

Douglas Schilling Landgraf <dougsland@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?(dougsland@redhat. |
                   |com)                        |



--- Comment #17 from Douglas Schilling Landgraf <dougsland@xxxxxxxxxx> ---
Hi, 

A few additional comments:

$ fedora-review --rpm-spec -n python-blivet1-1.20.3-3.fc25.src.rpm 
INFO: Processing local files: python-blivet1-1.20.3-3.fc25.src.rpm
INFO: Getting .spec and .srpm Urls from : Local files in /home/fedora
INFO:   --> SRPM url: file:///home/fedora/python-blivet1-1.20.3-3.fc25.src.rpm
INFO: Using review directory: /home/fedora/python-blivet1
ERROR: 'Cannot find spec file in srpm' (logs in
/home/fedora/.cache/fedora-review.log)

This is caused because your spec file is called python-blivet.spec not
python-blivet1.spec. 

I have other question now, I see you are package maint. of python-blivet
package, why not use the current spec to create a subpackage of python2 and
python3 ? No need to create a new package/review "python-blivet1", just add the
new subpackages and rebuild it, like:

python2-blivet
python3-blivet 

"Common spec file example"
https://fedoraproject.org/wiki/Packaging:Python#Example_common_spec_file

You can always examine the others reviews and specs:
https://lists.fedoraproject.org/archives/search?mlist=package-review%40lists.fedoraproject.org&q=python

Finally, recommend you to use global instead of define in your python-blivet:

https://fedoraproject.org/wiki/Packaging:Guidelines#.25global_preferred_over_.25define

>From my point of view, we should close this bug and please proceed in the
python-blivet spec. Please let me know your thoughts.

Thanks!

-- 
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]