1001800 - Linux nightly builds show conflicting configure args in about:buildconfig: " --enable-elf-hack [...] --disable-elf-hack"

Status: RESOLVED INVALID

(Firefox Build System :: General, defect)

Description

[Daniel Holbert [:dholbert]]

Just noticed this in the "about:buildconfig" page in my nightly:

=========
Configure arguments

--enable-update-channel=nightly --enable-update-packaging --with-google-api-keyfile=/builds/gapi.data --enable-crashreporter --enable-release --enable-elf-hack --enable-stdcxx-compat --enable-warnings-as-errors --enable-signmar --enable-profiling --disable-elf-hack --enable-js-diagnostics --with-ccache
=========

Note that this list contains both "--enable-elf-hack" and "--disable-elf-hack". We should probably pick one of those choices, instead of picking both. :)

I'm running:
 Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Firefox/31.0
 31.0a1 (2014-04-24)

Comment 1

[Daniel Holbert [:dholbert]]

I suspect the mozconfig for my nightly build is:
http://mxr.mozilla.org/mozilla-central/source/browser/config/mozconfigs/linux64/nightly

which has at the bottom:
> ac_add_options --disable-elf-hack # --enable-elf-hack conflicts with --enable-profiling

Before that, it includes a "common-opt" mozconfig, which gets us build/unix/mozconfig.linux, which has:
> ac_add_options --enable-elf-hack

It looks like both of those lines were added in this cset:
  http://hg.mozilla.org/mozilla-central/rev/f138a2a6c329
Adding dependency on that bug, bug 892355.

Comment 2

[Mike Hommey [:glandium]]

That's not a bug, that's how it's meant to be. (and autoconf conveniently only cares about the last on the command line). The real bug is that --enable-profiling requires --disable-elf-hack. Once /that/ is fixed, --disable-elf-hack can go away. (there's already a bug on file on that, don't have it off hand)