Closed Bug 1636265 Opened 5 years ago Closed 5 years ago

The switch to Python 3 broke the Eclipse CDT project generating build backend

Categories

(Firefox Build System :: General, defect, P3)

defect

Tracking

(firefox-esr68 unaffected, firefox75 unaffected, firefox76 unaffected, firefox77 unaffected, firefox78 fixed)

RESOLVED FIXED
mozilla78
Tracking Status
firefox-esr68 --- unaffected
firefox75 --- unaffected
firefox76 --- unaffected
firefox77 --- unaffected
firefox78 --- fixed

People

(Reporter: jwatt, Assigned: jwatt)

Details

(Keywords: regression)

Attachments

(1 file)

No description provided.

Python 3 doesn't allow strings to be written to files opened in binary mode
(it requires a byte array in that case). As it happens, we should really be
opening these Eclipse config files in text mode since it seems on Windows the
files use Windows line ending characters. So rather than change the strings
to byte arrays, this patch simply changes the code to open the files in text
mode.

Pushed by jwatt@jwatt.org: https://hg.mozilla.org/integration/autoland/rev/14d0cb2955bf Fix the Eclipse CDT build backend (broken by the switch to Python 3). r=botond
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: