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

Some test cases try to run ipy with -I and fail in release mode #1440

Open
slozier opened this issue May 7, 2022 · 3 comments
Open

Some test cases try to run ipy with -I and fail in release mode #1440

slozier opened this issue May 7, 2022 · 3 comments

Comments

@slozier
Copy link
Contributor

slozier commented May 7, 2022

This is basically like trying to import a module when running from the bin/Release folder without having set IRONPYTHONPATH. Because our standard libary is not in the normal location it fails to import.

For example.

bin/Release/net46/ipy.exe -I -c "import runpy"

will always fail because the standard library is not found on the path.

In 3.6, some tests, in test_gzip and test_macpath for example, use the test.support.script_helper.assert_python_ok method which runs ipy in isolated mode by default. This is causing failures.

A workaround was added to #1427 and should be reverted once this is resolved.


Edit for searchability: The -I command line options is used to isolate Python from the user's environment (implies -E and -s)

@BCSharp
Copy link
Member

BCSharp commented May 23, 2022

I know that this was done on purpose, but why the full Lib is not copied or linked into bin/Release/*/?

@slozier
Copy link
Contributor Author

slozier commented May 24, 2022

I know that this was done on purpose, but why the full Lib is not copied or linked into bin/Release/*/?

Not sure exactly what the reasoning was (if it was my call then I forgot all about it). We can certainly reconsider - I seem to recall a few cases of people having issues getting the release build to run because of this (e.g. #821).

@BCSharp
Copy link
Member

BCSharp commented May 24, 2022

I always assumed that it was to make the staging build simpler, but since my recent dive into the IronPython MSBuild scripts I don't think it makes much difference. Maybe worth trying and seeing what happens (beside having this issue go away).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants