Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Python Subprocess Error in using "cp"

I was trying to use subprocess calls to perform a copy operation (code below):

import subprocess
pr1 = subprocess.call(['cp','-r','./testdir1/*','./testdir2/'], shell = True)

and I got an error saying:

cp: missing file operand

Try `cp --help' for more information.

When I try with shell=False , I get

cp: cannot stat `./testdir1/*': No such file or directory

How do I get around this problem?

I'm using RedHat Linux GNOME Deskop version 2.16.0 and bash shell and Python 2.6

P.S. I read the question posted in Problems with issuing cp command with Popen in Python, and it suggested using shell = True option, which is not working for me as I mentioned :(

like image 394
Tapajit Dey Avatar asked Jul 26 '13 12:07

Tapajit Dey


3 Answers

When using shell=True, pass a string, not a list to subprocess.call:

subprocess.call('cp -r ./testdir1/* ./testdir2/', shell=True)

The docs say:

On Unix with shell=True, the shell defaults to /bin/sh. If args is a string, the string specifies the command to execute through the shell. This means that the string must be formatted exactly as it would be when typed at the shell prompt. This includes, for example, quoting or backslash escaping filenames with spaces in them. If args is a sequence, the first item specifies the command string, and any additional items will be treated as additional arguments to the shell itself.

So (on Unix), when a list is passed to subprocess.Popen (or subprocess.call), the first element of the list is interpreted as the command, all the other elements in the list are interpreted as arguments for the shell. Since in your case you do not need to pass arguments to the shell, you can just pass a string as the first argument.

like image 128
unutbu Avatar answered Sep 17 '22 16:09

unutbu


This is an old thread now, but I was just having the same problem.

The problem you were having with this call:

subprocess.call(['cp','-r','./testdir1/*','./testdir2/'], shell = False)

was that each of the parameters after the first one are quoted. So to the shell sees the command like this:

cp '-r' './testdir1/*' './testdir2/'

The problem with that is the wildcard character (*). The filesystem looks for a file with the literal name '*' in the testdir1 directory, which of course, is not there.

The solution is to make the call like the selected answer using the shell = True option and none of the parameters quoted.

like image 29
MorganGalpin Avatar answered Sep 20 '22 16:09

MorganGalpin


I know that the option of shell=True may be tempting but it's always inadvisable due to security issues. Instead, you can use a combination of the subprocess and glob modules.

For Python 3.5 or higher:

import subprocess
import glob

subprocess.run(['cp', '-r'] + glob.glob('./testdir1/*') + ['./testdir2/'])

For Python 3.4 or lower:

import subprocess
import glob

subprocess.call(['cp', '-r'] + glob.glob('./testdir1/*') + ['./testdir2/'])
like image 41
Antonio Serrano Avatar answered Sep 21 '22 16:09

Antonio Serrano