Closed Bug 1057305 Opened 5 years ago Closed 5 years ago

mach bootstrap doesn't support Elementary OS


(Firefox Build System :: General, defect)

Not set


(Not tracked)



(Reporter: Yoric, Assigned: yoloworkhard7, Mentored)



(Whiteboard: [lang=python][good first bug])


(1 file, 1 obsolete file)

Elementary OS is a fork of Ubuntu. Apparently, ./mach bootstrap fails with "NotImplementedError: Bootstrap support for this Linux distro not yet available."

We should fix this.
Mentor: dteller
Whiteboard: [lang=python][good first bug]
I am interested in working on this bug as I use Elementary OS and was stuck on this point while building firefox
The code lives here:

That's python/mozboot/ in your Firefox source directory.

My guess is that `distro` (defined on line 43) is something other than 'Elementary'.

Bonus points if you modify the exception raised on line 66 to display the value of `distro`.
Attached patch fix for the bug 1057305 (obsolete) — Splinter Review
Attachment #8477383 - Flags: review?(gps)
Attachment #8477383 - Flags: feedback+
Comment on attachment 8477383 [details] [diff] [review]
fix for the bug 1057305

I'm assuming that the previous patch was tested and that some versions of Elementary OS return 'Elementary' while others return '"elementary OS"'. If so, I would rather see `elif distro in ('Elementary', '"elementary OS"'):`
Attachment #8477383 - Flags: review?(gps)
Attachment #8477383 - Flags: feedback+
Product: Firefox → Core
Attached patch 200946.diffSplinter Review
Changed to fix the bug which prevented from running properly on Elementary OS
Attachment #8477383 - Attachment is obsolete: true
Attachment #8477498 - Flags: review?(gps)
Comment on attachment 8477498 [details] [diff] [review]

Review of attachment 8477498 [details] [diff] [review]:

Aside from a slightly malformed commit message, this looks good. I'll land it for you.
Attachment #8477498 - Flags: review?(gps) → review+
Assignee: nobody → yoloworkhard7
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.