Spec URL: https://rpm.silfreed.net:8002/index.cgi/raw-file/cdccc785d310/ale/ale.spec SRPM URL: http://www.silfreed.net/download/repo/packages/ale/ale-0.9.0-1.src.rpm Description: ALE is an image-processing program used for tasks such as image mosaicking, super-resolution, deblurring, noise reduction, anti-aliasing, and scene reconstruction. Its principle of operation is synthetic capture, combining multiple inputs representing the same scene.
License: it is GPLv3+ (ui/input.cc fro example) fftw is GPLv2+ or later -> compatible ImageMagick-devel tagged as freeware ?!? -> don't knwo if license is compatible... I may raise FE-Legal Missing BR (optionnals but seems useful) : ImageMagick-devel fftw-devel >= 3.0.0 There is also some configure options that would need some testing.(or advices from upstream). I would add this: --enable-wmx --with-imagemagick --with-fftw3 --with-hashmap=gnu scripts are present in a subdirectory. I expect they could be installed in /usr/bin but transform seems too generic name, but be valuable to rename as ale-transform and to warn upstream (or warn first). I only have tested F-8 x86_64 in mock - F-9 wasn't in good shape, so i will test later...(don't know if new option introduced others gcc43 bug).
Ok found: http://fedoraproject.org/wiki/Licensing ImageMagick is compatible with GPLv3+
I forgot to say that the string.h tweak that is done at the begining seems scary to me... I wonder why this is needed. (I will check futher...)
(In reply to comment #3) > I forgot to say that the string.h tweak that is done at the begining seems scary > to me... I wonder why this is needed. (I will check futher...) Which string.h tweak? The cstring patch for GCC 4.3? BTW, I'm also working on an update that addresses your BRs in the comment #1; building in mock right now.
Spec URL: https://rpm.silfreed.net:8002/index.cgi/file/3991dca98947/ale/ale.spec SRPM URL: http://www.silfreed.net/download/repo/packages/ale/ale-0.9.0-2.src.rpm %changelog * Mon Apr 21 2008 Douglas E. Warner <silfreed> 0.9.0-2 - adding additional configuration options (imagemagick, fftw3, wmx, gnu hashmap) and build requires to satisfy them - installing scripts
(In reply to comment #4) > (In reply to comment #3) > > I forgot to say that the string.h tweak that is done at the begining seems > scary > > to me... I wonder why this is needed. (I will check futher...) > > Which string.h tweak? The cstring patch for GCC 4.3? see the build.log between rm -f string.h-t string.h and mv string.h-t string.h It makes use of a custom string.h - I wonder why ? (can you ask upstream?) see http://koji.fedoraproject.org/koji/taskinfo?taskID=595945 At least it build with all our platforms. /usr/bin/ale seems to need exiftool and dcraw, so it would be fine to have: Requires: perl-Image-ExifTool Requires: dcraw I also forgot to say that ale 0.9.0 is supposed to be the unstable branch, whereas 8.11 is the known stable. I'm still in runtime testing... maybe we can have 0.9.0 anyway. I don't know how work the --device (undocumented) option, maybe that what we can expect as improvement with 9.0 version (don't have such device also). From the sources code, the device folder doesn't seems to have the headers compiled... Anyway, that would be optionnal for the review. To sum up: * Ask upstream about it's custom string.h * Add: Requires: perl-Image-ExifTool Requires: dcraw
Spec URL:https://rpm.silfreed.net:8002/index.cgi/file/a4ab611d128d/ale/ale.spec SRPM URL: http://www.silfreed.net/download/repo/packages/ale/ale-0.9.0-3.src.rpm %changelog * Fri May 09 2008 Douglas E. Warner <silfreed> 0.9.0-3 - adding Requires perl-Image-ExifTool, dcraw I've emailed the list asking about the string.h thing, that does seem a bit odd. It's not showing up on the archives yet so I must be moderated.
any news from the string.h thing ?
No news; it doesn't seem that my messages was posted to the list. I'm going to try re-subscribing to the list and resending.
Got the list thing worked out; ale uses gnulib to help compile on other platforms. Upstream said an option could be added to ignore the lib directory, but currently one doesn't exist.
OK - don't know much about gnulib for now - but I saw the same in various already approved packages (libvirt, etc). So OK ------------------ This package (ale) is APPROVED by me ------------------
ping ?
New Package CVS Request ======================= Package Name: ale Short Description: Combines multiple inputs of the same scene Owners: silfreed Branches: F-8, F-9 Cvsextras Commits: yes
cvs done.
This package (ale) with hasn't been imported into the cvs yet...
slow packager; just got back from Vacation, will try to get to it this week.
ale-0.9.0.1-1.fc8 has been submitted as an update for Fedora 8
ale-0.9.0.1-1.fc9 has been submitted as an update for Fedora 9
ale-0.9.0.1-1.fc8 has been pushed to the Fedora 8 stable repository. If problems still persist, please make note of it in this bug report.
ale-0.9.0.1-1.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report.