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

case sensitivity and "action did not update target" (object library/element) #72

Open
hb-- opened this issue Aug 13, 2013 · 3 comments
Open
Assignees
Milestone

Comments

@hb--
Copy link

hb-- commented Aug 13, 2013

On an ODS5 disk, with "$ set proc/parse=extended/case=blind" (and hacked mmk.cld):

$ mmk/id
%MMK-I-IDENT, this is the MadGoat Make Utility V4.1
-MMK-I-COPYRIGHT, Copyright (c) 2008, Matthew Madison.  See LICENSE.TXT in distribution kit for license information.
$
$ mmk "SYS$DISK:[.BIN-AXP]SQLITE3(alter)"
LIBRARY/REPLACE SYS$DISK:[.BIN-AXP]SQLITE3.OLB SYS$DISK:[.BIN-AXP]ALTER.OBJ
%MMK-I-ACTNOUPD, action did not update target SYS$DISK:[.BIN-AXP]SQLITE3.OLB(ALTER=SYS$DISK:[.BIN-AXP]ALTER.OBJ)
$ libr/list SYS$DISK:[.BIN-AXP]SQLITE3.OLB
Directory of ALPHA OBJECT library USR_ODS5:[BECKER_H.sqlite3.BIN-AXP]SQLITE3.OLB;1 on 13-AUG-2013 12:18:55
Creation date:  13-AUG-2013 12:18:22      Creator:  Librarian A09-30
Revision date:  13-AUG-2013 12:18:25      Library format:   3.0
Number of modules:      1                 Max. key length:  128
Other entries:          5                 Preallocated index blocks:    213
Recoverable deleted blocks:       74      Total index blocks used:        2
Max. Number history records:      20      Library history records:        2

alter
$ 

where alter.c was compiled with cc/name=as_is, which creates a module named after the file name in lowercase letters. However, as can be seen in the MMK-I-ACTNOUPD message, mmk uses a module name in uppercase letters.

@ghost ghost assigned tesneddon Sep 4, 2013
@tesneddon
Copy link
Member

I have looked at the code and I can see what is happening. I think that this will require some changes to allow MMK to respond to the setting of /CASE_LOOKUP. I think it is likely best to support retaining case of the module name at all times and using a new routine (dcl_strcmp) that compares correctly, based on DCL process configuration. A fix for this is being planned.

@tesneddon
Copy link
Member

Also, what is "hacked" about your MMK_CLD.CLD?

@hb--
Copy link
Author

hb-- commented Sep 4, 2013

************
File DISK$USER:[HB.MMK]MMK_CLD.CLD;2
   38       PARAMETER P1,LABEL=TARGET, VALUE(LIST,TYPE=$FILE)
   39       QUALIFIER DESCRIPTION,VALUE(TYPE=$FILE,REQUIRED),NONNEGATABLE
******
File DISK$USER:[HB.MMK]MMK_CLD.CLD;1
   38       PARAMETER P1,LABEL=TARGET, VALUE(LIST)
   39       QUALIFIER DESCRIPTION,VALUE(TYPE=$FILE,REQUIRED),NONNEGATABLE
************

Number of difference sections found: 1
Number of difference records found: 1

DIFFERENCES /IGNORE=()/MERGED=1
    DISK$USER:[HB.MMK]MMK_CLD.CLD;2-
    DISK$USER:[HB.MMK]MMK_CLD.CLD;1

@tesneddon tesneddon modified the milestones: V5.1 Release, MMK V5.2 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