This repository has been archived by the owner. It is now read-only.

MKVToolNix calls hard-coded strings executable in ac/endianess.m4 #1923

Closed
tgurr opened this Issue Mar 27, 2017 · 3 comments

Comments

3 participants
@tgurr

tgurr commented Mar 27, 2017

This results in an error running configure:

checking whether the byte order is big-endian... unknown
configure: error: Could not guess endianness, please use --with-words

Error:
  * In program cave perform install --hooks --managed-output --output-exclusivity with-others =media-video/mkvtoolnix-10.0.0:0::media-unofficial --destination installed --replacing =media-video/mkvtoolnix-9.9.0:0::installed --x-of-y 1 of 1:
  * When installing 'media-video/mkvtoolnix-10.0.0:0::media-unofficial' replacing { 'media-video/mkvtoolnix-9.9.0:0::installed' }:
  * When running an ebuild command on 'media-video/mkvtoolnix-10.0.0:0::media-unofficial':
  * Install failed for 'media-video/mkvtoolnix-10.0.0:0::media-unofficial' (paludis::ActionFailedError)


!!! ERROR in media-video/mkvtoolnix-10.0.0::media-unofficial:
!!! In econf at line 1503
!!! econf failed

on systems which have a multiarch layout and prefix the commands with the corresponding host triplet, so strings is for example:

/usr/x86_64-pc-linux-gnu/bin/x86_64-pc-linux-gnu-strings

The rest of the toolchain already gets detected without any problems by the build system and we've been sedding the call to the strings command on a distribution level for quite some time now.

    # TODO: report upstream, fix multiarch
    edo sed \
        -e "s:strings:/usr/$(exhost --target)/bin/$(exhost --tool-prefix)strings:g" \
        -i ac/endianess.m4
@mbunkus

This comment has been minimized.

Show comment
Hide comment
@mbunkus

mbunkus Mar 27, 2017

Owner

Thanks, I'll fix it.

Owner

mbunkus commented Mar 27, 2017

Thanks, I'll fix it.

@mbunkus

This comment has been minimized.

Show comment
Hide comment
@mbunkus

mbunkus Mar 27, 2017

Owner

Should be fixed by that commit. Please give it a try as I don't have a multiarch setup available to test with.

Owner

mbunkus commented Mar 27, 2017

Should be fixed by that commit. Please give it a try as I don't have a multiarch setup available to test with.

@tgurr

This comment has been minimized.

Show comment
Hide comment
@tgurr

tgurr Mar 28, 2017

Works fine, thanks!

tgurr commented Mar 28, 2017

Works fine, thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.