Push VIGRA changes to the right place

[Yuv]

Enblend embeds a modified version of the VIGRA 1.4 library. Can we push the necessary changes to VIGRA's upstream and use the system version rather than an embedded copy?

Details: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=542258

[Bruno Postle (brunopostle)]

This also affects Hugin, which contains a modified copy of vigra-1.5

Note that the fedora policy is getting stricter on included libraries, this bug could result in both Hugin and Enblend being dropped at some point: http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries

[Andreas Metzler (k-launchpad-downhill-at-eu-org)]

VIGRA 1.8.0 was released today. It contains the patches from hugin which were forwarded upstream <http://comments.gmane.org/gmane.comp.video.vigra/42>.

[Christoph Spiel (cspiel)]

Fixed in rev57a56d454739, though fix curtails functionality:
no alpha masks in output images.

[Bruno Postle (brunopostle)]

Confirmed, I can now build an enblend snapshot against the standard fedora vigra-1.8.0 package without problems. Thanks.

Regarding the lack of alpha masks in output, is this something that needs a patch pushing to vigra upstream? Being able to feed the output from enblend (or enfuse) as input for more blending is a valid use case.

[Christoph Spiel (cspiel)]

Export-with-alpha just does not _compile_ anymore with the new Vigra-library.
No trivial fix in sight. Patch highly welcome.

[Christoph Spiel (cspiel)]

We now have a patch that re-installs all prior functionality
with respect to loading and saving images with alpha-channels.
However, the copyright of the new (it is in fact a complete
rewrite) file "impexalpha.hxx" must be sorted out before we can
add it to the repository.

[Yuv (yuv)]

On 12-09-03 07:51 AM, Christoph Spiel wrote:
> We now have a patch that re-installs all prior functionality
> with respect to loading and saving images with alpha-channels.
> However, the copyright of the new (it is in fact a complete
> rewrite) file "impexalpha.hxx" must be sorted out before we can
> add it to the repository.

Thank you. Do I assume correctly that you rewrote the code?

What copyright issues need sorting out other than adding your name to
the credits where credit is due, e.g.
<http://hci.iwr.uni-heidelberg.de/vigra/doc/vigra/CreditsChangelog.html>
or line 3 of impexlpha.hxx itself?

Is the author of the rewrite OK with publishing the file under the
current Vigra license (MIT X11 license)?

Yuv

[Pablo d'Angelo (pablo.dangelo)]

Hi all,

if the impexalpha code still contains any of my old one, I hereby relicense it under MIT X11 license.

ciao
Pablo

[Christoph Spiel (cspiel)]

Yuval -

    The copyright issue boils down to the
question whether the new file "impexalpha.hxx" is a
"substantial portion" of the old Vigra code
"impex.hxx" (The quote comes from Vigra's MIT
license.) We are sorting out this one.

On another side we still have problems with the
Graph Cut algorithm when linked against new
Vigra. Maybe, Mikołaj can report more.

[Yuv (yuv)]

Christoph -
On 12-09-09 03:48 AM, Christoph Spiel wrote:
> The copyright issue boils down to the
> question whether the new file "impexalpha.hxx" is a
> "substantial portion" of the old Vigra code
> "impex.hxx" (The quote comes from Vigra's MIT
> license.) We are sorting out this one.

Why is "substantial portion" an issue? Do you intend to publish the
file under a different license, and if so which one?

How are you going about sorting this out?

> On another side we still have problems with the
> Graph Cut algorithm when linked against new
> Vigra. Maybe, Mikołaj can report more.

Are these technical problems or licensing problems?

Yuv

[Christoph Spiel (cspiel)]

The copyright problems were ironed out.

The new file "impexalpha.hxx" will be part of the Enblend
project under the same license as all other sources
namely GPL Version 2.

[Christoph Spiel (cspiel)]

Problems should be fixed in rev 29ec0c175c5a -- fingers crossed.

[tmodes (tmodes)]

Implemented in changeset cce900e8cc32c54f332a245d78140d5e3831e27a
Hugin requires now vigra 1.9 or later.
We still need out own vigra_ext/impexalpha.hxx because of a bug in vigras impexalpha.hxx which prevents saving of exr files with alpha channel.