Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make qualifiers camel-case in MMSQUALIFIERS #81

Open
tesneddon opened this issue Jun 3, 2014 · 2 comments
Open

Make qualifiers camel-case in MMSQUALIFIERS #81

tesneddon opened this issue Jun 3, 2014 · 2 comments
Assignees

Comments

@tesneddon
Copy link
Member

The builtin variable MMSQUALIFIERS is automatically generated from the incoming command line. However, MMS uses camel-case qualifiers, where as MMK uses all upper case.

For example:

MMK: /SKIP_INTERMEDIATES
MMS: /Skip_Intermediates

This isn't a huge priority. However, it should be determined if compatibility should be retained for older versions of MMK.

@tesneddon tesneddon added this to the V5.1 Release milestone Jun 3, 2014
@tesneddon tesneddon self-assigned this Jun 3, 2014
@hb--
Copy link

hb-- commented Jun 3, 2014

I admit, I may not be current with the version of mmk. So, for whatever it is worth, IF you want to make mmk qualifier compatible (and /Extended_syntax doesn't look like a camel, or?):

$ mms/id
%MMS-I-IDENT, MMS V3.8 � Copyright 2007 Hewlett-Packard Development Company, L.P.
$ mms/descr=x/forc/extend/act/changed=*/cms/skip/sca/ver/list/igno

all : anything and more
        -@ write sys$output "MMS=$(MMS)
        -@ write sys$output "MMSQUALIFIERS=$(MMSQUALIFIERS)
        -@ write sys$output "MMSTARGETS=$(MMSTARGETS)
        -@ write sys$output "MMSDESCRIPTION_FILE=$(MMSDESCRIPTION_FILE)
        -@ write sys$output "MMSARCH_NAME=$(MMSARCH_NAME)
        -@ write sys$output "MMSALPHA=$(MMSALPHA)
        -@ write sys$output "MMSIA64=$(MMSIA64)
        -@ write sys$output "MMSVAX=$(MMSVAX)
        -@ write sys$output "MMS$ARCH_NAME=$(MMS$ARCH_NAME)
        -@ write sys$output "MMS$CHANGED_LIST=$(MMS$CHANGED_LIST)
        -@ write sys$output "MMS$LIB_ELEMENT=$(MMS$LIB_ELEMENT)
        -@ write sys$output "MMS$TARGET=$(MMS$TARGET)
        -@ write sys$output "MMS$TARGET_NAME=$(MMS$TARGET_NAME)
        -@ write sys$output "MMS$TARGET_SPEC=$(MMS$TARGET_SPEC)
        -@ write sys$output "MMS$SOURCE=$(MMS$SOURCE)
        -@ write sys$output "MMS$SOURCE_LIST=$(MMS$SOURCE_LIST)
        -@ write sys$output "MMS$SOURCE_NAME=$(MMS$SOURCE_NAME)
        -@ write sys$output "MMS$STATUS=$(MMS$STATUS)
        -@ write sys$output "MMS$SEVEREST_STATUS=$(MMS$SEVEREST_STATUS)
MMS=MMS
MMSQUALIFIERS=/Skip_Intermediate/Extended_syntax/Cms/Force/Ignore=Warning/Sca_Library=SCA$LIBRARY:/Changed=(*)/List=SYS$OUTPUT:
MMSTARGETS=
MMSDESCRIPTION_FILE=LDA1:[USERS.DEMO]X.MMS;1
MMSARCH_NAME=IA64
MMSALPHA=
MMSIA64=IA64
MMSVAX=
MMS$ARCH_NAME=IA64
MMS$CHANGED_LIST=
MMS$LIB_ELEMENT=
MMS$TARGET=all
MMS$TARGET_NAME=all
MMS$TARGET_SPEC=all
MMS$SOURCE=anything
MMS$SOURCE_LIST=anything, and, more
MMS$SOURCE_NAME=
MMS$STATUS=%X00010001
MMS$SEVEREST_STATUS=%X00010001
$ 
$ mmk/id
%MMK-I-IDENT, this is the MMK Make Utility V5.0
-MMK-I-COPYRIGHT, Copyright (c) 2008, Matthew Madison.
    Copyright (c) 2013, Endless Software Solutions.
  See LICENSE.TXT in distribution kit for license information.
$   
$ mmk/descr=x/forc/extend/act/changed=*/cms/skip/sca/ver/list/igno
%DCL-W-VALREQ, missing qualifier or keyword value - supply all required values
 \EXTEND\
$ 
$ mmk/descr=x/forc/extend=mms/act/changed=*/cms/skip/sca/ver/list/igno
%DCL-W-IVQUAL, unrecognized qualifier - check validity, spelling, and placement
 \CHANGED\
$ 
$ mmk/descr=x/forc/extend=mms/act/cms/skip/sca/ver/list/igno
%DCL-W-IVQUAL, unrecognized qualifier - check validity, spelling, and placement
 \SCA\
$ 
$ mmk/descr=x/forc/extend=mms/act/cms/skip/ver/list/igno
%DCL-W-IVQUAL, unrecognized qualifier - check validity, spelling, and placement
 \LIST\
$ mmk/descr=x/forc/extend=mms/act/cms/skip/ver/igno
%MMK-F-NOOPNDSC, could not open description file x
-RMS-E-FNF, file not found
$ 
$ mmk/descr=x.mms/forc/extend=mms/act/cms/skip/ver/igno
MMS=MMK
MMSQUALIFIERS=/DESCRIPTION=x.mms/VERIFY/SKIP_INTERMEDIATES/FORCE/EXTENDED_SYNTAX=(MMS_SYNTAX)/CMS/IGNORE
MMSTARGETS=
MMSDESCRIPTION_FILE=EISNER$DRA3:[DECUSERVE_USER.BECKER_H]X.MMS;7
MMSARCH_NAME=Alpha
MMSALPHA=1
MMSIA64=
MMSVAX=
MMS$ARCH_NAME=Alpha
MMS$CHANGED_LIST=ANYTHING,AND,MORE
MMS$LIB_ELEMENT=
MMS$TARGET=ALL
MMS$TARGET_NAME=ALL
MMS$TARGET_SPEC=
MMS$SOURCE=ANYTHING
MMS$SOURCE_LIST=ANYTHING,AND,MORE
MMS$SOURCE_NAME=ANYTHING
MMS$STATUS=%X00010001
MMS$SEVEREST_STATUS=
$

And in a lame attempt to see the lowercase targets, etc.:

$ mmk/descr=x.mms/extend=case
%SYSTEM-F-ACCVIO, access violation, reason mask=00, virtual address=FFFFFFFFFFEE
ADCF, PC=00000000000D798C, PS=0000001B

  Improperly handled condition, image exit forced.
    Signal arguments:   Number = 0000000000000005
                        Name   = 000000000000000C
                                 0000000000000000
                                 FFFFFFFFFFEEADCF
                                 00000000000D798C
                                 000000000000001B

    Register dump:
    R0  = 0000000000000001  R1  = 000000000000002C  R2  = 0000000000023260
    R3  = FFFFFFFFFFEEADCF  R4  = 0000000000000000  R5  = 00000000000F7174
    R6  = 0000000000000008  R7  = 0000000000115240  R8  = 0000000000026F2C
    R9  = 0000000000000001  R10 = 000000000000FFFF  R11 = 0000000000000800
    R12 = FFFFFFFFFFFFC5A4  R13 = 00000000000234D0  R14 = 0000000000000000
    R15 = 000000000000001C  R16 = 0000000000115238  R17 = 756F247379732000
    R18 = 000000000000002C  R19 = 0000000000000001  R20 = FFFFFFFFFFFFFFD7
    R21 = 0000000000115211  R22 = 0000000000023260  R23 = 00000000000D875C
    R24 = 756F24737973202C  R25 = 0000000000000002  R26 = FFFFFFFF80AF6C90
    R27 = 000000007BEDE420  R28 = 00000000000D7900  R29 = 000000007AD2EC90
    SP  = 000000007AD2EC90  PC  = 00000000000D798C  PS  = 100000000000001B
$ 

@tesneddon
Copy link
Member Author

Thank you for the input. Some of those issues have been resolved and merged into the master branch. /EXTENDED_SYNTAX has been corrected. Also, the /DESCRIPTION value has a default of ".MMS". It is no longer necessary to include the file type.

I will look at the other comments and see if they apply. At this point there are no plans to make MMK qualifier compatible, although it is a good idea. I am also currently investigating the whole lower/upper/mixed case issue.

@tesneddon tesneddon removed this from the V5.1 Release milestone Aug 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants