Skip to content

configure --enable-debug sub options cannot be overridden #24416

Closed
@pnkfelix

Description

@pnkfelix

Spawned off of #24405.

configure --enable-debug will enable CFG_ENABLE_LLVM_ASSERTIONS, CFG_ENABLE_DEBUG_ASSERTIONS, CFG_ENABLE_DEBUG_JEMALLOC,
and there is no way to turn them off.

The options --disable-{llvm-assertions,debug-assertions,debug-jemalloc} will all be silently ignored.

An analogous approach to that used by #24408 would probably work here.

But it might be better still to revisit how we are handling the option parsing in configure here, (for example, an invocation like configure --enable-debug --disable-debug has many potential interpretations, including emitting an error outright, but you might be surprised which one we choose under the current configure implementation). The strategy of "ignore arguments that already match the default" seems like a big pitfall in the script.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions