Comment 1 Alexander Kläser 2012-04-19 13:24:30 CEST

Gut wäre es, wenn stdout und stderr durchgereicht werden.

Comment 2 Lukas Walter 2012-04-19 14:52:18 CEST

Wurde entsprechend umgesetzt.

Comment 3 Alexander Kläser 2012-04-20 08:52:34 CEST

(In reply to comment #2)
> Wurde entsprechend umgesetzt.

Super! Ich frage mich, ob shell=True im Aufruf notwendig ist?

Comment 4 Lukas Walter 2012-04-20 12:30:28 CEST

Nicht unbedingt. "shell=True" vernlasst subprocess.Popen den ersten Parameter genau so auszuwerten wie die shell einen String auswerten würde.

Ohne ihn erwartet der Popen-Konstruktor als ersten Parameter eine Liste:
subprocess.Popen(['<absolute path to script>'], ...])

Comment 5 Lukas Walter 2012-04-20 12:33:35 CEST

(In reply to comment #4)
> Nicht unbedingt. "shell=True" vernlasst subprocess.Popen den ersten Parameter
> genau so auszuwerten wie die shell einen String auswerten würde.
> 
> Ohne ihn erwartet der Popen-Konstruktor als ersten Parameter eine Liste:
> subprocess.Popen(['<absolute path to script>'], ...])

Korregiere:
subprocess.Popen(absolute path to script, arg, arg, arg ...], ...)

Comment 6 Lukas Walter 2012-04-20 12:34:07 CEST

subprocess.Popen([absolute path to script, arg, arg, arg ...], ...)

Comment 7 Philipp Hahn 2012-04-23 09:34:02 CEST

(In reply to comment #3)
> (In reply to comment #2)
> > Wurde entsprechend umgesetzt.
> 
> Super! Ich frage mich, ob shell=True im Aufruf notwendig ist?

Für die API-Kompatibilität, da runScripts prinzipiell auch von extern verwendet werden könnte.

Comment 8 Philipp Hahn 2012-06-12 11:20:21 CEST

(In reply to comment #1)
> Gut wäre es, wenn stdout und stderr durchgereicht werden.

Bitte unbedingt die damalige Problematik aus Bug #22517 bei der QA beachten, nicht daß das Problem erneut auftritt. (Vom kurzen überfliegen sollte das hier nicht problematisch sein, da damals stdout und stderr in eine (begrenzte) Pipe umgeleitet wurden; da jetzt die Ausgabe an stdout/stderr durchgereicht wird, sollte das kein Problem sein.)

Comment 9 Andreas Büsching 2012-06-22 13:04:54 CEST

Funktioniert:

root@server:/etc/univention/templates/scripts# ucr commit /etc/ldap/slapd.conf 
Multifile: /etc/ldap/slapd.conf
stdout
stderr
root@server:/etc/univention/templates/scripts# cat etc/ldap/slapd.conf
#!/bin/bash

echo "stdout"
echo "stderr" >&2

ChangeLog-Eintrag fehlt

Comment 10 Lukas Walter 2012-06-22 13:45:25 CEST

Changelogeintrag wurde hinzugefügt.

\item stdout will not be redirected to /dev/null anymore when executing scripts (\ucsBug{26840}).

Comment 11 Andreas Büsching 2012-06-22 15:41:46 CEST

(In reply to comment #10)
> Changelogeintrag wurde hinzugefügt.

Ist drin

Comment 12 Stefan Gohmann 2012-07-20 15:25:01 CEST

UCS 3.0-2 has been released: 
  http://forum.univention.de/viewtopic.php?f=54&t=1905

If this error occurs again, please use "Clone This Bug".