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

Request for documentation improvement for configure_in_place field #853

Closed
garrettkajmowicz-sophos opened this issue Jan 6, 2022 · 3 comments · Fixed by #856
Closed

Comments

@garrettkajmowicz-sophos
Copy link

I'm working with the configure_make() command, and the configure_in_place option isn't clear. I wouldn't worry too much about it except that a number of other commands seem to require that it be set, and that when I do set it I get errors from the configure step about "config.log: Read-only file system" and so on.

Clarification of the difference in operation when this value is True/False (as well as guidance as to when someone might want to use it) would be greatly appreciated.

@jsharpe
Copy link
Member

jsharpe commented Jan 6, 2022

By default configure_make attempts to do an out of tree build. This is broken in some codebases and they require you do run configure within the source tree itself. This is the idea behind configure_in_place, however it symlinks most of the source files into the build directory and so if your configure scripts attempt to modify any of the files from that original tree then it will fail as you are seeing. It probably needs to be switched to use a copy rather than a symlink to fix this behaviour.

@jsharpe
Copy link
Member

jsharpe commented Jan 6, 2022

In fact I previously started to fix this: #596 but never landed the PR

@garrettkajmowicz-sophos
Copy link
Author

I would definitely be in-favor of seeing this land.

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