can not set MOZ_XXX in .mozconfig

RESOLVED INVALID

Status

SeaMonkey
Build Config
RESOLVED INVALID
14 years ago
13 years ago

People

(Reporter: Thomas Leineweber, Assigned: (default assignee for unassigned bugs. does not receive bugmail))

Tracking

Trunk
Sun
Solaris

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

14 years ago
User-Agent:       Mozilla/5.0 (X11; U; SunOS sun4u; en-US; rv:1.5b) Gecko/20030809 Mozilla Firebird/0.6.1+
Build Identifier: Mozilla/5.0 (X11; U; SunOS sun4u; en-US; rv:1.5b) Gecko/20030809 Mozilla Firebird/0.6.1+

With my Solaris 2.6-environment, I can not set environment variables like
MOZ_PHOENIX=1 MOZ_THUNDERBIRD=1 in my .mozconfig file.

I put the following lines in .mozconfig:

  export MOZ_THUNDERBIRD=1
  mk_add_options MOZ_THUNDERBIRD=1

the following happens

[leineweb@theoden] make -f client.mk build
mozilla/build/autoconf/mozconfig2client-mk: MOZ_THUNDERBIRD=1: is not an identifier
make 
make[1]: Entering directory `/home-local/leineweb/tmp/thunderbird/mozilla'
....

I think, it is caused by /bin/sh (which is given as interpreting shell in
mozconfig2client-mk) not being a bash.

In contrast to it, my regular shell is:
  [leineweb@theoden] echo $BASH_VERSION 
  2.05.0(1)-release

So, I have to set the environment varibles outside the .mozconfig which
is slightly annoying, as I compile the different versions of mozilla
(SeaMonkey, Firebird, Thunderbird)

Reproducible: Always

Steps to Reproduce:
(Reporter)

Updated

14 years ago
OS: SunOS → Solaris
See http://mozilla.org/build/unix-details.html#s3 the first item in the "The
following software should be installed" list.
(Reporter)

Comment 2

14 years ago
I have read the given page and although I tried with setting the environment
variable SHELL, the given error message remains. It remains with the following
different commands:

  make SHELL=ksh -f client.mk build

  SHELL=ksh make -f clientk.mk build

(substitute ksh with /usr/bin/ksh gives the error message, too)

  export SHELL=ksh  ( or /usr/bin/ksh)
  make -f clientk.mk build

Example:

  [leineweb@theoden] make SHELL=ksh -f client.mk build
  mozilla/build/autoconf/mozconfig2client-mk: MOZ_THUNDERBIRD=1: is not an 
identifier
  cd /home-local/leineweb/tmp/thunderbird/mozilla
  ./configure
  ./configure: MOZ_THUNDERBIRD=1: is not an identifier
  *** Fix above errors and then restart with "make -f client.mk build"
  make: *** [/home-local/leineweb/tmp/thunderbird/mozilla/Makefile] Error 1

for your information: make is GNU Make version 3.79.1

the offending scripts (configure, mozconfig2client, perhaps some more)
are not called within an instance of the shell given in the SHELL variable,
but only use the /bin/sh given in their bang-line (when called from within the
makefiles)

>  export MOZ_THUNDERBIRD=1
try this:
MOZ_THUNDERBIRD=1
export MOZ_THUNDERBIRD

Comment 4

14 years ago
Do not use 'export'.  Just setting 'MOZ_THUNDERBIRD=1' should suffice.
Status: UNCONFIRMED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → INVALID
(Reporter)

Comment 5

14 years ago
If 'export' shall not be used, there should be no examples on mozilla.org
for .mozconfig-files, where it is used. Example:

http://www.mozilla.org/projects/thunderbird/build.html

Comment 6

14 years ago
It's not that 'export' cannot be used.  It's just not 'required' to be used. 
The mozconfig file is basically a shell script so what is allowed in the file is
really dependent upon your shell (/bin/sh).

Scott, can you remove the bash-isms from the thunderbird build instructions?

Comment 7

14 years ago
Chris, will things still build correctly for windows and other platforms without
that item?

Note: The Firebird build instructions also use export MOZ_PHOENIX in their build
instructions. See: 

http://lxr.mozilla.org/mozilla/source/browser/README.html

We'll need them to remove this line too.

Comment 8

14 years ago
I do not know of any platform which requires the 'export' directive to expose
that variable to configure.  We've been using the 'VAR=1' method on the various
ports tinderboxes for years.
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.