Project

General

Profile

Feature #3790

cp -f listfile too sensitive to white space

Added by Andrei Gaponenko almost 7 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Marc Mengel
Target version:
v1_2_5
Start date:
04/30/2013
Due date:
% Done:

100%

Estimated time:
2.00 h
Spent time:
0.10 h
Duration:

Description

Hi,

ifdh cp -f listfile

works when listfile lines are exactly

<src><space><dst>

With extra white space

<src><space><space><space><dst>

it fails with

ERROR: too many url strings specified

Syntax: globus-url-copy [-help | -usage] [-version[s]] [-vb] [-dbg] [-b | -a]
                        [-q] [-r] [-rst] [-f <filename>]
                        [-s <subject>] [-ds <subject>] [-ss <subject>]
                        [-tcp-bs <size>] [-bs <size>] [-p <parallelism>]
                        [-notpt] [-nodcau] [-dcsafe | -dcpriv]
                        <sourceURL> <destURL>

Use -help to display full usage
LOCK - Tue Apr 30 21:46:28 UTC 2013 freed /grid/data/mu2e/LOCK/LOCKS/20130430.21:46:28.0.fnpc3029.18894.mu2eana.gandr
LOCK - Tue Apr 30 21:46:28 UTC 2013 lock  /grid/data/mu2e/LOCK/LOCKS/20130430.21:46:28.0.fnpc3029.18894.mu2eana.gandr

Please allow an arbitrary amount of spaces and tabs as the field separator.

Andrei

History

#1 Updated by Marc Mengel over 6 years ago

  • Status changed from New to Assigned
  • Assignee set to Marc Mengel
  • Target version set to v1_2_5
  • % Done changed from 0 to 10
  • Estimated time set to 2.00 h

This looks to be in ifdh_cp.cc, function expandfile( std::string fname )
around line 235, we explicitly find the (assumedly only) space and split
the arguments out. We should look for spaces or tabs and find the bounds
of the whitespace.

#2 Updated by Marc Mengel over 6 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 10 to 100

#3 Updated by Marc Mengel over 6 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF