[Bug 812059] Review Request: python-tw2-jqplugins-flot - jQuery flot (plotting) for ToscaWidgets2

[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=812059

Michel Alexandre Salim <michel+fdr@xxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|nobody@xxxxxxxxxxxxxxxxx    |michel+fdr@xxxxxxxxxxxx

--- Comment #4 from Michel Alexandre Salim <michel+fdr@xxxxxxxxxxxx> 2012-05-05 02:41:00 EDT ---
Same as -gritter -- directory ownership needs fixing


* 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 ./tw2.jqplugins.flot-2.0.1.tar.gz
~/rpmbuild/SOURCES/tw2.jqplugins.flot-2.0.1.tar.gz 
  c155703bfd1471b67558e181d41d2065900c56682d81f0cf2ed43b165e65c95e 
./tw2.jqplugins.flot-2.0.1.tar.gz
  c155703bfd1471b67558e181d41d2065900c56682d81f0cf2ed43b165e65c95e 
/home/michel/rpmbuild/SOURCES/tw2.jqplugins.flot-2.0.1.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
  - [X] rpmlint [2/2]
    Harmless spellcheck "warnings" on both RPMs
    - [X] on src.rpm
      $ rpmlint
/var/lib/mock/fedora-17-x86_64/result/python-tw2-jqplugins-flot-2.0.1-2.fc17.src.rpm 
      python-tw2-jqplugins-flot.src: W: spelling-error Summary(en_US) jQuery ->
j Query, query, equerry
      python-tw2-jqplugins-flot.src: W: summary-not-capitalized C jQuery flot
(plotting) for ToscaWidgets2
      python-tw2-jqplugins-flot.src: W: spelling-error %description -l en_US
performant -> perform ant, perform-ant, performance
      python-tw2-jqplugins-flot.src: W: spelling-error %description -l en_US
jQuery -> j Query, query, equerry
      1 packages and 0 specfiles checked; 0 errors, 4 warnings.
    - [X] on x86_64.rpm
      $ rpmlint
/var/lib/mock/fedora-17-x86_64/result/python-tw2-jqplugins-flot-2.0.1-2.fc17.noarch.rpm
      python-tw2-jqplugins-flot.noarch: W: spelling-error Summary(en_US) jQuery
-> j Query, query, equerry
      python-tw2-jqplugins-flot.noarch: W: summary-not-capitalized C jQuery
flot (plotting) for ToscaWidgets2
      python-tw2-jqplugins-flot.noarch: W: spelling-error %description -l en_US
performant -> perform ant, perform-ant, performance
      python-tw2-jqplugins-flot.noarch: W: spelling-error %description -l en_US
jQuery -> j Query, query, equerry
      1 packages and 0 specfiles checked; 0 errors, 4 warnings.
  - [X] Language & locale [2/2]
    - [X] Spec in US English
    - [X] Spec legible
  - [-] Build [2/3]
    - [X] Koji results
      local mock in lieue of Koji
    - [X] BRs complete
    - [ ] Directory ownership
      The wildcard makes this package also owns the following, which
      should be owned by packages it depend on:
      %{python_sitelib}/tw2
      %{python_sitelib}/tw2/jqplugins

      Perhaps change this to:
      %{python_sitelib}/tw2.jqplugins.flot*
      %{python_sitelib}/tw2/jqplugins/flot
  - [X] Spec inspection [5/5]
    - [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
      note that you can just use 0%{?rhel} rather than
      %{?rhel}%{!?rhel:0}

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