search

kieranjol / IFIscripts

Detailed documentation is available here: http://ifiscripts.readthedocs.io/en/latest/index.html
http://ifiscripts.readthedocs.io/en/latest/index.html
MIT License
50 stars 34 forks source link

manifest_normalise.py script break #389

Open raecasey opened 4 years ago

raecasey commented 4 years ago

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 commented 4 years ago

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 commented 4 years ago

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