Bug 357461

For 1.4.1-1:

* URL
  - (For sourceforge hosted projects), I usually think it is
    is better to use http://ampasctl.sourceforge.net/

* .pc file
  - For %_libdir/pkgconfig/CTL.pc:
-----------------------------------------------------
Libs: -L${libdir}   -lIlmCtlSimd -lIlmCtlMath -lIlmCtl -lImath -lHalf -lIex
-lIlmThread  
Cflags: -I/usr/include/OpenEXR   -I${CTL_includedir}
-----------------------------------------------------
    OpenEXR-devel ships OpenEXR.pc so please use this.

* rpmlint
-----------------------------------------------------
ctl-debuginfo.i386: W: spurious-executable-perm
/usr/src/debug/ctl-1.4.1/IlmCtl/CtlParser.cpp
ctl-debuginfo.i386: W: spurious-executable-perm
/usr/src/debug/ctl-1.4.1/IlmCtl/CtlInterpreter.cpp
-----------------------------------------------------
  - Please fix the permissions.

* License
  - Perhaps I can think this is BSD, however this is a bit
    different from BSD and once I want to ask how we can regard
    the term of "being governed by the laws of foo" like this
    package.

I'm pretty sure this BSD variant is Free, but GPL incompatible, due to the "This
license shall be governed by the laws of the State of California, and subject to
the jurisdiction of the courts therein."

I'm asking the FSF for feedback, please hold until I get it.

License is ok, Free but GPL Incompatible.

Use the short name identifier:

License: AMPAS BSD

Lifting FE-Legal.

Spec URL:
http://kwizart.fedorapeople.org/SRPMS/ctl.spec
SRPM URL:
http://kwizart.fedorapeople.org/SRPMS/ctl-1.4.1-2.fc7.kwizart.src.rpm
Description: The Color Transformation Language

%changelog
- Improve license URL
- Use IlmBase.pc for pkg-config
- Fix perms in debuginfo

I've changed BuidRequires since configure checks for IlmBase not OpenEXR
actually (spitted since OpenEXR 1.6.0)

I wonder if I would change the name from ctl to CTL since There is case problem
with theses package (same plans for openexr_ctl to OpenEXR_CTL since we already
have OpenEXR which seems more clean...)

Well, almost okay, however would you recheck if such a large patch
as ctl-1.4.1-disable_rpath.patch is really needed to remove rpath?

I guess fixing (removing) hardcode_libdir_flag_spec from configure
(maybe from libtool is okay) will remove rpath, as written in
"Removing Rpath" section of 
http://fedoraproject.org/wiki/Packaging/Guidelines

(In reply to comment #4)

> I wonder if I would change the name from ctl to CTL since There is case problem
> with theses package (same plans for openexr_ctl to OpenEXR_CTL since we already
> have OpenEXR which seems more clean...)

It seems to make sense, so please change the name to CTL if you want
to.

Spec URL:
http://kwizart.fedorapeople.org/SPECS/CTL.spec
SRPM URL:
http://kwizart.fedorapeople.org/SRPMS/CTL-1.4.1-3.fc7.kwizart.src.rpm
Description: The Color Transformation Language

%changelog
- Change package name from ctl to CTL
- Use patch libtool method to remvove rpath

Well, I will recheck your newest srpm soon.
By the way, I would appreciate it if you would review my review request
bug 365131 .

Okay, good.

---------------------------------------------------------------
    This package (CTL) is APPROVED by me
---------------------------------------------------------------

ping?

New Package CVS Request
=======================
Package Name: CTL
Short Description: The Color Transformation Language
Owners: kwizart
Branches: F-7 F-8

cvs done.