[Bug 815135] Review Request: atf - Automated Testing Framework

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Michel Alexandre Salim <michel+fdr@xxxxxxxxxxxx> 2012-05-01 00:33:27 EDT ---
I've tested the generated against lutok, and atf-run works fine if
libatf-c++-devel is installed when lutok is configured and built.

One change needs to be made to the packaging -- it's generally fine if
subpackages do not have documentation, if they depend on the main package --
but in your case, since the libatf-* subpackages are not dependent on atf, some
of them need to include at least the license text (COPYING). Basically any
installable subset of atf needs to carry the license text.

Looks like adding the LICENSE text to libatf-c is sufficient since all the
other subpackages depend on it eventually.

* TODO Review [90%]
  - [X] Names [2/2]
    - [X] Package name
    - [X] Spec name
  - [X] Package version [2/2]
 http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Package_Versioning
    - [X] Version number
   http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Version_Tag
    - [X] Release tag
   http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Release_Tag
  
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Pre-Release_packages
  - [X] Meets
[[http://fedoraproject.org/wiki/Packaging/Guidelines][guidelines]]
  - [X] Source files match upstream
  $ sha256sum atf-0.15.tar.gz ../SOURCES/atf-0.15.tar.gz 
  0c7242a107c7e308feed8fac45a194a6f6c8d90283add576cfc3dab0fcd61b2b 
atf-0.15.tar.gz
  0c7242a107c7e308feed8fac45a194a6f6c8d90283add576cfc3dab0fcd61b2b 
../SOURCES/atf-0.15.tar.gz

  - [X] [[http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries][No
bundled libraries]]
  - [X] License [4/4]
    - [X] License is Fedora-approved
    - [X] No licensing conflict
    - [X] License field accurate
    - [X] License included iff packaged by upstream
  - [-] rpmlint [1/2]
    - [X] on src.rpm
      ✗ rpmlint ./atf-0.15-1.fc17.src.rpm 
      1 packages and 0 specfiles checked; 0 errors, 0 warnings.
    - [ ] on x86_64.rpm
      ✗ rpmlint ~/Downloads/*atf*.rpm
      atf.x86_64: W: devel-file-in-non-devel-package /usr/bin/atf-config
      libatf-c.x86_64: W: no-documentation
      libatf-c++.x86_64: W: no-documentation
      8 packages and 0 specfiles checked; 0 errors, 3 warnings.

      Note: since libatf-c and libatf-c++ do not require the base package,
      you must at least include the COPYING file as documentation

  - [X] Language & locale [2/2]
    - [X] Spec in US English
    - [X] Spec legible
  - [X] Build [3/3]
    - [X] Koji results
      http://koji.fedoraproject.org/koji/taskinfo?taskID=4038205
    - [X] BRs complete
    - [X] Directory ownership
  - [X] Spec inspection [8/8]
    - [X] ldconfig for libraries
    - [X] No duplicate files
    - [X] File permissions
    - [X] Filenames must be UTF-8
    - [X] no BuildRoot
([[https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag][except if
targeting RHEL5]])
    - [X] Macro usage consistent
    - [X] Documentation [1/1]
      - [X] %doc files are non-essential
    - [X] Development [4/4]
      - [X] Headers in -devel
      - [X] If versioned .so's, unversioned in -devel
      - [X] -devel, -static requires main
      - [X] No .la

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
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]