Skip to content

Use python2 for the build whenever possible and bump the version. #132

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 26, 2018

Conversation

marmistrz
Copy link
Contributor

@marmistrz marmistrz commented Feb 26, 2018

If python2 seems to be a valid Python 2 interpreter, the configure
script will use it. Otherwise, it will fall back to python, as
previously.

We also bump the crate version so that this change can be propagated
further the dependency tree.

Closes #129.


This change is Reviewable

If python2 seems to be a valid Python 2 interpreter, the configure
script will use it. Otherwise, it will fall back to python, as
previously.

We also bump the crate version so that this change can be propagated
further the dependency tree.

Closes servo#129.
@jdm
Copy link
Member

jdm commented Feb 26, 2018

@bors-servo r+
Thanks!

@bors-servo
Copy link
Contributor

📌 Commit 53a632f has been approved by jdm

@bors-servo
Copy link
Contributor

⌛ Testing commit 53a632f with merge 57b7967...

bors-servo pushed a commit that referenced this pull request Feb 26, 2018
Use python2 for the build whenever possible and bump the version.

If python2 seems to be a valid Python 2 interpreter, the configure
script will use it. Otherwise, it will fall back to python, as
previously.

We also bump the crate version so that this change can be propagated
further the dependency tree.

Closes #129.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/mozjs/132)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - status-appveyor, status-travis
Approved by: jdm
Pushing 57b7967 to master...

@bors-servo bors-servo merged commit 53a632f into servo:master Feb 26, 2018
marmistrz added a commit to marmistrz/rust-mozjs that referenced this pull request Mar 1, 2018
bors-servo pushed a commit to servo/rust-mozjs that referenced this pull request Mar 1, 2018
Bump the versions to propagate servo/mozjs#132

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-mozjs/392)
<!-- Reviewable:end -->
bors-servo pushed a commit to servo/servo that referenced this pull request Mar 7, 2018
Update mozjs_sys

Pick up servo/mozjs#132

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/20232)
<!-- Reviewable:end -->
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Mar 8, 2018
Pick up servo/mozjs#132

Source-Repo: https://github.com/servo/servo
Source-Revision: 72d09202f4e5207cee606e122729dd007e6b5ac8

--HG--
extra : subtree_source : https%3A//hg.mozilla.org/projects/converted-servo-linear
extra : subtree_revision : adc5d3dd7e14186227285981b3f8c74158cfbafd
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Oct 2, 2019
Pick up servo/mozjs#132

Source-Repo: https://github.com/servo/servo
Source-Revision: 72d09202f4e5207cee606e122729dd007e6b5ac8

UltraBlame original commit: 2ace79cd8e52c1937fc45ef7167faff376fb2ad5
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Oct 2, 2019
Pick up servo/mozjs#132

Source-Repo: https://github.com/servo/servo
Source-Revision: 72d09202f4e5207cee606e122729dd007e6b5ac8

UltraBlame original commit: 2ace79cd8e52c1937fc45ef7167faff376fb2ad5
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Oct 2, 2019
Pick up servo/mozjs#132

Source-Repo: https://github.com/servo/servo
Source-Revision: 72d09202f4e5207cee606e122729dd007e6b5ac8

UltraBlame original commit: 2ace79cd8e52c1937fc45ef7167faff376fb2ad5
bhearsum pushed a commit to mozilla-releng/staging-firefox that referenced this pull request May 1, 2025
Pick up servo/mozjs#132

Source-Repo: https://github.com/servo/servo
Source-Revision: 72d09202f4e5207cee606e122729dd007e6b5ac8
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.

3 participants