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

Build always regenerating manpage #108

Closed
hobbitalastair opened this issue Oct 26, 2016 · 1 comment
Closed

Build always regenerating manpage #108

hobbitalastair opened this issue Oct 26, 2016 · 1 comment

Comments

@hobbitalastair
Copy link
Contributor

hobbitalastair commented Oct 26, 2016 •
edited
Loading

Since 84ca965, if help2man is not installed configure will warn but make install will fail since flex.1 is never actually generated.

Ideally either the build would succeed without installing the man page or configure would fail if help2man was not installed.

EDIT: I seem to be completely wrong here. I'll take another look at what was causing the failure.
@hobbitalastair
Copy link
Contributor Author

Hmm - this actually appears to have been caused by 7cfb440. Although the man page is distributed with flex, the build system will always attempt to regenerate it since the flex binary has been rebuilt.

@hobbitalastair hobbitalastair mentioned this issue Oct 27, 2016
Closed
@hobbitalastair hobbitalastair changed the title Late failure if help2man is not installed Build always regenerating manpage Oct 27, 2016
@westes westes closed this as completed in 92311a5 Oct 27, 2016
@Explorer09 Explorer09 mentioned this issue Oct 29, 2016
Closed
alagoutte pushed a commit to alagoutte/flex that referenced this issue Dec 19, 2016
@hobbitalastair @alagoutte
Only regenerate the man page when required.
a94f726 
Make the flex binary an order-only prerequisite, and add back the
prerequisites from before 7cfb440. This prevents rebuilding the man page
whenever the flex binary is rebuilt, which causes problems if help2man
is not installed and will never work when cross compiling.

Fixes westes#108.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hobbitalastair