Review Space Stripping...

[tesneddon]

Do some testing of MMS to determine where (and if) we should be stripping back whitespace as much. MMK currently does. There seems to be a couple places where this may create some issues.

[craigberry]

MMS currently supports spaces in dependencies by quoting them, but MMK doesn't. Here's an example:

$ type spaces.mms
targ1 : "targ 2"
@ write sys$output "targ1 depends on "$(MMS$SOURCE)""

"targ 2" :
@ write sys$output "targ 2 is "$(MMS$TARGET)""
$ mms/ident
%MMS-I-IDENT, MMS V3.9-01 © Copyright 2012 Hewlett-Packard Development Company, L.P.
$ mms/log/descrip=spaces.mms
%MMS-I-DRVOUTFIL, Using output file SYS$OUTPUT:.;
%MMS-I-DRVDEPFIL, Using description file D0:[craig]spaces.mms;4
%MMS-I-DRVNOFMSSUP, MMS is installed without support for FMS.
%MMS-I-DRVSUBCLI, Using DCL for the subprocess CLI.
%MMS-I-DRVQUALIF, Using non-defaulted qualifiers /Log
%MMS-I-GWKBEGWLK, Starting the build at target targ1
%MMS-I-GMTIMFND, Time for "targ 2" is 17-NOV-1858 00:00:00.00
%MMS-I-GWKWILLEX, MMS will try executing action line to update target "targ 2"
%MMS-I-EXEPROCID, PID of created subprocess is %X0003AA66.

targ 2 is "targ 2"
%MMS-I-GWKEXESTS, Status of executed command is %X00010001.
-RMS-S-NORMAL, normal successful completion
%MMS-I-GWKUPDONE, Completed update for target "targ 2"
%MMS-I-GWKOLDNOD, Target targ1 is older than "targ 2"
%MMS-I-GWKWILLEX, MMS will try executing action line to update target targ1

targ1 depends on "targ 2"
%MMS-I-GWKEXESTS, Status of executed command is %X00010001.
-RMS-S-NORMAL, normal successful completion
%MMS-I-GWKUPDONE, Completed update for target targ1
$ mmk/ident
%MMK-I-IDENT, this is the MMK Make Utility V5.0
-MMK-I-COPYRIGHT, Copyright (c) 2008, Matthew Madison.
Copyright (c) 2012, Endless Software Solutions.
See LICENSE.TXT in distribution kit for license information.
$ mmk/log/descrip=spaces.mms
%MMK-I-OPENDESC, reading description file D0:[craig]spaces.mms;4
%MMK-I-CHECKUPD, checking need to update target TARG1
%MMK-I-TRGLKUF, lookup for target TARG1 failed - update required
%MMK-I-CHECKUPD, checking need to update target "TARG
%MMK-I-TRGLKUF, lookup for target "TARG failed - update required
%MMK-I-PERFUPD, performing actions to update "TARG
targ 2 is "TARG"
%MMK-I-TRGNUPD, target TARG1 is out of date - update required
%MMK-I-CHECKUPD, checking need to update target 2"
%MMK-I-TRGLKUF, lookup for target 2" failed - update required
%MMK-I-PERFUPD, performing actions to update 2"
targ 2 is 2"
%MMK-I-TRGNUPD, target TARG1 is out of date - update required
%MMK-I-PERFUPD, performing actions to update TARG1
targ1 depends on "TARG"

[tesneddon]

Thanks for the report, @craigberry . I will look into adding quoted string support to action targets and sources.