Skip to content

Miscellaneous pip fixes - #3158

Merged
coolreader18 merged 8 commits into
mainfrom
pip-warnings
Oct 1, 2021
Merged

coolreader18 merged 8 commits into
mainfrom
pip-warnings

Conversation

@coolreader18

Copy link
Copy Markdown
Member

Fix pip warnings about inconsistency between distutils' sysconfig and the actual sysconfig; fix error when installing package it would say something like __env_pip__.zip is not a file

@coolreader18
coolreader18 force-pushed the pip-warnings branch 2 times, most recently from 46f5732 to 32aa7d4 Compare September 30, 2021 22:04
Comment thread Cargo.toml
opt-level = 3

[profile.test]
opt-level = 3

@youknowone youknowone Oct 1, 2021 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

doesn't this obfuscate backtrace and debug information?
how do you think about running tests with release build job if this is desired?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It should still keep debug info I'm pretty sure - the default for test enables debuginfo and I'm pretty sure it's orthogonal to opt-level: https://doc.rust-lang.org/nightly/cargo/reference/profiles.html#test

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

When opt-level is not 0, the inlined callstacks are omitted, the progress is not linear to the code and some breakpoint doesn't work.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think the default development environment is better to be friendly to programmers unless there are unavoidable reasons

@coolreader18
coolreader18 merged commit b38ce3d into main Oct 1, 2021
@coolreader18
coolreader18 deleted the pip-warnings branch October 1, 2021 18:34
@fanninpm

fanninpm commented Oct 1, 2021

Copy link
Copy Markdown
Contributor

13 unexpected successes from a test that wasn't in the codebase when you opened this — sounds like another reason for bors.

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