Closed Bug 732474 Opened 13 years ago Closed 13 years ago

Install simplejson on windows and linux builders

Categories

(Infrastructure & Operations Graveyard :: CIDuty, task)

x86
Linux
task
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: jpr, Assigned: bhearsum)

References

Details

Attachments

(1 file)

Install simplejson on windows builders to support gyp conversion on Windows.
I'll get started on this on Monday.
Assignee: nobody → bhearsum
As noted on bug 643692, the Linux buildservers need the same thing
Also found bug 656572.
Attached patch simplejson opsi package (deleted) — Splinter Review
Attachment #603797 - Flags: review?(rail)
Comment on attachment 603797 [details] [diff] [review] simplejson opsi package Looks good to me. Could you also update the windows ref platform docs?
Attachment #603797 - Flags: review?(rail) → review+
(In reply to Rail Aliiev [:rail] from comment #5) > Comment on attachment 603797 [details] [diff] [review] > simplejson opsi package > > Looks good to me. Could you also update the windows ref platform docs? will do!
Attachment #603797 - Flags: checked-in+
set to deploy across the whole build pool. should be available everywhere after 24h at the most. ref image has also been updated, filed bug 733865 to get a new image of it taken
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Per comment 2, has this updated the Linux build servers as well?
Do we need it on Mac, too?
Status: RESOLVED → REOPENED
Component: Release Engineering → Release Engineering: Machine Management
QA Contact: release → armenzg
Resolution: FIXED → ---
Summary: Install simplejson on windows builders → Install simplejson on windows and linux builders
From the results on the builders for alder, it appears that the problem does not exist on Mac (I assume that they have 2.6+). (Note that broken OSX64 debug build is due to a bug in the imported webrtc code which should be fixed soon.)
bhearsum - please hold off on more work on this for now - after talking to ted (who is back part-time today), we removed the dependency on json. Verifying that this works. Thanks for your time on this; we appreciate it.
Your welcome, thanks for letting me right away. Should we close this as INCOMPLETE for now?
Verified that with ted's change json isn't needed.
Status: REOPENED → RESOLVED
Closed: 13 years ago13 years ago
Resolution: --- → INCOMPLETE
Product: mozilla.org → Release Engineering
Product: Release Engineering → Infrastructure & Operations
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: