Bug 720411 (OpenImageIO)

Rpmlint output: (comments inline)

$ rpmlint OpenImageIO-*.rpm
OpenImageIO.src: W: spelling-error %description -l en_US plugins -> plug ins, plug-ins, plugging
OpenImageIO.src: W: invalid-url Source0: https://download.github.com/OpenImageIO-oiio-Release-0.10.0-12-g8055b0f.tar.gz HTTP Error 404: Not Found

I have not been able to find a permanent download link for github as it appears to generate the archives on the fly and then the link expires sometime later (<1 hr).


OpenImageIO.x86_64: W: spelling-error %description -l en_US plugins -> plug ins, plug-ins, plugging
OpenImageIO.x86_64: W: private-shared-object-provides /usr/lib64/python2.7/site-packages/OpenImageIO.so OpenImageIO.so()(64bit)

I looked at filtering this but it didn't seem to fit the guideline requirements.


4 packages and 0 specfiles checked; 0 errors, 4 warnings.

OpenImageIO bundles pugixml which has been removed from the SRPM and review request 720040 submitted to correct this.

I'm taking this one too :)

+ OK
- NA
? Issue

+ Package meets naming and packaging guidelines
+ Spec file matches base package name.
+ Spec has consistant macro usage.
+ Meets Packaging Guidelines.
+ License
? License field in spec matches

^^^
[ankur@ankur OpenImageIO-oiio-8055b0f]$ find . -type f -execdir licensecheck '{}' \; | sed '/UNKNOWN/d' | sed -r 's/.\/.*://' | sort -d | uniq -c
     26  BSD (2 clause)
    188  BSD (3 clause)
     20  GENERATED FILE
     23  GPL (v2)
      1  LGPL (v2 or later) (with incorrect FSF address)
     24  MIT/X11 (BSD like)
    212  *No copyright* GENERATED FILE

So, there are quite a few Licenses in this package ;)
To see what files use what, please remove the last three commands in the command chain above.

? License file included in package

^^^
Only the BSD license is included

+ Spec in American English
+ Spec is legible.
+ Sources match upstream md5sum:
The SOURCE0 URL gives a 404. Probably because it generates a tar on the fly when you click download. 

[ankur@ankur SPECS]$ spectool  -g OpenImageIO.spec
Getting https://download.github.com/OpenImageIO-oiio-Release-0.10.0-12-g8055b0f.tar.gz to ./OpenImageIO-oiio-Release-0.10.0-12-g8055b0f.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:--  0:00:10 --:--:--     0
curl: (22) The requested URL returned error: 404

The md5sums are okay though.
[ankur@ankur SPECS]$ md5sum ~/dump/OpenImageIO-oiio-Release-0.10.0-12-g8055b0f.tar.gz ../SOURCES/OpenImageIO-oiio-Release-0.10.0-12-g8055b0f.tar.gz 
29097c71383ba39a4d50bd3de1b6322d  /home/ankur/dump/OpenImageIO-oiio-Release-0.10.0-12-g8055b0f.tar.gz
29097c71383ba39a4d50bd3de1b6322d  ../SOURCES/OpenImageIO-oiio-Release-0.10.0-12-g8055b0f.tar.gz

- Package needs ExcludeArch
+ BuildRequires correct
- Spec handles locales/find_lang
- Package is relocatable and has a reason to be.

^^^
%defattr, buildroot, clean section not required anymore

+ Package is code or permissible content.
- Doc subpackage needed/used.
+ Packages %doc files don't affect runtime.

+ Headers/static libs in -devel subpackage.
+ Spec has needed ldconfig in post and postun
- .pc files in -devel subpackage/requires pkgconfig
+ .so files in -devel subpackage.
+ -devel package Requires: %{name} = %{version}-%{release}
- .la files are removed.

- Package is a GUI app and has a .desktop file

