> From: intrigeri <intrig...@boum.org> > Date: Sat, 4 Aug 2012 18:13:37 +0200 > > Hi, > > a...@boum.org wrote (29 Jul 2012 12:31:45 GMT) : > > Implemented in tails git, branch feature/more_whisperback_logs, > > merged into experimental. Please test ! > > Great! > > I'll test it, but probably not before 10 days. > Thats not urgent!
> > + except OSError:
> > + debugging_info += "sudo command not found\n" % debug_command
>
> Why the "% debug_command"?
>
Well because I forgot to delete it...
> > + except subprocess.CalledProcessError:
> > + debugging_info += "debugging command returned an error\n" %
> > debug_command
>
> The "% debug_command" lets me think a %s is missing in the
> format string, isn't it?
>
Same as above, old shit staying around. Corrected in git.
> > +debug_command() {
> > + echo
> > + echo "===== output of command $@ ====="
> > + "$@"
> > +}
>
> I think this won't work once commands that span multiple words are
> used; or am I wrong, that is, do the changes introduced by commit
> 38293d actually work?
Well it seems me so (see the call to `dmidecode -s system-manufacturer`).
--
pgpuQoUz0V0Rx.pgp
Description: PGP signature
_______________________________________________ tails-dev mailing list tails-dev@boum.org https://mailman.boum.org/listinfo/tails-dev
