Closed Bug 1186896 Opened 9 years ago Closed 9 years ago

./mach update-uuids is broken from the recent IDL Parser changes

Categories

(Firefox Build System :: Mach Core, enhancement)

enhancement
Not set
normal

Tracking

(firefox42 fixed)

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: RyanVM, Assigned: new_one)

References

Details

Attachments

(1 file)

ryanvm@ubuntu:~/repos/mozilla-inbound$ ./mach update-uuids nsIAudioChannelAgent
Error running mach:

    ['update-uuids', 'nsIAudioChannelAgent']

The error occurred in the implementation of the invoked mach command.

This should never occur and is likely a bug in the implementation of that
command. Consider filing a bug for this issue.

If filing a bug, please include the full output of mach, including this error
message.

The details of the failure are as follows:

AttributeError: 'module' object has no attribute 'IDLParser'

  File "/home/ryanvm/repos/mozilla-inbound/tools/mach_commands.py", line 183, in update_uuids
    parser = xpidl.IDLParser(outputdir=tmpdir)
Flags: needinfo?(peter.elmers)
Updated two import statements in mach_commands to the new path of the xpidl module. Grepped for any other "import xpidl" in the code base and found none outside the idl-parser itself.
Attachment #8637934 - Flags: review?(ryanvm)
Assignee: nobody → peter.elmers
Status: NEW → ASSIGNED
Flags: needinfo?(peter.elmers)
Comment on attachment 8637934 [details] [diff] [review]
Fix import statements for mach update-uuids

I'm not a peer of this code.
Attachment #8637934 - Flags: review?(ryanvm) → review?(gps)
Comment on attachment 8637934 [details] [diff] [review]
Fix import statements for mach update-uuids

Review of attachment 8637934 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for fixing!
Attachment #8637934 - Flags: review?(gps) → review+
(In reply to Gregory Szorc [:gps] from comment #3)
> Comment on attachment 8637934 [details] [diff] [review]
> Fix import statements for mach update-uuids
> 
> Review of attachment 8637934 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Thanks for fixing!

Thanks for review! Should I run any tests, or is this ready for check in?
(In reply to Peter Elmers [:new_one] from comment #4)
> Thanks for review! Should I run any tests, or is this ready for check in?

Please use needinfo! I almost missed this and only got to it just now because I'm behind on non-flagged bugmail.

As long as it works locally, this is good for checkin.
Ah, will use needinfo next time! The fix does work locally.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/992a26539e87
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: