[MacPorts] #46754: llvm-3.6 does not recognize -assertions
#46754: llvm-3.6 does not recognize -assertions ----------------------+------------------------ Reporter: pguyot@… | Owner: jeremyhu@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.3 Keywords: | Port: llvm ----------------------+------------------------ Even when {{{-assertions}}} is passed, assertion mode is ON. I believe {{{--enable-assertions}}} is NOP and {{{--disable-assertions}}} should be passed instead as assertions are enabled by default. This probably is also the case with other versions of llvm. {{{./configure --help}}} indeed prints: {{{ --enable-assertions Compile with assertion checks enabled (default is YES) }}} A patch is attached that fixes this. I didn't bump {{{revision}}}, though. -- Ticket URL: <https://trac.macports.org/ticket/46754> MacPorts <https://www.macports.org/> Ports system for OS X
#46754: llvm-3.* does not recognize -assertions -----------------------+------------------------ Reporter: pguyot@… | Owner: jeremyhu@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.3 Resolution: | Keywords: Port: llvm | -----------------------+------------------------ Description changed by pguyot@…: Old description:
Even when {{{-assertions}}} is passed, assertion mode is ON.
I believe {{{--enable-assertions}}} is NOP and {{{--disable-assertions}}} should be passed instead as assertions are enabled by default. This probably is also the case with other versions of llvm.
{{{./configure --help}}} indeed prints:
{{{ --enable-assertions Compile with assertion checks enabled (default is YES) }}}
A patch is attached that fixes this. I didn't bump {{{revision}}}, though.
New description: Even when {{{-assertions}}} is passed, assertion mode is ON. I believe {{{--enable-assertions}}} is NOP and {{{--disable-assertions}}} should be passed instead as assertions are enabled by default. This probably is also the case with other versions of llvm. {{{./configure --help}}} indeed prints: {{{ --enable-assertions Compile with assertion checks enabled (default is YES) }}} A patch is attached that fixes this for llvm-3.6. I didn't bump {{{revision}}}, though. Yet, bumping revision is required at least for llvm-3.5 as the -assertions variant is available as a binary package (but with assertion-mode as ON). -- -- Ticket URL: <https://trac.macports.org/ticket/46754#comment:1> MacPorts <https://www.macports.org/> Ports system for OS X
#46754: llvm-3.* does not recognize -assertions -------------------------------------------+------------------------ Reporter: pguyot@… | Owner: jeremyhu@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.3 Resolution: | Keywords: Port: llvm-3.5, llvm-3.6, llvm-3.7 | -------------------------------------------+------------------------ Changes (by pguyot@…): * port: llvm => llvm-3.5, llvm-3.6, llvm-3.7 Old description:
Even when {{{-assertions}}} is passed, assertion mode is ON.
I believe {{{--enable-assertions}}} is NOP and {{{--disable-assertions}}} should be passed instead as assertions are enabled by default. This probably is also the case with other versions of llvm.
{{{./configure --help}}} indeed prints:
{{{ --enable-assertions Compile with assertion checks enabled (default is YES) }}}
A patch is attached that fixes this for llvm-3.6. I didn't bump {{{revision}}}, though. Yet, bumping revision is required at least for llvm-3.5 as the -assertions variant is available as a binary package (but with assertion-mode as ON).
New description: Even when {{{-assertions}}} is passed, assertion mode is ON. I believe {{{--enable-assertions}}} is NOP and {{{--disable-assertions}}} should be passed instead as assertions are enabled by default. This is also the case with at least 3.7 and 3.5 and maybe other versions as well. {{{./configure --help}}} indeed prints: {{{ --enable-assertions Compile with assertion checks enabled (default is YES) }}} A patch is attached that fixes this for llvm-3.6. I didn't bump {{{revision}}}, though. Yet, bumping revision is required at least for llvm-3.5 as the -assertions variant is available as a binary package (but with assertion-mode as ON). -- -- Ticket URL: <https://trac.macports.org/ticket/46754#comment:2> MacPorts <https://www.macports.org/> Ports system for OS X
#46754: llvm-3.* does not recognize -assertions -----------------------------------------+------------------------ Reporter: pguyot@… | Owner: jeremyhu@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.3 Resolution: | Keywords: Port: llvm-3.5 llvm-3.6 llvm-3.7 | -----------------------------------------+------------------------ Changes (by larryv@…): * cc: larrryv@… (added) * port: llvm-3.5, llvm-3.6, llvm-3.7 => llvm-3.5 llvm-3.6 llvm-3.7 -- Ticket URL: <https://trac.macports.org/ticket/46754#comment:3> MacPorts <https://www.macports.org/> Ports system for OS X
#46754: llvm-3.* does not recognize -assertions -------------------------------------------------+------------------------- Reporter: pguyot@… | Owner: jeremyhu@… Type: defect | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 2.3.3 Resolution: | Keywords: Port: llvm-3.3 llvm-3.4 llvm-3.5 | llvm-3.6 llvm-3.7 | -------------------------------------------------+------------------------- Changes (by jeremyhu@…): * status: new => assigned * port: llvm-3.5 llvm-3.6 llvm-3.7 => llvm-3.3 llvm-3.4 llvm-3.5 llvm-3.6 llvm-3.7 Comment: Yep, it also impacts 3.3 and 3.4. I'll be updating this soon. -- Ticket URL: <https://trac.macports.org/ticket/46754#comment:4> MacPorts <https://www.macports.org/> Ports system for OS X
#46754: llvm-3.* does not recognize -assertions -------------------------------------------------+------------------------- Reporter: pguyot@… | Owner: jeremyhu@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.3.3 Resolution: fixed | Keywords: Port: llvm-3.3 llvm-3.4 llvm-3.5 | llvm-3.6 llvm-3.7 | -------------------------------------------------+------------------------- Changes (by jeremyhu@…): * status: assigned => closed * resolution: => fixed Comment: r132960 -- Ticket URL: <https://trac.macports.org/ticket/46754#comment:5> MacPorts <https://www.macports.org/> Ports system for OS X
participants (1)
-
MacPorts