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

manifest_normalise.py script break #389

Open
raecasey opened this issue Aug 6, 2020 · 2 comments
Open

manifest_normalise.py script break #389

raecasey opened this issue Aug 6, 2020 · 2 comments

Comments

@raecasey
Copy link
Contributor

raecasey commented Aug 6, 2020

manifest_normalise.py breaks on macmini.
works fine on es2 and yujings laptop.
maybe it's an issue with macmini rather than anything else.
if it is workign fine on es2 and laptops then we will be ok

terminal output:
Last login: Thu Aug 6 11:50:47 on ttys002
macminis-mini:~ macmini$ manifest_normalise.py -h
/Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 8:
Normalises manifests so that they adhere to ifiscripts manifest.
This includes removing lines with hashes.
Removing lines with just whitespace.
Creating a new sidecar called _modified_manifest.md5
The goal is to allow batchaccession to run on the new manifests.
: command not found
/Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 9: import: command not found
/Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 10: import: command not found
/Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 11: import: command not found
/Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 12: import: command not found
/Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 14: syntax error near unexpected token format='%(asctime)s - %(message)s',' /Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 14: logging.basicConfig(format='%(asctime)s - %(message)s', level=logging.DEBUG)'
macminis-mini:~ macmini$

@kieranjol
Copy link
Owner

Hey - so this was fixed locally on es2 but a problem happened when trying to push to github - so this fix never made its way up there. working on the fix now.

@kieranjol
Copy link
Owner

I was able to push the fixes i made a few days ago so an update.py will sort this out now hopefully

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

2 participants