python subprocess: „su - -c” als Variable

Peter N. schrieb:

Als Beispiel mal ein Verzeichnis als root erstellen, was so auch funktioniert: [...]

Warum nicht "os.mkdir('/tmp/datda')"?

Und das funktioniert auch mit: os.mkdir('/abc') Wohlgemerkt: ich starte das Skript nicht als root!

Das Konzept an sich ist etwas fragwürdig, wenn ein Script Root braucht, startet man es halt als root, statt darin mit su herumzumachen, aber was solls...

Du könntest dir eine Funktion machen, in der du das Escaping machst, usw:

1

#!python3

2

def shell_escape(cmd):

3

    return ' '.join("'"+x.replace("'","'\\''")+"'" for x in cmd)

4

def as_root(cmd):

5

    return ['su', '-', '-c', shell_escape(cmd)]

Oder du könntest "sudo -- " stat "su - -c" nehmen, da musst du nur die Listen verketten, und musst das nicht mehr extra escapen.

shell=True sollte man ja vermeiden, damit man nicht je nach Shell falsch escapt, aber su -c führt das zeug auch in einer Shell aus, damit ist man wieder am Anfang.

1

#!python3

2

def as_root(cmd):

3

    return ['sudo', '--', *cmd]

Am besten wäre es aber immer, gar nicht erst root zu benötigen.

Peter N. schrieb:

Kann ich "su", "-", "-c" irgendwie funktionsfähig in eine Variable packen und falls ja, dann wie?

Klar, einfach dran denken, das die eckigen Klammern eine Liste bedeuten. (Und auch run eine liste erwartet)

1

root = ["su", "-", "-c"] 

2

# oder besser zu lessen:

3

root = "su - -c".split()

4

5

Pfad = "/tmp/.abc/"

6

PfadTmp = "def"

7

8

# und dann liste neinfach "addieren"

9

subprocess.run(root + ["mkdir -p " + Pfad+  PfadTmp])

Aha, so geht das also! Vielen Dank Andreas, funktioniert perfekt:

1

Root = ["su", "-", "-c"]; Pfad = "/tmp/.abc/"; PfadTmp = "def"

2

subprocess.run(Root + ['mkdir -p ' + Pfad + PfadTmp])

Für die paar root Befehle die ich brauche möchte ich nicht das komplette Skript als root laufen lassen, da könnten verschiedene Änderungen schon verheerende Folgen haben... gelöst, größten Dank an alle Beteiligten!

Wenn das wirklich in /tmp angelegt werden soll, brauchst Du doch keinen root

Daniel A. schrieb:

1

> #!python3

2

> def shell_escape(cmd):

3

>     return ' '.join("'"+x.replace("'","'\\''")+"'" for x in cmd)

4

>

OMG, Dein Ernst? Obwohl es shlex gibt?