666424 - configure: command 'no' not found on a glib-less system

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Joey Armstrong [:joey]]

checking for glib-config... no
checking for GLIB - version >= 1.2.0... no
*** The glib-config script installed by GLIB could not be found
*** If GLIB was installed in PREFIX, make sure PREFIX/bin is in
*** your path, or set the GLIB_CONFIG environment variable to the
*** full path to glib-config.
/mozilla/sandbox/gml/configure: line 23769: no: command not found


ac_cv_path_GLIB_CONFIG=no is the culprit.

if test -z "${GLIB_GMODULE_LIBS}" -a -n "${GLIB_CONFIG}"; then
   GLIB_GMODULE_LIBS=`$GLIB_CONFIG gmoudle --libs`
fi

Comment 1

[Joey Armstrong [:joey]]

Attachment: extra conditional to test GLIB_CONFIG before using the cmd

Comment 2

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Comment on attachment 541217 [details] [diff] [review]
extra conditional to test GLIB_CONFIG before using the cmd

Generally in configure we don't bother breaking long lines.  r=me either way though.

Comment 3

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

http://hg.mozilla.org/projects/build-system/rev/fb91a23bd158

Comment 4

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

http://hg.mozilla.org/mozilla-central/rev/fb91a23bd158

Comment 5

[Matt Brubeck (:mbrubeck)]

Backed out during investigation of Android browser-chrome test failures:
http://hg.mozilla.org/mozilla-central/rev/00bb08972e46

Comment 6

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Relanded
http://hg.mozilla.org/mozilla-central/rev/b4c7d0319ea1