[Bug 1611596] New: Review Request: tendrl-commons - Common api for tendrl gluster web admin

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

Bug ID: 1611596
           Summary: Review Request: tendrl-commons - Common api for tendrl
                    gluster web admin
           Product: Fedora
           Version: rawhide
         Component: Package Review
          Severity: medium
          Priority: medium
          Assignee: [e-mail]
          Reporter: [e-mail]
        QA Contact: [e-mail]
                CC: [e-mail]

Spec URL:
https://copr-be.cloud.fedoraproject.org/results/tendrl/release/epel-7-x86_64...

SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/tendrl/release/epel-7-x86_64...

Description:

Hi I have finished packaging up tendrl-commons which is Common code usable by
all Tendrl components http://www.tendrl.org. Currently I am using fedora copr
for building the package. I would appreciate a review so that I can get it into
Fedora Extras!.

Fedora Account System Username: timothyasir

[Bug 1611596] Review Request: tendrl-commons - Common api for tendrl gluster web admin

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

Robert-André Mauchin  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
             Blocks|                            |177841 (FE-NEEDSPONSOR)

--- Comment #1 from Robert-André Mauchin  ---
Source0: %{name}-%{version}.tar.gz

- You need to point to a URL for the Source0

- %setup → %autosetup

- %{__python} setup.py build → %py2_build

- %{__python} setup.py install --single-version-externally-managed -O1
--root=$RPM_BUILD_ROOT --record=INSTALLED_FILES → %py2_install

- are you sure it's not compatible with Python 3? Python 2 is being phased
out.

- You need a sponsor, see
https://fedoraproject.org/wiki/Join_the_package_collection_maintainers

Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor