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

Fix crashing on unknown platforms #1670

Merged
merged 1 commit into from
Jul 15, 2023
Merged

Conversation

siepkes
Copy link
Contributor

@siepkes siepkes commented Jun 19, 2023

platform-info has an issue which causes it to fail hard on unknown platforms. This issue was fixed in newer versions however the latest released version (2.0.1) does not have the fix in it.

@netlify
Copy link

netlify bot commented Jun 19, 2023

Deploy Preview for maturin-guide ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 2309112
🔍 Latest deploy log https://app.netlify.com/sites/maturin-guide/deploys/64b25bb310d5d40008177277
😎 Deploy Preview https://deploy-preview-1670--maturin-guide.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@messense
Copy link
Member

Thanks, but usually we only allow crates.io dependencies, so we need to wait for a new platform-info release.

@T-256
Copy link

T-256 commented Jul 15, 2023

@siepkes you can pin 2.0.2 instead of git rev as it's now live on crates.io.

@siepkes siepkes force-pushed the unknown-platform-crash branch from e02c714 to 6ed84f8 Compare July 15, 2023 08:40
platform-info 2.0.1 has an issue which causes it to fail hard (i.e. panic) on unknown platforms.
@siepkes siepkes force-pushed the unknown-platform-crash branch from 6ed84f8 to 2309112 Compare July 15, 2023 08:41
@siepkes
Copy link
Contributor Author

siepkes commented Jul 15, 2023

@T-256 Thanks for the heads-up! I've updated the PR.

Copy link
Member

@messense messense left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@messense messense added this pull request to the merge queue Jul 15, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 15, 2023
@messense messense merged commit 1e75177 into PyO3:main Jul 15, 2023
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