Skip to content
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

Do not use 'links' in Cargo.toml (fixes #97) #98

Merged
merged 1 commit into from Feb 2, 2017
Merged

Conversation

@nox
Copy link
Member

nox commented Jan 31, 2017

This change is Reviewable

@jdm
Copy link
Member

jdm commented Jan 31, 2017

If someone verifies that an application relying on core-foundation-rs still builds after this change, r=me.

@nox nox force-pushed the nox:rm-links branch from a4cfdec to 1c6bdc8 Feb 1, 2017
@nox
Copy link
Member Author

nox commented Feb 1, 2017

I checked with the cocoa examples.

@bors-servo r=jdm

@bors-servo
Copy link
Contributor

bors-servo commented Feb 1, 2017

📌 Commit 1c6bdc8 has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Feb 1, 2017

Testing commit 1c6bdc8 with merge 8a1d8a7...

bors-servo added a commit that referenced this pull request Feb 1, 2017
Do not use 'links' in Cargo.toml (fixes #97)

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/core-foundation-rs/98)
<!-- Reviewable:end -->
@jrmuizel
Copy link
Collaborator

jrmuizel commented Feb 2, 2017

I think you may need to kick the ci (travis had a bad day today).

@KiChjang KiChjang closed this Feb 2, 2017
@KiChjang KiChjang reopened this Feb 2, 2017
@nox
Copy link
Member Author

nox commented Feb 2, 2017

@bors-servo r=jdm

@bors-servo
Copy link
Contributor

bors-servo commented Feb 2, 2017

💡 This pull request was already approved, no need to approve it again.

  • This pull request is currently being tested. If there's no response from the continuous integration service, you may use retry to trigger a build again.
@bors-servo
Copy link
Contributor

bors-servo commented Feb 2, 2017

📌 Commit 1c6bdc8 has been approved by jdm

@nox
Copy link
Member Author

nox commented Feb 2, 2017

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Feb 2, 2017

Testing commit 1c6bdc8 with merge 7bb5bc4...

bors-servo added a commit that referenced this pull request Feb 2, 2017
Do not use 'links' in Cargo.toml (fixes #97)

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/core-foundation-rs/98)
<!-- Reviewable:end -->
@nox
Copy link
Member Author

nox commented Feb 2, 2017

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Feb 2, 2017

Testing commit 1c6bdc8 with merge b4b71a8...

bors-servo added a commit that referenced this pull request Feb 2, 2017
Do not use 'links' in Cargo.toml (fixes #97)

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/core-foundation-rs/98)
<!-- Reviewable:end -->
@nox
Copy link
Member Author

nox commented Feb 2, 2017

@bors-servo retry force clean

@nox
Copy link
Member Author

nox commented Feb 2, 2017

@bors-servo r- clean force

@nox
Copy link
Member Author

nox commented Feb 2, 2017

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Feb 2, 2017

Testing commit 1c6bdc8 with merge c1660ff...

bors-servo added a commit that referenced this pull request Feb 2, 2017
Do not use 'links' in Cargo.toml (fixes #97)

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/core-foundation-rs/98)
<!-- Reviewable:end -->
@nox nox merged commit 1c6bdc8 into servo:master Feb 2, 2017
1 of 2 checks passed
1 of 2 checks passed
homu Testing commit 1c6bdc8 with merge c1660ff...
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@nox nox deleted the nox:rm-links branch Feb 2, 2017
jdm pushed a commit that referenced this pull request Feb 1, 2018
Update bitflags to 1.0 and bump version

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/core-graphics-rs/98)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.