Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

libjxl easyconfig fails to install if asciidoc is installed in OS #13827

Closed
boegel opened this issue Aug 25, 2021 · 6 comments · Fixed by #13829
Closed

libjxl easyconfig fails to install if asciidoc is installed in OS #13827

boegel opened this issue Aug 25, 2021 · 6 comments · Fixed by #13829
Milestone

Comments

@boegel
Copy link
Member

boegel commented Aug 25, 2021

libjxl added in #13453 fails for me with:

  File "/usr/bin/a2x", line 76
    print '%s: %s' % (PROG,msg)
                 ^
SyntaxError: invalid syntax
$ rpm -qf /usr/bin/a2x
asciidoc-8.6.8-5.el7.noarch
@boegel boegel added this to the next release (4.4.2?) milestone Aug 25, 2021
@branfosj
Copy link
Member

CMake option: JPEGXL_ENABLE_MANPAGES=OFF

@boegel
Copy link
Member Author

boegel commented Aug 25, 2021

@branfosj Hmm, no?

CMake Warning:
  Manually-specified variables were not used by the project:

    CMAKE_Fortran_COMPILER
    CMAKE_Fortran_FLAGS
    JPEGXL_ENABLE_MANPAGES

@branfosj
Copy link
Member

Ah. They've added that option since 0.3.7. :(

It looks like it will need patching out: https://github.com/libjxl/libjxl/blob/9e9bce86164dc4d01c39eeeb3404d6aed85137b2/CMakeLists.txt#L287

Or adding asciidoc as a build dependency.

@Micket
Copy link
Contributor

Micket commented Aug 25, 2021

We could look to update libjxl as well (they released it just after i made the PR)

@boegel
Copy link
Member Author

boegel commented Aug 25, 2021

I looked into asciidoc once, it was... messy.

So bumping to libjxl 0.5 and using -DJPEGXL_ENABLE_MANPAGES=OFF then?

@branfosj
Copy link
Member

Yes, bump libjxl to 0.5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants