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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃悰: build_combine_archive in utils.py will create multiple manifest.xml files #5

Open
1 task done
luciansmith opened this issue Jan 19, 2023 · 0 comments
Open
1 task done
Labels
bug Something isn't working

Comments

@luciansmith
Copy link
Contributor

Contact Details

No response

Location

Other (please describe below)

Version

Production (Deployed on a .org domain)

What browsers are you seeing the problem on?

No response

What happened?

If you run build_combine_archive on a directory that already has a manifest.xml file, it will create a new manifest.xml file and add that to the same zip file. This makes it impossible to run on run.biosimulations, since when unzipping the omex file, it asks the user what to do when unpacking two files with the same name, and since it's a batch process, it gets no reply and fails instead. See

https://run.biosimulations.org/runs/63c9a9af3e38811aad1597fc#tab=log

What made it happen?

Run build_combine_archive on a directory containing a manifest.xml file.

Relevant log output

inflating: contents/manifest.xml   
replace contents/manifest.xml? [y]es, [n]o, [A]ll, [N]one, [r]ename:  NULL
(EOF or read error, treating as "[N]one" ...)
srun: error: xanadu-51: task 0: Exited with exit code 1

Code of Conduct

  • I agree to follow this project's Code of Conduct
@luciansmith luciansmith added the bug Something isn't working label Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant