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.

Sign up for GitHub

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

Jump to bottom

Fails to build if xmllint is unavailable #257

Closed
ip1981 opened this issue May 16, 2014 · 4 comments
Closed

Fails to build if xmllint is unavailable #257

ip1981 opened this issue May 16, 2014 · 4 comments
Assignees
@edolstra
Labels
build-problem Nix fails to compile or test; also improvements to build process

Comments

@ip1981
Copy link

ip1981 commented May 16, 2014

Nix fails to build if xmllint is unavailable, but configure does not complain about missing xmllint. It just sets it to false:

configure.ac:154:AC_PATH_PROG(xmllint, xmllint, false)

# make V=1
false --nonet --xinclude doc/manual/manual.xml -o doc/manual/manual.xmli.tmp
doc/manual/local.mk:16: recipe for target 'doc/manual/manual.xmli' failed
@shlevy
Copy link
Member

shlevy commented May 16, 2014

I'm not certain, but I think this is because the official tarballs have the manual pre-built, so as long as you didn't change anything you should be fine without xmllint. Would be nice if it failed early when building from git though

@ip1981
Copy link
Author

ip1981 commented May 16, 2014

In official tarball if you run make clean it will remove all generated files :-) man pages, NEWS, etc. And btw NEWS requires w3m to build.

@gliptak
Copy link

gliptak commented Sep 28, 2014

Yes, without xmllint it fails with:

$ make
Makefile:27: Makefile.config: No such file or directory
  GEN    Makefile.config
  GEN    doc/manual/manual.xmli
make: *** [doc/manual/manual.xmli] Error 1

@edolstra edolstra added the build-problem Nix fails to compile or test; also improvements to build process label Feb 1, 2018
@shlevy shlevy added the backlog label Apr 1, 2018
@nh2 nh2 mentioned this issue Jul 3, 2019
Merged
@nh2
Copy link
Contributor

nh2 commented Jul 3, 2019

PR to fix it: #2979

@nh2 nh2 mentioned this issue Oct 26, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@edolstra edolstra

Labels
build-problem Nix fails to compile or test; also improvements to build process
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@gliptak @ip1981 @nh2 @shlevy @edolstra