[Bug 1115299] Review Request: ceph-deploy - Admin and deploy tool for Ceph

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

 



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



--- Comment #5 from Christopher Meng <i@xxxxxxxx> ---
Not finished yet:

1. Source0:        %{name}-%{version}.tar.xz

Please use the full URL.

2. BuildRequires:  python-devel

->

BuildRequires:  python2-devel

3. CEPH_DEPLOY_NO_VENDOR=1 %{__python} setup.py build

->

CEPH_DEPLOY_NO_VENDOR=1 %{__python2} setup.py build

4. %{python_sitelib}/*

->

%{python2_sitelib}/*

--------------------------

Although RHEL6 is an important market of ceph indeed, you still need to care
about macros:

%if 0%{?rhel} && 0%{?rhel} <= 6
%{!?__python2: %global __python2 /usr/bin/python2}
%{!?python2_sitelib: %global python2_sitelib %(%{__python2} -c "from
distutils.sysconfig import get_python_lib; print(get_python_lib())")}
%endif

5. Requires:       python-remoto >= 0.0.17

This is a new package in Fedora + EPEL(?/RHEL?), I don't think you need to
specify the version.

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review





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