+ Package compiles and builds on at least one arch.
+ Package has no duplicate files in %files.
+ Package doesn't own any directories other packages own.
+ Package owns all the directories it creates.
? No rpmlint output.
[ankur@ankur SRPMS]$ rpmlint /var/lib/mock/fedora-rawhide-i386/result/*.rpm ../SPECS/OpenImageIO.spec OpenImageIO-0.10.0-1.fc14.src.rpm 
OpenImageIO.i686: W: private-shared-object-provides /usr/lib/python2.7/site-packages/OpenImageIO.so OpenImageIO.so
OpenImageIO.src: W: invalid-url Source0: https://download.github.com/OpenImageIO-oiio-Release-0.10.0-12-g8055b0f.tar.gz HTTP Error 404: Not Found
../SPECS/OpenImageIO.spec: W: invalid-url Source0: https://download.github.com/OpenImageIO-oiio-Release-0.10.0-12-g8055b0f.tar.gz HTTP Error 404: Not Found
OpenImageIO.src: W: invalid-url Source0: https://download.github.com/OpenImageIO-oiio-Release-0.10.0-12-g8055b0f.tar.gz HTTP Error 404: Not Found
5 packages and 1 specfiles checked; 0 errors, 4 warnings.
[ankur@ankur SRPMS]$ 

Please do look at the warning.

+ final provides and requires are sane:
== OpenImageIO-0.10.0-1.fc16.i686.rpm ==
Provides:
OpenImageIO.so  
libOpenImageIO.so.0.10  
OpenImageIO = 0.10.0-1.fc16
OpenImageIO(x86-32) = 0.10.0-1.fc16

Requires:
/sbin/ldconfig  
/sbin/ldconfig  
libGL.so.1  
libGLEW.so.1.6  
libGLU.so.1  
libHalf.so.6  
libICE.so.6  
libIex.so.6  
libIlmImf.so.6  
libIlmThread.so.6  
libImath.so.6  
libOpenImageIO.so.0.10  
libQtCore.so.4  
libQtGui.so.4  
libQtOpenGL.so.4  
libSM.so.6  
libX11.so.6  
libXext.so.6  
libboost_filesystem-mt.so.1.46.1  
libboost_program_options-mt.so.1.46.1  
libboost_python-mt.so.1.46.1  
libboost_regex-mt.so.1.46.1  
libboost_system-mt.so.1.46.1  
libboost_thread-mt.so.1.46.1  
libc.so.6  
libc.so.6(GLIBC_2.0)  
libc.so.6(GLIBC_2.1)  
libc.so.6(GLIBC_2.1.3)  
libc.so.6(GLIBC_2.11)  
libc.so.6(GLIBC_2.2)  
libc.so.6(GLIBC_2.3.2)  
libc.so.6(GLIBC_2.3.4)  
libc.so.6(GLIBC_2.4)  
libc.so.6(GLIBC_2.7)  
libc.so.6(GLIBC_2.8)  
libdl.so.2  
libgcc_s.so.1  
libgcc_s.so.1(GCC_3.0)  
libgcc_s.so.1(GLIBC_2.0)  
libjasper.so.1  
libjpeg.so.62  
libjpeg.so.62(LIBJPEG_6.2)  
libm.so.6  
libm.so.6(GLIBC_2.0)  
libm.so.6(GLIBC_2.1)  
libpng12.so.0  
libpng12.so.0(PNG12_0)  
libpthread.so.0  
libpthread.so.0(GLIBC_2.0)  
libpthread.so.0(GLIBC_2.2)  
libpthread.so.0(GLIBC_2.3.2)  
libpugixml.so.1.0  
libpython2.7.so.1.0  
libstdc++.so.6  
libstdc++.so.6(CXXABI_1.3)  
libstdc++.so.6(GLIBCXX_3.4)  
libstdc++.so.6(GLIBCXX_3.4.11)  
libstdc++.so.6(GLIBCXX_3.4.15)  
libstdc++.so.6(GLIBCXX_3.4.5)  
libstdc++.so.6(GLIBCXX_3.4.9)  
libtiff.so.3  
libz.so.1  
python(abi) = 2.7
rtld(GNU_HASH)  

== OpenImageIO-0.10.0-1.fc16.src.rpm ==
Provides:

Requires:
boost-devel  
glew-devel  
qt-devel  
OpenEXR-devel  
ilmbase-devel  
python2-devel  
txt2man  
libpng  
libtiff-devel  
zlib-devel  
jasper-devel  
pugixml-devel  

== OpenImageIO-debuginfo-0.10.0-1.fc16.i686.rpm ==
Provides:
OpenImageIO-debuginfo = 0.10.0-1.fc16
OpenImageIO-debuginfo(x86-32) = 0.10.0-1.fc16

Requires:

== OpenImageIO-devel-0.10.0-1.fc16.i686.rpm ==
Provides:
OpenImageIO-devel = 0.10.0-1.fc16
OpenImageIO-devel(x86-32) = 0.10.0-1.fc16

Requires:
OpenImageIO(x86-32) = 0.10.0-1.fc16
libOpenImageIO.so.0.10  



SHOULD Items:

+ Should build in mock.
+ Should build on all supported archs
- Should function as described.
+ Should have sane scriptlets.
+ Should have subpackages require base package with fully versioned depend.
+ Should have dist tag
+ Should package latest version
- check for outstanding bugs on package. (For core merge reviews)

Issues:

1. Please check the licensing
2. Please check the rpmlint warning. 

The rest looks okay :)

(In reply to comment #4)
> [ankur@ankur OpenImageIO-oiio-8055b0f]$ find . -type f -execdir licensecheck
> '{}' \; | sed '/UNKNOWN/d' | sed -r 's/.\/.*://' | sort -d | uniq -c
>      26  BSD (2 clause)
>     188  BSD (3 clause)
>      20  GENERATED FILE
>      23  GPL (v2)
>       1  LGPL (v2 or later) (with incorrect FSF address)
>      24  MIT/X11 (BSD like)
>     212  *No copyright* GENERATED FILE

Neat trick! I didn't know about this one. The good news, the LGPL is part of the documentation that I'm not including in the package (it's out of date). The bad news is I didn't realize that TBB was a bundled library (all the other GPL hits). I've downloaded the source... we'll see how easy it will be to package.


> The SOURCE0 URL gives a 404. Probably because it generates a tar on the fly
> when you click download. 

Yup.


> %defattr, buildroot, clean section not required anymore

Thought I had already got rid of most of that... 


> ? No rpmlint output.
> [ankur@ankur SRPMS]$ rpmlint /var/lib/mock/fedora-rawhide-i386/result/*.rpm
> ../SPECS/OpenImageIO.spec OpenImageIO-0.10.0-1.fc14.src.rpm 
> OpenImageIO.i686: W: private-shared-object-provides
> /usr/lib/python2.7/site-packages/OpenImageIO.so OpenImageIO.so
> OpenImageIO.src: W: invalid-url Source0:
> https://download.github.com/OpenImageIO-oiio-Release-0.10.0-12-g8055b0f.tar.gz
> HTTP Error 404: Not Found
> ../SPECS/OpenImageIO.spec: W: invalid-url Source0:
> https://download.github.com/OpenImageIO-oiio-Release-0.10.0-12-g8055b0f.tar.gz
> HTTP Error 404: Not Found
> OpenImageIO.src: W: invalid-url Source0:
> https://download.github.com/OpenImageIO-oiio-Release-0.10.0-12-g8055b0f.tar.gz
> HTTP Error 404: Not Found
> 5 packages and 1 specfiles checked; 0 errors, 4 warnings.
> [ankur@ankur SRPMS]$ 
> 
> Please do look at the warning.

Yup, but not much I can do about it. Basically it's a C++ python library but it doesn't meet the requirements for manual filtering. I don't think it will hurt anything.

I'm taking a look at TBB, I'll post a new package when I have something.

Thanks!

Richard

Ok! I talked with upstream and they said just don't use TBB since it's not needed for *nix.

This takes care of the GPL issue.

I also moved the headers in the -devel package to /usr/include/OpenImageIO because some of the names were too generic to leave in /usr/include.

SPEC:
http://hobbes1069.fedorapeople.org/OpenImageIO/OpenImageIO.spec

SRPM:
http://hobbes1069.fedorapeople.org/OpenImageIO/OpenImageIO-0.10.0-2.fc14.src.rpm

Hello!

Issues have been fixed.

Confirmed that TBB is not being used:
from build.log->  165 -- TBB will not be used

rpmlint warning is ignorable. 

XXX APPROVED XXX

New Package SCM Request
=======================
Package Name: OpenImageIO
Short Description: Library for reading and writing images
Owners: hobbes1069
Branches: f14 f15
InitialCC:

Git done (by process-git-requests).

OpenImageIO-0.10.0-2.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/OpenImageIO-0.10.0-2.fc14

OpenImageIO-0.10.0-2.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/OpenImageIO-0.10.0-2.fc15

OpenImageIO-0.10.0-2.fc15 has been pushed to the Fedora 15 testing repository.

OpenImageIO-0.10.0-2.fc15 has been pushed to the Fedora 15 stable repository.

OpenImageIO-0.10.0-2.fc14 has been pushed to the Fedora 14 stable repository.

Package Change Request
======================
Package Name: OpenImageIO
New Branches: f16
Owners: hobbes1069
InitialCC: 

Looks like I missed the branching for F16 somehow.

Git done (by process-git-requests).

Package Change Request
======================
Package Name: OpenImageIO
New Branches: el6
Owners: hobbes1069
InitialCC:

Git done (by process-git-requests).

Package Change Request
======================
Package Name: OpenImageIO
New Branches: epel7
Owners: hobbes1069
InitialCC:

Git done (by process-git-requests).