Skip to content
This repository was archived by the owner on Mar 20, 2024. It is now read-only.

Add an env var for cmake generator #28

Merged
merged 1 commit into from
Dec 12, 2018
Merged

Add an env var for cmake generator #28

merged 1 commit into from
Dec 12, 2018

Conversation

staktrace
Copy link
Contributor

@staktrace staktrace commented Dec 12, 2018

Some discussion in rust-lang/cmake-rs#66, it's not clear to me what the best solution is yet but I'm putting up this patch here anyway. It certainly solves my issue and unblocks me, so please feel free to merge this if it's acceptable.


This change is Reviewable

@jdm
Copy link
Member

jdm commented Dec 12, 2018

I declare this unobjectionable.
@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 2661d55 has been approved by jdm

@bors-servo
Copy link
Contributor

⌛ Testing commit 2661d55 with merge 91bd167...

bors-servo pushed a commit that referenced this pull request Dec 12, 2018
Add an env var for cmake generator

Some discussion in rust-lang/cmake-rs#66, it's not clear to me what the best solution is yet but I'm putting up this patch here anyway. It certainly solves my issue and unblocks me, so please feel free to merge this if it's acceptable.

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

☀️ Test successful - status-travis
Approved by: jdm
Pushing 91bd167 to master...

@bors-servo bors-servo merged commit 2661d55 into servo:master Dec 12, 2018
staktrace added a commit to staktrace/webrender that referenced this pull request Dec 13, 2018
This picks up servo/libfreetype2#28 which is desirable for building
pathfinder as part of webrender in Firefox CI on Windows.
@staktrace staktrace deleted the cmake branch December 27, 2018 16:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants