search

valeriupredoi / bgcval2

Package for BGCVal v2.0
3 stars 0 forks source link

Resolving the on-going issues with Jasmin permissions. #115

Closed ledm closed 8 months ago

ledm commented 8 months ago

Resolving the on-going issues with Jasmin permissions.

Continues from the merged #114.

valeriupredoi commented 8 months ago

am also deleting the branch, hopefully no more need for it, if not, jyst create a new one, bud - this has been a tricky one!

ledm commented 8 months ago

Yeah, hopefully we can put this one to bed now. But I suspect that it's not over yet.

ledm commented 8 months ago

I was right. Something else is broken here.

ledm commented 8 months ago

We never considered looking at a new job! We're checking permissions of a file that doesn't exist yet and it fails. I'll write a fix now.

valeriupredoi commented 8 months ago

isn't there a check on outFolder's existence somewhere up the stack? I could swear I saw one

ledm commented 8 months ago

No check, I've added this fix to the PR in #116.