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

Try fixing CI #193

Closed
wants to merge 3 commits into from
Closed

Try fixing CI #193

wants to merge 3 commits into from

Conversation

bschwind
Copy link
Contributor

CI is failing when trying to install libusb, I'm investigating why in this PR.

  • Check test_all.sh is passed.
  • Add fix #{ISSUE_NUMBER} if the corresponding issue exists.
  • Fill out ## Changelog section. If the change is for only internal use, please write None to the section.

@bschwind
Copy link
Contributor Author

bschwind commented Sep 28, 2024

Possibly related:

microsoft/vcpkg#41154

This should be fixed once this is merged.

ihnorton added a commit to TileDB-Inc/TileDB that referenced this pull request Sep 30, 2024
Fix rtools40 build. vcpkg-tool released a change which no longer treated environment variables as case-insensitive, then removed the variables -- breaking tools which depend on those variables.

Explanation: microsoft/vcpkg#41199 (comment)

x-ref:
```
- microsoft/vcpkg-tool#1501
- via cameleon-rs/cameleon#193
- via microsoft/vcpkg#41154
```
---
TYPE: NO_HISTORY
DESC: Fix rtools40 build.
@Y-Nak
Copy link
Member

Y-Nak commented Oct 2, 2024

Thank you for the investigation!
I confirmed that the issue is fixed on the vcpkg side and thus fixed in the master.

Close as completed.

@Y-Nak Y-Nak closed this Oct 2, 2024
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.

2 participants