Bug 713963
Summary: | Review Request: rpkg - Utility for interacting with rpm+git packaging systems | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Jesse Keating <jkeating> |
Component: | Package Review | Assignee: | leigh scott <leigh123linux> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | dcantrell, fedora-package-review, herrold, leigh123linux, notting, pahan |
Target Milestone: | --- | Flags: | leigh123linux:
fedora-review+
gwync: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2011-06-28 22:43:11 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Jesse Keating
2011-06-16 22:18:01 UTC
this needs changing from %files -n pyrpkg %doc COPYING README # For noarch packages: sitelib %{python_sitelib}/* to %files -n pyrpkg %defattr(-,root,root,-) %doc COPYING README %{python_sitelib}/pyrpkg %{python_sitelib}/rpkg-1.0-py?.?.egg-info as far as I know you should remove the comment from "rm -rf $RPM_BUILD_ROOT" in the install section. make these changes and I will approve it Another point BuildArch: noarch BuildRequires: python-devel, python-setuptools # We br these things for man page generation due to imports BuildRequires: GitPython, koji, python-pycurl %if 0%{?rhel} == 5 || 0%{?rhel} == 4 BuildRequires: python-hashlib BuildRequires: python-argparse %endif needs to be BuildArch: noarch BuildRequires: python-devel, python-setuptools # We br these things for man page generation due to imports BuildRequires: GitPython, koji, python-pycurl %if 0%{?rhel} == 6 || 0%{?rhel} == 4 BuildRequires: python-hashlib BuildRequires: python-argparse %endif to build on RHEL6 Good catches. That's what I get for uploading specs late at night. New stuff uploaded: Spec URL: http://jkeating.fedorapeople.org/review/rpkg.spec SRPM URL: http://jkeating.fedorapeople.org/review/rpkg-1.0-2.f14.src.rpm the defattr stuff is actually not necessary anymore. Rpm since 4.4 has a sane default. (In reply to comment #3) > the defattr stuff is actually not necessary anymore. Rpm since 4.4 has a sane > default. Thanks for the info. Sadly rpmlint on el6 still sees it as a error (when I get time I will file a bugreport against rpmlint). [leigh@main_pc SPECS]$ rpmlint rpkg-1.0-1.el6.src.rpm rpkg.src: I: enchant-dictionary-not-found en_US rpkg.src:74: E: files-attr-not-set rpkg.src:75: E: files-attr-not-set rpkg.src:76: E: files-attr-not-set P.S you forgot to update the spec file Do you have a proxy or something setup? I did put a new spec file up there, and just checked it in my browser, it has the right content. Make that: Spec URL: http://jkeating.fedorapeople.org/review/rpkg.spec SRPM URL: http://jkeating.fedorapeople.org/review/rpkg-1.1-1.f14.src.rpm Ahem. Spec URL: http://jkeating.fedorapeople.org/review/rpkg.spec SRPM URL: http://jkeating.fedorapeople.org/review/rpkg-1.1-2.f14.src.rpm source sha1sum matches upstream spec file conforms to packaging guide lines licence is acceptable and correct builds ok @ koji for F16 http://koji.fedoraproject.org/koji/taskinfo?taskID=3139353 Package approved New Package SCM Request ======================= Package Name: rpkg Short Description: Utility for interacting with rpm+git packaging systems Owners: jkeating Branches: f14 f15 el6 el5 InitialCC: Git done (by process-git-requests). Built for rawhide. |