[Bug 1506722] Review Request: python-octaviaclient - Client for OpenStack Octavia ( Load Balancer as a Service)

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

 



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

Robert-André Mauchin <zebob.m@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |zebob.m@xxxxxxxxx
            Summary|Review Request:             |Review Request:
                   |python-octaviaclient-       |python-octaviaclient -
                   |Client for OpenStack        |Client for OpenStack
                   |Octavia (Load Balancer as a |Octavia (Load Balancer as a
                   |Service)                    |Service)



--- Comment #1 from Robert-André Mauchin <zebob.m@xxxxxxxxx> ---
Hello,

 - Did you forget to put the Version and Release infos?

Version:        XXX
Release:        XXX

⇒

Version:        1.1.0
Release:        1%{?dist}

 - There is no %changelog entry either!

* Thu Oct 26 2017 Carlos Goncalves <cgoncalves@xxxxxxxxxx> - 1.1.0-1
- Initial RPM release

 - No need to complicate things, use version instead of upstream_version and
remove upstream_version definition

%autosetup -n %{name}-%{version}

 - You could use the %{py2_dist/%{py3_dist macros to handle your BR and R. See
https://fedoraproject.org/wiki/Packaging:Python#Requires_and_BuildRequires_with_standardized_names

 - BuildRequires:  openstack-macros

There is no package openstack-macros in Fedora repositories. You'll have to
package it beforehand.

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




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux