Skip to content

Some prerequisites to include gecko-media in Servo's dependency tree#19201

Merged
bors-servo merged 5 commits intomasterfrom
bumps
Nov 13, 2017
Merged

Some prerequisites to include gecko-media in Servo's dependency tree#19201
bors-servo merged 5 commits intomasterfrom
bumps

Conversation

@nox
Copy link
Copy Markdown
Contributor

@nox nox commented Nov 13, 2017

These are commits that I keep rebasing every time #19152 bitrots.

It also includes the cmake bump that supports jobservers for real.


This change is Reviewable

@highfive
Copy link
Copy Markdown

Heads up! This PR modifies the following files:

  • @wafflespeanut: python/tidy/servo_tidy/tidy.py
  • @edunham: python/tidy/servo_tidy/tidy.py

@highfive highfive added the S-awaiting-review There is new code that needs to be reviewed. label Nov 13, 2017
@jdm
Copy link
Copy Markdown
Member

jdm commented Nov 13, 2017

@bors-servo: r+

@bors-servo
Copy link
Copy Markdown
Contributor

📌 Commit 0c2ccc8 has been approved by jdm

@highfive highfive assigned jdm and unassigned Manishearth Nov 13, 2017
@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-awaiting-review There is new code that needs to be reviewed. labels Nov 13, 2017
@bors-servo
Copy link
Copy Markdown
Contributor

⌛ Testing commit 0c2ccc8 with merge b74a89a...

bors-servo pushed a commit that referenced this pull request Nov 13, 2017
Some prerequisites to include gecko-media in Servo's dependency tree

These are commits that I keep rebasing every time #19152 bitrots.

It also includes the `cmake` bump that supports jobservers for real.
@bors-servo
Copy link
Copy Markdown
Contributor

☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev
Approved by: jdm
Pushing b74a89a to master...

@bors-servo bors-servo merged commit 0c2ccc8 into master Nov 13, 2017
@highfive highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Nov 13, 2017
@SimonSapin SimonSapin deleted the bumps branch November 14, 2017 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants