848624 - Change to IDL file without UUID change led to no change to interfaces.xpt

Status: NEW

(Firefox Build System :: General, defect)

Description

[Joe Drew (not getting mail)]

The change linked in the URL field caused intermittent failures because interfaces.xpt was not reliably changed in the packaged app; I am told by glandium and ted that this is likely because the UUID wasn't changed, and xpt.py doesn't compare methods or constants.

Comment 1

[(not currently active) Ted Mielczarek]

the xpt.py code in question is Interface.__cmp__:
http://mxr.mozilla.org/mozilla-central/source/xpcom/typelib/xpt/tools/xpt.py#885

glandium: are you using this in the packager somewhere? I think I optimized this for what xpt.py needed and no farther.

Comment 2

[Mike Hommey [:glandium]]

Yes, it's used in mozbuild.mozpack.files.XPTFile.copy