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

reference implementation install fails #47

Closed
hmenager opened this issue May 26, 2015 · 7 comments
Closed

reference implementation install fails #47

hmenager opened this issue May 26, 2015 · 7 comments

Comments

@hmenager
Copy link
Member

I tried a clean install of cwltool today, and it seems to fail because "sandboxjs" lib is missing:

~/cwl$ cwltool
Traceback (most recent call last):
  File "/home/hmenager/cwl/bin/cwltool", line 9, in 
    load_entry_point('cwltool==1.0.20150525010411', 'console_scripts', 'cwltool')()
  File "/home/hmenager/cwl/local/lib/python2.7/site-packages/pkg_resources.py", line 353, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/home/hmenager/cwl/local/lib/python2.7/site-packages/pkg_resources.py", line 2321, in load_entry_point
    return ep.load()
  File "/home/hmenager/cwl/local/lib/python2.7/site-packages/pkg_resources.py", line 2048, in load
    entry = __import__(self.module_name, globals(),globals(), ['__name__'])
  File "/home/hmenager/cwl/local/lib/python2.7/site-packages/cwltool/main.py", line 3, in 
    import draft1tool
  File "/home/hmenager/cwl/local/lib/python2.7/site-packages/cwltool/draft1tool.py", line 4, in 
    import sandboxjs
ImportError: No module named sandboxjs

any pointers?

@hmenager
Copy link
Member Author

Thank you Paul, I'm just wondering wether it should still be on the master branch or not. it is called only from the draft1tool module, and I guess draft1tool is about draft 1, but the master should now only handle draft2 shouldn't it?

@hmenager
Copy link
Member Author

Sorry, I just noticed draft2 also uses sandboxjs, so the problem actually is that sanboxjs module is not in the master branch.

@tetron
Copy link
Member

tetron commented May 27, 2015

Sorry about that, I removed it but had a stale sandboxjs pyc so I didn't realize it broke things. I will fix it and look into setting up a testing server to catch these things in the future.

@tetron
Copy link
Member

tetron commented May 27, 2015

This should be resolved, can you confirm?

@hmenager
Copy link
Member Author

fixed, thanks peter!

@pgrosu
Copy link

pgrosu commented May 28, 2015

Thanks Peter :)

mr-c added a commit that referenced this issue Sep 6, 2016
make tests installable and runnable after install
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

No branches or pull requests

3 participants