913246 - configure does not run correctly with relative path to alternate $MOZCONFIG

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Corey Ford [:coyotebush]]

New at least in the last day or two: my Android mozconfig doesn't work quite right.

$ MOZCONFIG=.mozconfig-android ./mach build
 0:00.15 /usr/bin/make -f client.mk -s
 0:00.31 Adding client.mk options from /Users/cford/code/mozilla-central/.mozconfig-android:
 0:00.31     CONFIG_GUESS=arm-linux-androideabi
 0:00.31     MOZ_OBJDIR=./objdir-droid
 0:00.31     AUTOCLOBBER=1
 0:00.31     FOUND_MOZCONFIG := /Users/cford/code/mozilla-central/.mozconfig-android
 0:00.49 Clobber not needed.
 0:00.51 cd ./objdir-droid
 0:00.51 /Users/cford/code/mozilla-central/configure
 0:00.53 Specified MOZCONFIG ".mozconfig-android" does not exist!
 0:00.53 Fix above errors before continuing.
 0:00.53 *** Fix above errors and then restart with               "/usr/bin/make -f client.mk build"
 0:00.53 make[2]: *** [configure] Error 1
 0:00.53 make[1]: *** [objdir-droid/Makefile] Error 2
 0:00.53 make: *** [build] Error 2
 0:00.58 0 compiler warnings present.

|MOZCONFIG=`pwd`/.mozconfig-android|, however, works.

It looks like .mozconfig-android is getting copied to objdir-droid/.mozconfig (without the -android), and that that name mismatch breaks things.

Comment 1

[Gregory Szorc [:gps]]

.mozconfig-android isn't a valid path. Try MOZCONFIG=./mozconfig-android (note the slash).

Comment 2

[Corey Ford [:coyotebush]]

(In reply to Gregory Szorc [:gps] from comment #1)
> .mozconfig-android isn't a valid path.

How so?

> Try MOZCONFIG=./mozconfig-android (note the slash).

$ MOZCONFIG=./.mozconfig-android ./mach build
 0:00.15 /usr/bin/make -f client.mk -s
 0:00.31 Adding client.mk options from /Users/cford/code/mozilla-central/./.mozconfig-android:
 0:00.31     CONFIG_GUESS=arm-linux-androideabi
 0:00.31     MOZ_OBJDIR=./objdir-droid
 0:00.31     AUTOCLOBBER=1
 0:00.31     FOUND_MOZCONFIG := /Users/cford/code/mozilla-central/./.mozconfig-android
 0:00.49 Clobber not needed.
 0:00.51 cd ./objdir-droid
 0:00.51 /Users/cford/code/mozilla-central/configure
 0:00.52 Specified MOZCONFIG "./.mozconfig-android" does not exist!
 0:00.52 Fix above errors before continuing.
 0:00.52 *** Fix above errors and then restart with               "/usr/bin/make -f client.mk build"
 0:00.52 make[2]: *** [configure] Error 1
 0:00.52 make[1]: *** [objdir-droid/Makefile] Error 2
 0:00.52 make: *** [build] Error 2
 0:00.53 0 compiler warnings present.

Comment 3

[Corey Ford [:coyotebush]]

(To be clear, once I've run configure with the full path, just |.mozconfig-android| works for rebuilds.)

Comment 4

[Gregory Szorc [:gps]]

Gah - I read the initial comment wrong. Teaches me to work while under the influence of a cold.

The problem is that configure is executing from the objdir. So MOZCONFIG=relativepath is different if pwd is srcdir or objdir. Things executing from srcdir are fine. But your mozconfig is getting lost as we chdir.

I can't recall if we ever supported this or if it's a regression.

Comment 5

[Gregory Szorc [:gps]]

Since you are complaining about it not working any more, it must be a regression. My guess says bug 911634.

Comment 6

[Corey Ford [:coyotebush]]

Right, I think that's what's happening.

My `history` suggests that I've never used anything but |MOZCONFIG=.mozconfig-android| from the beginning. Although to be fair https://wiki.mozilla.org/Android#Setup_Fennec_mozconfig does include $PWD in its instructions.

And as I noted, |.mozconfig-android| *does* end up copied to |objdir-droid/.mozconfig|.

Comment 8

[Gregory Szorc [:gps]]

I can't reproduce this :/

Can you run the following:

$ hg up 34ca46861c28^1
$ MOZCONFIG=./.mozconfig-android ./mach build

Comment 9

[Corey Ford [:coyotebush]]

That works okay. (My current tip, which doesn't work, is df8f342e9a6b.)

Comment 10

[Mike Hommey [:glandium]]

Attachment: Fix configure with relative MOZCONFIG

Comment 11

[Mike Hommey [:glandium]]

Attachment: Fix configure with relative MOZCONFIG

Comment 12

[Corey Ford [:coyotebush]]

That does the trick for me.

Comment 13

[Mike Hommey [:glandium]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/1b29c0885eb5

Comment 14

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/1b29c0885eb5