Skip to content
New issue

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

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

Already on GitHub? # to your account

Use autotools standard ${docdir} #2468

Merged
merged 1 commit into from
Jun 16, 2023
Merged

Use autotools standard ${docdir} #2468

merged 1 commit into from
Jun 16, 2023

Conversation

h3xx
Copy link
Contributor

@h3xx h3xx commented Aug 26, 2022

Specified with --docdir=<path>, the $(docdir) variable is already available, and has the same default as what it was being set to.

By specifying it where it was, it was overriding what the --docdir option specified.

With the change, the value from the configure script's --docdir option will be respected instead.

Specified with `--docdir=<path>`, the `$(docdir)` variable is already
available, and has the same default as what it was being set to.

By specifying it here, it overrides what the `--docdir` option
specifies.
Copy link
Contributor

@itchyny itchyny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you.

@itchyny itchyny merged commit 076d0a2 into jqlang:master Jun 16, 2023
@h3xx h3xx deleted the fix-docdir branch June 16, 2023 15:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants