launchpad-reviewers team mailing list archive

[Scott Moser <smoser@xxxxxxxxxx>]

The proposal to merge lp:~smoser/maas/vdenv-fix-mawk into lp:maas has been updated.

Description changed to:

vdenv: fix usage of awk if awk is 'mawk' rather than 'gawk'

The issue is that /usr/bin/awk is an alternative. and if it is provided by mawk rather than gawk (as it was in roaksoaks case), then it fails just like this:
  $ mawk '{gsub("*","");} $1 == key { print $2 }' "key=$1:" /dev/null
  mawk: line 1: regular expression compile failed (missing operand)
  *

new syntax works in gawk or mawk or busybox awk.


For more details, see:
https://code.launchpad.net/~smoser/maas/vdenv-fix-mawk/+merge/116067
-- 
https://code.launchpad.net/~smoser/maas/vdenv-fix-mawk/+merge/116067
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~smoser/maas/vdenv-fix-mawk into lp:maas.