Angel Pons has posted comments on this change. ( https://review.coreboot.org/28953 )
Change subject: treewide: use /usr/bin/env where appropriate ......................................................................
Patch Set 1:
(3 comments)
Patch Set 1:
(3 comments)
Thank you for review! :-)
Will change `/usr/bin/env python` to `/usr/bin/env python2` where appropriate.
Nice, thank you too :)
https://review.coreboot.org/#/c/28953/1/util/amdtools/k8-compare-pci-space.p... File util/amdtools/k8-compare-pci-space.pl:
https://review.coreboot.org/#/c/28953/1/util/amdtools/k8-compare-pci-space.p... PS1, Line 5: use warnings;
This has the same effect as `-w` flag. […]
Oh, I see, thank you.
https://review.coreboot.org/#/c/28953/1/util/chromeos/extract_blobs.sh File util/chromeos/extract_blobs.sh:
https://review.coreboot.org/#/c/28953/1/util/chromeos/extract_blobs.sh@17 PS1, Line 17: set -x
This flag makes shell print every command invocation to stderr. It was in shebang.
Right, I missed that. Thanks.
https://review.coreboot.org/#/c/28953/1/util/ipqheader/mbn_tools.py File util/ipqheader/mbn_tools.py:
https://review.coreboot.org/#/c/28953/1/util/ipqheader/mbn_tools.py@1 PS1, Line 1: #!/usr/bin/env python
This probably would have been the case with previous shebang as well though. […]
Yes, it definitely was. Thanks.