Red Hat Bugzilla – Bug 443467
Review Request: ale - Combines multiple inputs of the same scene
Last modified: 2008-06-20 15:14:56 EDT
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
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) :
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
> 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.
* Mon Apr 21 2008 Douglas E. Warner <email@example.com> 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
> > 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
mv string.h-t string.h
It makes use of a custom string.h - I wonder why ? (can you ask upstream?)
At least it build with all our platforms.
/usr/bin/ale seems to need exiftool and dcraw, so it would be fine to have:
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
Anyway, that would be optionnal for the review.
To sum up:
* Ask upstream about it's custom string.h
* Fri May 09 2008 Douglas E. Warner <firstname.lastname@example.org> 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
New Package CVS Request
Package Name: ale
Short Description: Combines multiple inputs of the same scene
Branches: F-8, F-9
Cvsextras Commits: yes
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.