Skip to content
This repository was archived by the owner on Jul 10, 2023. It is now read-only.

Update winapi to 0.3.4 #91

Closed
wants to merge 1 commit into from
Closed

Conversation

debris
Copy link
Contributor

@debris debris commented Jan 20, 2018

winapi features enabled

  • std - to make it c_void pointer compatible link with the one used by heapsize
  • heapapi - to enable heapapi link

This change is Reviewable

@jdm
Copy link
Member

jdm commented Jan 20, 2018

@bors-servo r+

@bors-servo
Copy link

📌 Commit b1a5eca has been approved by jdm

@bors-servo
Copy link

⌛ Testing commit b1a5eca with merge 6836052...

bors-servo pushed a commit that referenced this pull request Jan 20, 2018
Update winapi to 0.3.4

`winapi` features enabled

- `std` - to make it `c_void` pointer compatible [link](https://docs.rs/winapi/*/x86_64-pc-windows-msvc/src/winapi/lib.rs.html#31) with the one used by heapsize
- `heapapi` - to enable `heapapi` [link](https://github.com/retep998/winapi-rs#why-am-i-getting-errors-about-unresolved-imports)

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

Servo is not using this crate anymore. @debris, out of curiosity, where are you using it?

@debris
Copy link
Contributor Author

debris commented Jan 20, 2018

@SimonSapin we are using it in https://github.com/paritytech/parity

@bors-servo
Copy link

💔 Test failed - status-travis

@debris
Copy link
Contributor Author

debris commented Jan 21, 2018

build failure is unrelated No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.. Can someone restart travis?

@jdm
Copy link
Member

jdm commented Jan 21, 2018

@bors-servo retry

@bors-servo
Copy link

⌛ Testing commit b1a5eca with merge 2f02d80...

bors-servo pushed a commit that referenced this pull request Jan 21, 2018
Update winapi to 0.3.4

`winapi` features enabled

- `std` - to make it `c_void` pointer compatible [link](https://docs.rs/winapi/*/x86_64-pc-windows-msvc/src/winapi/lib.rs.html#31) with the one used by heapsize
- `heapapi` - to enable `heapapi` [link](https://github.com/retep998/winapi-rs#why-am-i-getting-errors-about-unresolved-imports)

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

debris commented Jan 21, 2018

lol, either one of the machines cached something or we are very unlucky :D

@bors-servo
Copy link

💔 Test failed - status-travis

@debris debris mentioned this pull request Jan 22, 2018
@debris debris closed this Jan 22, 2018
@debris debris deleted the update_winapi branch January 22, 2018 09:45
bors-servo pushed a commit that referenced this pull request Jan 28, 2018
Update winapi to 0.3.4

reopened #91

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/heapsize/93)
<!-- Reviewable:end -->
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.

4 participants