374278 - xpidl can't find IDL files for java when building firefox with --with-libxul-sdk

Status: RESOLVED INVALID

Description

[:Mook]

When building firefox with --with-libxul-sdk and --enable-javaxpcom (i.e. Firefox-on-xulrunner with javaxpcom explicitly enabled for Firefox), the build dies building the Java interface for nsIDictionary:

$(topobjdir)/xulrunner/dist/host/bin/host_xpidl -m java -w -I$(topsrcdir)/extensions/xml-rpc/idl -I../../../dist/idl -o _javagen/org/mozilla/interfaces/nsIDictionary $(topsrcdir)/extensions/xml-rpc/idl/nsIDictionary.idl
$(topsrcdir)/extensions/xml-rpc/idl/nsIDictionary.idl:46: can't open included file nsISupports.idl for reading

Since nsISupports is in xulrunner's dist/idl of course it can't find it in Firefox's dist/idl.  I think the fix is to use $(XPIDL_FLAGS) as the .h and .xpt generation does; filing bug first before I try to patch.

Comment 1

[:Mook]

Created attachment 258864 [details] [diff] [review]
use XPIDL_FLAGS instead

The guess appears to work (at least, the build got past that point).  Looking into unrelated problems now.

Sorry about the default QA, I initially filed as a Firefox/Build Config thing then realized it should be in Core instead, and forgot to change the QA.

Comment 2

[Benjamin Smedberg]

--enable-javaxpcom is only meant for builds that are making lixul