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

PSI easyblock is broken, install_step fails #272

Closed
boegel opened this issue Oct 4, 2013 · 6 comments
Closed

PSI easyblock is broken, install_step fails #272

boegel opened this issue Oct 4, 2013 · 6 comments

Comments

@boegel
Copy link
Member

boegel commented Oct 4, 2013

Despite efforts for last-minute fixes, the PSI easyblock is broken.

The installation fails because os.path.basename yields an empty string when called on a path that ends with a /, i.e. os.path.basename("/path/to/psi/") returns "".

Because of this, the PSI builds fails as follows:

== 2013-10-04 15:00:38,528 main.EB_PSI ERROR EasyBuild crashed with an error (at easybuild/easyblocks/p/psi.py:133 in install_step): Failed to copy obj and unpacked sources to install dir: [Errno 17] File exists: '/path/to/software/PSI/4.0b4-ictce-4.1.13-mt/'

This problem will be fixed in EasyBuild v1.9, together with adding support for the new PSI version 4.0b5 (see easybuilders/easybuild-easyconfigs#434).

@wpoely86
Copy link
Member

wpoely86 commented Oct 4, 2013

Fixed in #270

@boegel
Copy link
Member Author

boegel commented Oct 4, 2013

Actually, no, it's not fixed.

The problem is the following line in install_step:

for subdir in ['obj', os.path.basename(self.cfg['start_dir'])]:

The second element of that list will be "" (empty string), which makes the installation fail...

Can you fixed that in #270, and retest a PSI build to make 100% sure?

@wpoely86
Copy link
Member

wpoely86 commented Oct 4, 2013

I will fix it but retest a PSI build will take a couple of hours...

@boegel
Copy link
Member Author

boegel commented Oct 4, 2013

I realize that, but EasyBuild v1.8.0 is out already anway (with the bug, elas :().

@wpoely86
Copy link
Member

wpoely86 commented Oct 6, 2013

Fully fixed in #270.

@boegel
Copy link
Member Author

boegel commented Oct 10, 2013

follow-up via #270

@boegel boegel closed this as completed Oct 10, 2013
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