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

Missing AC_SUBST(dbdir) breaks installation with 4.8.0#306

Closed
atsampson opened this issue Dec 6, 2023 · 2 comments
Closed

Missing AC_SUBST(dbdir) breaks installation with 4.8.0 #306

atsampson opened this issue Dec 6, 2023 · 2 comments

Comments

@atsampson
Copy link

Commit 526a8ce removed the --with-dbfile option from configure.ac. However, it accidentally removed AC_SUBST(dbdir) as well - so the use of @dbdir@ in Makefile.in won't get expanded. This will cause make install to either fail or create a nonsense directory.

Adding AC_SUBST(dbdir) fixes it, but since there's no guarantee that any files will end up there now, it may be better to remove dbdir from Makefile.in entirely?

It'd also be a good idea to test make install as part of the release process.

@wcawijngaards
Copy link
Member

Thank you for the report! That escaped notice, the fix is to remove it from Makefile.in, since it is no longer used. The configure option is printed to inform the user in the help output and backwards compatibility, but there is no need for database directory or file. The commit removes the creation of the @dbdir@ directory, and also fixes dbfile references in the file edit and uninstall information, and in the nsd.conf man page that listed the database option in an example and database file under the FILES section of the man page.

Apart from briefly creating a spurious @dbdir@ directory, it does not misbehave during install for me. The fix removes that directory from being created. A test would likely not have caught it, since the mkdir command succeeds.

wcawijngaards added a commit that referenced this issue Dec 6, 2023
@wcawijngaards
Copy link
Member

The additional commit fixes that for the xfrd.state and zone.list files, their directory is created at install time. This is more precise than creating the dbdir, it makes their directories also if they have been set, with configure options, to be stored at different locations.

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

No branches or pull requests

2 participants