Opened 7 years ago

Closed 4 years ago

#8896 closed enhancement (fixed)

Please include OpenEXR headers

Reported by: stefank Owned by: nobody
Priority: normal Milestone: R1
Component: - General Version: R1/alpha3
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

The Haiku source code already builds the OpenEXR libraries and includes the resulting .so libraries in the distribution. It would be useful if the matching header files would also be included in the Haiku distribution so that applications can use OpenEXR.

Change History (8)

comment:1 Changed 7 years ago by korli

OpenEXR should anyway be outsourced to HaikuPorts and used by the EXRTranslator through an optional package.

comment:2 Changed 7 years ago by axeld

There should then be a dev package containing the headers. I think this already works this way for ICU. An alternative to adding the library to the system would be to statically link it within the translator.

comment:3 in reply to:  2 Changed 7 years ago by leavengood

Replying to axeld:

There should then be a dev package containing the headers. I think this already works this way for ICU.

One thing I've always hated about Debian-based Linux systems is the separation of libraries and their headers into separate packages. It sounds good in theory but in practice is annoying on Linux where almost every user is compiling something at some point.

Of course Haiku is different and should mostly have binary packages, so maybe this is OK for Haiku ;)

An alternative to adding the library to the system would be to statically link it within the translator.

Though I'm generally one to dislike this sort of code duplication, I think it makes sense given how much open source libraries love to break their APIs between releases. All we need is the EXR translator (or any of them) broken because someone updated the OpenEXR library, or installed a newer one in their home directory, which our runtime_loader would load instead of the system one.

In fact we might want to do this with all our translators.

comment:4 Changed 7 years ago by leavengood

Actually I have a question for stefank:

What application is using OpenEXR? Could it be changed to use the Translation Kit on Haiku? Or does it use something special in OpenEXR?

comment:5 in reply to:  2 Changed 7 years ago by korli

Replying to axeld:

There should then be a dev package containing the headers. I think this already works this way for ICU.

I dunno how a dev package is supposed to be built. With a bep file?

comment:6 Changed 7 years ago by scottmc

There's a partially working .bep attached to this ticket: http://ports.haiku-files.org/ticket/144 But I see that the current version of openEXR is now at 1.7.1, so might update it in the process. I won't have time to do this for r1alpha4, but if someone else wants to try to get it working in time that's ok with me.

comment:7 Changed 4 years ago by diver

Probably should be merged in #10267.

comment:8 Changed 4 years ago by pulkomandy

Resolution: fixed
Status: newclosed

There is already an openexr package including development files.

Note: See TracTickets for help on using tickets.