Merge lp:~nuclearbob/utah/posix-shell-scripts into lp:utah
Proposed by
Max Brustkern
Status: | Merged |
---|---|
Merged at revision: | 837 |
Proposed branch: | lp:~nuclearbob/utah/posix-shell-scripts |
Merge into: | lp:utah |
Diff against target: |
153 lines (+32/-20) 7 files modified
conf/utah/dns (+1/-1) debian/changelog (+6/-0) debian/utah-client.postinst (+6/-4) debian/utah.postinst (+8/-6) debian/utah.postrm (+6/-4) examples/utah-user-setup.sh (+4/-4) examples/utah_logs.sh (+1/-1) |
To merge this branch: | bzr merge lp:~nuclearbob/utah/posix-shell-scripts |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Max Brustkern (community) | Needs Resubmitting | ||
Javier Collado (community) | Approve | ||
Review via email:
|
Description of the change
After Oli's email, I thought it would be wise to remove bashisms from our scripts and make sure they're using sh in the bang line. We certainly don't have to stick to the indefinitely if we need bash, but all these changes were easy and should make us more in line with standards.
To post a comment you must log in.
On 15 March 2013 17:58, Max Brustkern <email address hidden> wrote: nprint( utah.config. vmpath) " | python) nprint( utah.config. vmpath) " | python)
> Max Brustkern has proposed merging lp:~nuclearbob/utah/posix-shell-scripts into lp:utah.
> echo "$@"
> @@ -28,7 +28,7 @@
> export AUTO=""
> fi
>
> -VMDIR=$(echo -e "import utah.config\
> +VMDIR=$(printf "import utah.config\
Although this is POSIX compliant, it's not "pythonic". What about this:
VMDIR=$(python -c "import utah.config; print(utah. config. vmpath) ")
Despite python being whitespace significant language ';' still works
to bundles commands on the same line.
Unless I am missing something and python in the above pipe is something else...