538144 - "configure: line 18450: NS_HILDONFM: command not found"

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1262774750.1262781171.18213.gz&fulltext=1
OS X 10.5.2 mozilla-central build on 2010/01/06 02:45:50
{
checking for conic... checking for libhildonmime... checking for libosso... checking for hildon-fm-2... /builds/slave/mozilla-central-macosx/build/configure: line 18450: NS_HILDONFM: command not found
}

http://tinderbox.mozilla.org/showlog.cgi?log=Thunderbird/1262768093.1262776913.2276.gz&fulltext=1
MacOSX 10.5 comm-central build on 2010/01/06 00:54:53
Same.

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1262768113.1262771771.8722.gz&fulltext=1
OS X 10.5 comm-central-trunk build on 2010/01/06 00:55:13
No error: not sure why.

NB: I didn't check Linux and Windows.

Comment 1

[Turuoka (Kyoya Sakaguchi)]

Attachment: Patch v1.0

Comment 2

[(not currently active) Ted Mielczarek]

Comment on attachment 421247 [details] [diff] [review]
Patch v1.0

Nice catch, thanks!

Comment 3

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

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

Comment 4

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

http://hg.mozilla.org/mozilla-central/rev/50a66336ba73

oops, sorry for the spam.

Comment 5

[Serge Gautherie (:sgautherie)]

Comment on attachment 421247 [details] [diff] [review]
Patch v1.0


"approval1.9.2.1=?":
Zero risk, build config typo.

Comment 6

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 421247 [details] [diff] [review]
Patch v1.0

Build config change, should take when the tree re-opens for 1.9.2.3

Comment 7

[LegNeato]

Comment on attachment 421247 [details] [diff] [review]
Patch v1.0

a=LegNeato for 1.9.2.5. Please ONLY land this on mozilla-1.9.2 default, as we
are still working on 1.9.2.4 on the relbranch

Comment 8

[(no longer active)]

1.9.2's configure.in does not have this code, so it's not affected (I guess).

Comment 9

[Serge Gautherie (:sgautherie)]

(In reply to comment #8)
> 1.9.2's configure.in does not have this code, so it's not affected (I guess).

Ftr, m-1.9.2 doesn't have this code anymore, because bug 543976 removed it in the meantime.