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

Python -s option not accepting arguments #1759

Closed
johaenns opened this issue Mar 22, 2018 · 7 comments
Closed

Python -s option not accepting arguments #1759

johaenns opened this issue Mar 22, 2018 · 7 comments
Assignees

Comments

@johaenns
Copy link

#1605 is not resolved.

Steps:

Call SCIRun -s anyscript.py arg1

Result:

-normal build
[PYTHON] Python 3.4.3 (default, Mar 20 2018, 14:57:40)
[GCC 4.8.5] on linux
Please note all args after script file name will be passed to python and not SCIRun!
[2018-03-22 17:08:49.533] [root] [error] File load failed (arg1): null xml returned.

-headless build
Please note all args after script file name will be passed to python and not SCIRun!
[SCIRun] RUNNING PYTHON SCRIPT: "anyscript.py"
[SCIRun] Attempting load of arg1
[SCIRun] File does not exist: arg1

Expected result:
SCIRun runs the script and passes arg1 as an argument.

opensuse leap 42.3, SCIRun v5.0-beta.S.

@dcwhite
Copy link
Member

dcwhite commented Mar 23, 2018

Thanks, I will work on this tonight.

@dcwhite dcwhite self-assigned this Mar 23, 2018
@dcwhite dcwhite added this to the 5.0-beta.T [Internal] milestone Mar 23, 2018
@dcwhite
Copy link
Member

dcwhite commented Mar 23, 2018

Looks like the problem is Linux-only. I kept "fixing" it on OSX. I'll try to get it fixed tomorrow.

@dcwhite
Copy link
Member

dcwhite commented Mar 23, 2018

I couldn't replicate on Win10 Ubuntu--perhaps it's OpenSUSE only. This will have to wait until I'm back next week when I can run my VM.

@dcwhite
Copy link
Member

dcwhite commented Apr 25, 2018

@johaenns Still can't replicate this on OpenSUSE. Please try the latest build. I'm building headless too to see if that makes a difference.

@dcwhite
Copy link
Member

dcwhite commented Apr 25, 2018

Also works in headless build mode. If it's still broken on your machine, you'll have to send me your script, or I'll need to debug on your machine directly.

@johaenns
Copy link
Author

I couldn't reproduce it anymore either, I assume it's fixed.

@dcwhite
Copy link
Member

dcwhite commented May 25, 2018

OK!

@dcwhite dcwhite closed this as completed May 25, 2018
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