Project

General

Profile

Actions

Bug #798

closed

pysetup.sh has syntax error in 2.0 branch

Added by Amber Herold over 14 years ago. Updated over 14 years ago.

Status:
Closed
Priority:
Immediate
Assignee:
Category:
-
Target version:
Start date:
08/03/2010
Due date:
% Done:

0%

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

Description

Jim, can I update pysetup.sh in the branch on line 68 to match that line in the trunk?

if python setup.py $@ &>>$logfile;

has a sytax error at > and would become

if python setup.py $@ >>$logfile 2>&1;

I'll make the change if you don't see a problem with it.

Actions #1

Updated by Jim Pulokas over 14 years ago

  • Status changed from Assigned to In Test
  • Assignee changed from Jim Pulokas to Amber Herold

no problem. The syntax that generates the error is supposedly proper in current versions of bash, but CentOS is using an older bash.

Actions #2

Updated by Amber Herold over 14 years ago

  • Status changed from In Test to In Code Review
  • Assignee changed from Amber Herold to Jim Pulokas

Jim,
I made the change that I mentioned before,
could you please verify that I did not introduce any typos with the revision.
I tested it on my centos installation.

Actions #3

Updated by Jim Pulokas over 14 years ago

  • Status changed from In Code Review to Closed

looks good, and tested ok on fedora too

Actions

Also available in: Atom PDF