[bug #41758] VMS Make incorrectly reports archives support present.

2014-10-19 Thread Paul D. Smith
Update of bug #41758 (project make): Status:None = Fixed Open/Closed:Open = Closed Fixed Release:None = SCM

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-06-21 Thread John E. Malmberg
On 6/20/2014 8:53 AM, h.becker wrote: On 06/14/2014 07:44 PM, John E. Malmberg wrote: globalvalue is visible in /STANDARD=VAXC and /STANDARD=RELAXED which is the default. Is that what you refer to as 'ANSI compatible VMS extension'? The default gives at least fair diagnostics for VAXC

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-06-20 Thread h.becker
On 06/14/2014 07:44 PM, John E. Malmberg wrote: In arscan.c, I suggest to use LBR$_HDRTRUNC instead of LIB_W_HDRTRUNC. A globalvalue unsigned int LBR$_HDRTRUNC; replaces the #define LIB_W_HDRTRUNC 2525184 and the check changes to if ((status != LBR$_HDRTRUNC) ... Globalvalue is the right

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-06-20 Thread h.becker
On 06/14/2014 07:49 PM, John E. Malmberg wrote: Agreed, on VMS the module name space is separate from the filesystem. As pointed out before, GNU make does not have a way to make that distinction. So GNU make can only use case sensitive module name comparisons on VMS with ODS-5 support.

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-06-14 Thread John E. Malmberg
On 6/10/2014 3:24 PM, h.becker wrote: On 06/06/2014 05:15 AM, John Malmberg wrote: lbr$set_module, the documentation says: snip So the size of struct mhddef may not be enough and returning more than that is not an error. Allocating LBR$C_MAXHDRSIZ bytes for the buffer is probably the best

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-06-14 Thread John E. Malmberg
On 6/6/2014 9:39 AM, h.becker wrote: On 06/06/2014 05:28 AM, John E. Malmberg wrote: On 5/31/2014 7:45 AM, h.becker wrote: On 05/15/2014 05:51 AM, John Malmberg wrote: While I have not checked on VAX, on both Alpha and Itanium the module name can have both upper and lower case in it. This

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-06-10 Thread h.becker
On 06/06/2014 05:15 AM, John Malmberg wrote: Follow-up Comment #5, bug #41758 (project make): Previous patch was not tested on VAX. When tested on VAX this exposed a bug in the lbr$set_module() that needed to be worked around. Small clean up of comments and code based on feedback from

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-06-06 Thread h.becker
On 06/06/2014 05:28 AM, John E. Malmberg wrote: On 5/31/2014 7:45 AM, h.becker wrote: On 05/15/2014 05:51 AM, John Malmberg wrote: While I have not checked on VAX, on both Alpha and Itanium the module name can have both upper and lower case in it. This will mainly show up when the source

[bug #41758] VMS Make incorrectly reports archives support present.

2014-06-05 Thread John Malmberg
Follow-up Comment #5, bug #41758 (project make): Previous patch was not tested on VAX. When tested on VAX this exposed a bug in the lbr$set_module() that needed to be worked around. Small clean up of comments and code based on feedback from Hartmut Becker. Copyright assignment is now on file

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-06-05 Thread John E. Malmberg
On 5/31/2014 7:45 AM, h.becker wrote: On 05/15/2014 05:51 AM, John Malmberg wrote: Follow-up Comment #4, bug #41758 (project make): One more comment on (obecjt) module names in a VMS (obejct) library. At least for I64 there can be lowercase module names in an object library. I didn't check on

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-06-02 Thread h.becker
On 06/02/2014 06:22 AM, John E. Malmberg wrote: On 5/31/2014 7:21 AM, h.becker wrote: On 05/15/2014 05:51 AM, John Malmberg wrote: Follow-up Comment #4, bug #41758 (project make): This fixes 8 failing tests in the features/archives test. Wildcard access now works correctly. In addition the

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-06-02 Thread h.becker
On 06/02/2014 06:22 AM, John E. Malmberg wrote: I have not traced down why the perl touch method is not working on VMS object files used for the librarian tests. All I know right now is that it does not work for me. Since I have a workaround, I am not investigating it. As far as I can see,

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-06-01 Thread John E. Malmberg
On 5/31/2014 7:21 AM, h.becker wrote: On 05/15/2014 05:51 AM, John Malmberg wrote: Follow-up Comment #4, bug #41758 (project make): I admit I never looked at all the archive/library code before. So I may need some education. The patch works for my simple test but that worked with the

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-05-31 Thread h.becker
On 05/15/2014 05:51 AM, John Malmberg wrote: Follow-up Comment #4, bug #41758 (project make): I admit I never looked at all the archive/library code before. So I may need some education. The patch works for my simple test but that worked with the unpatched version as well. So what's really

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-05-31 Thread h.becker
On 05/15/2014 05:51 AM, John Malmberg wrote: Follow-up Comment #4, bug #41758 (project make): One more comment on (obecjt) module names in a VMS (obejct) library. At least for I64 there can be lowercase module names in an object library. I didn't check on Alpha. On VAX I couldn't convince the C

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-05-14 Thread John E. Malmberg
On 5/3/2014 4:30 AM, h.becker wrote: On 05/02/2014 03:17 PM, John E. Malmberg wrote: Sounds reasonable. I have not used image libraries in user mode code so was not aware that they were also .olb. Hmm, the VMS linker uses imagelib.olb and you probably linked against shareable images from

[bug #41758] VMS Make incorrectly reports archives support present.

2014-05-14 Thread John Malmberg
Follow-up Comment #4, bug #41758 (project make): Patch uploaded: Fix archive support for VMS for wildcard and VPATH. Add VMS library support for all VMS library types. ar.c: struct ar_glob_state: Save suffix for VMS. (ar_glob_match): Use saved suffix on VMS. (ar_glob): Save suffix for

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-05-03 Thread h.becker
On 05/02/2014 03:17 PM, John E. Malmberg wrote: Sounds reasonable. I have not used image libraries in user mode code so was not aware that they were also .olb. Hmm, the VMS linker uses imagelib.olb and you probably linked against shareable images from that library. So you may talk about

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-05-03 Thread h.becker
On 05/02/2014 03:17 PM, John E. Malmberg wrote: Then after the tests are working correctly, to use built-ins where appropriate to speed things up. A one-shell implementation will also help. As I mentioned before, I have code for one-shell. I wouldn't use one-shell to implement the library

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-05-02 Thread h.becker
On 05/02/2014 05:44 AM, John Malmberg wrote: Follow-up Comment #3, bug #41758 (project make): Updated default.c patch with fixed shared image library rule. That's not a complete fix. It still has the wrong target. The suffix of such a library is .olb, not .exe. The suffix is identical to the

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-05-02 Thread John E. Malmberg
On 5/2/2014 5:22 AM, h.becker wrote: On 05/02/2014 05:44 AM, John Malmberg wrote: Follow-up Comment #3, bug #41758 (project make): Updated default.c patch with fixed shared image library rule. That's not a complete fix. It still has the wrong target. The suffix of such a library is .olb, not

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-05-01 Thread h.becker
On 04/30/2014 06:22 AM, John Malmberg wrote: Follow-up Comment #2, bug #41758 (project make): Updated diff for default.c to suppress messages about VMS library creation if not existing. Updated the library creation to create the correct library type for the known library file suffixes for

[bug #41758] VMS Make incorrectly reports archives support present.

2014-05-01 Thread John Malmberg
Follow-up Comment #3, bug #41758 (project make): Updated default.c patch with fixed shared image library rule. Added shared image library to tests/scripts/vms/library. (file #31296, file #31297) ___ Additional Item Attachment: File

[bug #41758] VMS Make incorrectly reports archives support present.

2014-04-29 Thread John Malmberg
Follow-up Comment #2, bug #41758 (project make): Updated diff for default.c to suppress messages about VMS library creation if not existing. Updated the library creation to create the correct library type for the known library file suffixes for VMS/Unix. Updated archives test for VMS library

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-04-28 Thread h.becker
On 04/27/2014 11:50 PM, John E. Malmberg wrote: Help libraries are .hlb, not .hlp. all: h.hlb(h) My bad, sorry for the misleading typo, but I can't get that to work $ copy nla0: h.hlp $ copy nla0: h.h $ mc [.make]make -f- all: h.hlb(h) Exit if f$search(h.hlb) .eqs. then

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-04-28 Thread John E. Malmberg
On 4/28/2014 1:51 PM, h.becker wrote: On 04/27/2014 11:50 PM, John E. Malmberg wrote: Help libraries are .hlb, not .hlp. all: h.hlb(h) My bad, sorry for the misleading typo, but I can't get that to work You need a bit more, as the rule is based on the extension of the source file,

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-04-27 Thread h.becker
On 04/26/2014 12:42 AM, John Malmberg wrote: I have attached a preliminary patch to the default.c module that fixes the library rules as follows: 1. Create the library if it does not exist when inserting a module into the library. Wouldn't it make sense to suppress the check and creation of

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-04-27 Thread John E. Malmberg
On 4/27/2014 10:04 AM, h.becker wrote: On 04/26/2014 12:42 AM, John Malmberg wrote: I have attached a preliminary patch to the default.c module that fixes the library rules as follows: 1. Create the library if it does not exist when inserting a module into the library. Wouldn't it make sense

[bug #41758] VMS Make incorrectly reports archives support present.

2014-04-25 Thread John Malmberg
Follow-up Comment #1, bug #41758 (project make): I have attached a preliminary patch to the default.c module that fixes the library rules as follows: 1. Create the library if it does not exist when inserting a module into the library. 2. Support all the types of library that VMS does. Tests

Re: [bug #41758] VMS Make incorrectly reports archives support present.

2014-03-03 Thread h.becker
On 03/03/2014 01:16 AM, John Malmberg wrote: URL: http://savannah.gnu.org/bugs/?41758 Summary: VMS Make incorrectly reports archives support present. Project: make Submitted by: wb8tyw Submitted on: Mon 03 Mar 2014 12:16:43 AM