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

doc: A recipe in build-unix.md does not work #29755

Closed
hebasto opened this issue Mar 28, 2024 · 5 comments · Fixed by #29756
Closed

doc: A recipe in build-unix.md does not work #29755

hebasto opened this issue Mar 28, 2024 · 5 comments · Fixed by #29756

Comments

@hebasto
Copy link
Member

hebasto commented Mar 28, 2024

This does not work anymore:

Alternatively, or in addition, debugging information can be skipped for compilation. The default compile flags are
-g -O2, and can be changed with:

./configure CXXFLAGS="-O2"

However, I don't think that an immediate action should be taken. Anyway, it will be changed after migration to CMake.

@TheCharlatan
Copy link
Contributor

Mmh, just needs a note to override the debug flag?

@fanquake
Copy link
Member

fanquake commented Mar 28, 2024

This does not work

Exactly what doesn't work?

Anyway, it will be changed after migration to CMake.

This is irrelevant. We have to maintain the autotools branches for years into the future, and aren't going to leave them broken/with incorrect docs just because something else might exist.

@hebasto
Copy link
Member Author

hebasto commented Mar 28, 2024

This does not work

Exactly what doesn't work?

The resulted flags still contain the -g flag.

Anyway, it will be changed after migration to CMake.

This is irrelevant. We have to maintain the autotools branches for years into the future, and aren't going to leave them broken/with incorrect docs just because something else might exist.

Right.

@fanquake
Copy link
Member

Mmh, just needs a note to override the debug flag?

Yep, looks like that's all that's required.

@hebasto
Copy link
Member Author

hebasto commented Mar 28, 2024

Mmh, just needs a note to override the debug flag?

Yep, looks like that's all that's required.

Ok. Done in #29756.

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

Successfully merging a pull request may close this issue.

3 participants