Project

General

Profile

Actions

Bug #911

closed

webcaller.py

Added by Gabriel Lander about 14 years ago. Updated about 14 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Amber Herold
Category:
Python scripting
Target version:
Start date:
10/12/2010
Due date:
% Done:

0%

Estimated time:
Affected Version:
Appion/Leginon 2.1.0
Show in known bugs:
No
Workaround:

Description

I've run into problems with webcaller.py not being able to make subdirectories.
if "/path/glander/align" doesn't exist, then "/path/glander/align/maxlike1" can't be created and webcaller.py fails.
I changed webcaller.py to use "mkdir -p" from a subprocess call instead of "os.mkdir", since this will create necessary subdirectories

Actions #1

Updated by Amber Herold about 14 years ago

r14870

Actions #2

Updated by Amber Herold about 14 years ago

  • Status changed from Assigned to In Code Review
  • Assignee changed from Amber Herold to Gabriel Lander
  • Target version set to Appion/Leginon 2.1.0

Nice catch Gabe. We've been hoping to change all our shell commands to use the absolute path to avoid any problems that could occur if the command has been modified on the specific machine. Christopher and I found that os actually has a function that should create the needed subdirectories, so I have replaced the shell command with it.

Take a look at it and let me know what you think. I will try to merge this into the 2.1 branch before release as well...seems this could be a very annoying problem.

Actions #3

Updated by Gabriel Lander about 14 years ago

  • Status changed from In Code Review to In Test

aha! This is great - I didn't know that os had this function. And we should indeed always use the absolute path in shell commands. I tested & this works fine at my end.

Actions #4

Updated by Amber Herold about 14 years ago

  • Status changed from In Test to Merge
  • Assignee changed from Gabriel Lander to Amber Herold
Actions #5

Updated by Amber Herold about 14 years ago

Merged to 2.1 in r14903

Actions #6

Updated by Amber Herold about 14 years ago

  • Status changed from Merge to Closed
Actions

Also available in: Atom PDF