Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 737474 - =media-libs/ilmbase-2.5.2 - Fails to build with musl
Summary: =media-libs/ilmbase-2.5.2 - Fails to build with musl
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Bernd
URL:
Whiteboard:
Keywords: PATCH, PullRequest
Depends on:
Blocks: musl-porting
  Show dependency tree
 
Reported: 2020-08-16 20:53 UTC by tonemgub
Modified: 2020-08-18 10:13 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
musl patch (musl.patch,2.08 KB, patch)
2020-08-16 21:40 UTC, tonemgub
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description tonemgub 2020-08-16 20:53:44 UTC
Build fails with:

/var/tmp/portage/media-libs/ilmbase-2.5.2/work/openexr-2.5.2/IlmBase/IexMath/IexMathFpu.cpp: In function 'void Iex_2_5::FpuControl::restoreControlRegs(const ucontext_t&, bool)':
/var/tmp/portage/media-libs/ilmbase-2.5.2/work/openexr-2.5.2/IlmBase/IexMath/IexMathFpu.cpp:284:38: error: 'struct _fpstate' has no member named 'cw'; did you mean 'cwd'?
  284 |     setCw ((ucon.uc_mcontext.fpregs->cw & cwRestoreMask) | cwRestoreVal);
      |                                      ^~
      |                                      cwd
/var/tmp/portage/media-libs/ilmbase-2.5.2/work/openexr-2.5.2/IlmBase/IexMath/IexMathFpu.cpp:287:20: error: 'struct Iex_2_5::FpuControl::_fpstate_64' has no member named 'magic'
  287 |     setMxcsr (kfp->magic == 0 ? kfp->mxcsr : 0, clearExceptions);


Fixed in current ilmbase trunk. Patch:
https://github.com/AcademySoftwareFoundation/openexr/commit/c7af102e6bce6638add2f38576ffe9c6741ba768

Reproducible: Always



Expected Results:  
Should build
Comment 1 tonemgub 2020-08-16 21:40:10 UTC
Created attachment 654998 [details, diff]
musl patch
Comment 2 Bernd 2020-08-17 19:05:17 UTC
Thanks for the patch. I noticed the discussion while watching the issues and/or mailing list, but didn't realize it is relevant for us. 
I'm going to update the ebuild.
Comment 3 Bernd 2020-08-17 19:55:31 UTC
Do I have to remove the already stabilized arches from bug #717474 and run the stabilization request again?
The fix is very minor.
Comment 4 David Seifert gentoo-dev 2020-08-17 20:03:28 UTC
(In reply to Bernd from comment #3)
> Do I have to remove the already stabilized arches from bug #717474 and run
> the stabilization request again?
> The fix is very minor.

no
Comment 5 Larry the Git Cow gentoo-dev 2020-08-18 10:13:55 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=b91a3525b936c1510e18570acb34edac890e7cff

commit b91a3525b936c1510e18570acb34edac890e7cff
Author:     Bernd Waibel <waebbl@gmail.com>
AuthorDate: 2020-08-17 20:16:16 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2020-08-18 10:13:41 +0000

    media-libs/ilmbase: fix build against musl
    
    Reported-by: tonemgub@cool.fr.nf <tonemgub@cool.fr.nf>
    Closes: https://bugs.gentoo.org/737474
    Package-Manager: Portage-3.0.2, Repoman-2.3.23
    Signed-off-by: Bernd Waibel <waebbl@gmail.com>
    Closes: https://github.com/gentoo/gentoo/pull/17152
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>

 media-libs/ilmbase/files/ilmbase-2.5.2-musl.patch | 50 +++++++++++++++++++++++
 media-libs/ilmbase/ilmbase-2.5.2.ebuild           |  2 +
 2 files changed, 52 insertions(+)