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

bison: add missing build dependency #44245

Closed
wants to merge 1 commit into from

Conversation

sethrj
Copy link
Contributor

@sethrj sethrj commented May 17, 2024

Building on an "empty" rocky8 image, spack install bison fails because it's missing the texinfo dependency.

/tmp/root/spack-stage/spack-stage-bison-3.8.2-46ttmnqkblgfi4vpdckagxhibiywho5p/spack-src/build-aux/missing: line 81: makeinfo: command not found
WARNING: 'makeinfo' is missing on your system.
         You should only need it if you modified a '.texi' file, or
         any other file indirectly affecting the aspect of the manual.
         You might want to install the Texinfo package:
         <https://www.gnu.org/software/texinfo/>
         The spurious makeinfo call might also be the consequence of
         using a buggy 'make' (AIX, DU, IRIX), in which case you might
         want to install GNU make:
         <https://www.gnu.org/software/make/>
make[2]: *** [Makefile:9582: /tmp/root/spack-stage/spack-stage-bison-3.8.2-46ttmnqkblgfi4vpdckagxhibiywho5p/spack-src/doc/bison.info] Error 127
make[2]: Leaving directory '/tmp/root/spack-stage/spack-stage-bison-3.8.2-46ttmnqkblgfi4vpdckagxhibiywho5p/spack-src/spack-build'
make[1]: *** [Makefile:10377: all-recursive] Error 1
make[1]: Leaving directory '/tmp/root/spack-stage/spack-stage-bison-3.8.2-46ttmnqkblgfi4vpdckagxhibiywho5p/spack-src/spack-build

Copy link

spackbot-app bot commented May 17, 2024

Hi @sethrj! I noticed that the following package(s) don't yet have maintainers:

  • bison

Are you interested in adopting any of these package(s)? If so, simply add the following to the package class:

    maintainers("sethrj")

If not, could you contact the developers of this package and see if they are interested? You can quickly see who has worked on a package with spack blame:

$ spack blame bison

Thank you for your help! Please don't add maintainers without their consent.

You don't have to be a Spack expert or package developer in order to be a "maintainer," it just gives us a list of users willing to review PRs or debug issues relating to this package. A package can have multiple maintainers; just add a list of GitHub handles of anyone who wants to volunteer.

@sethrj sethrj added this to the v0.22.1 milestone May 17, 2024
@alalazo
Copy link
Member

alalazo commented May 17, 2024

Did you check why we need makeinfo ? Aren't we using a release tarball for bison? (I can also try to check this one later)

@alalazo alalazo self-assigned this May 17, 2024
@alecbcs
Copy link
Member

alecbcs commented May 20, 2024

I'm unable to reproduce the error (bison installs successfully) on a bare rhel8 image when building bison%gcc.
@sethrj are you using a different compiler?

@alecbcs alecbcs self-assigned this May 20, 2024
@alecbcs
Copy link
Member

alecbcs commented May 21, 2024

Duplicate of #44208

@alecbcs alecbcs marked this as a duplicate of #44208 May 21, 2024
@sethrj
Copy link
Contributor Author

sethrj commented May 21, 2024

@alecbcs Sorry, it looks like this is rocky 9, based on the Nvidia distribution: nvidia/cuda:12.4.1-devel-rockylinux9

@haampie
Copy link
Member

haampie commented May 21, 2024

I couldn't reproduce on ubuntu either. There must be some additional reason...

@haampie
Copy link
Member

haampie commented May 21, 2024

Ok, the real missing dependency is diffutils.

diff -u /tmp/root/spack-stage/spack-stage-bison-3.8.2-wf5jurn7clwd3m33gzmha6rberf6r6oo/spack-src/doc/cross-options.texi~ /tmp/root/spack-stage/spack-stage-bison-3.8.2-wf5jurn7clwd3m33gzmha6rberf6r6oo/spack-src/doc/cross-options.texi.tmp || true

@alecbcs
Copy link
Member

alecbcs commented May 21, 2024

@haampie where are you seeing the dependency on diffutils?

@haampie
Copy link
Member

haampie commented May 21, 2024

I thought I linked that, but it's used in doc/local.mk.

Hm, on second thought it looks like it's only used to dump to stdout, smth like diff -u a b || true.

Maybe the real issue is some race?

@haampie
Copy link
Member

haampie commented May 21, 2024

Ok, no, this is the actual issue:

/tmp/root/spack-stage/spack-stage-bison-3.8.2-wf5jurn7clwd3m33gzmha6rberf6r6oo/spack-src/build-aux/move-if-change doc/bison.help.tmp /tmp/root/spack-stage/spack-stage-bison-3.8.2-wf5jurn7clwd3m33gzmha6rberf6r6oo/spack-src/doc/bison.help
/tmp/root/spack-stage/spack-stage-bison-3.8.2-wf5jurn7clwd3m33gzmha6rberf6r6oo/spack-src/build-aux/move-if-change: line 69: cmp: command not found

and cmp is also from diffutils.

@sethrj sethrj deleted the bison-makeinfo branch May 21, 2024 20:56
@sethrj
Copy link
Contributor Author

sethrj commented May 21, 2024

Maybe there are many issues 🙃

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

Successfully merging this pull request may close these issues.

None yet

4 participants