bugmake - Bugs: bug #18517, Compilation error in...

 
 

bug #18517: Compilation error in find_directory() in dir.c on Windows platforms

Submitter:  Rafi Einstein <rafi_einstein>
Submitted:  Wed 13 Dec 2006 04:43:17 PM UTC
   
 
Severity:  3 - Normal Item Group:  Build/Install
Status:  Later Privacy:  Public
Assigned to:  None Open/Closed:  Closed
Component Version:  4.0 Operating System:  MS Windows
Fixed Release:  None Triage Status:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 14 Dec 2006 05:57:47 AM UTC, comment #1: 

Hi all.

The make source code is undergoing some relatively major contortions right now as I wrestle it to an ISO C standard behavior and throw out support for K&R C.  Along with this is a change to use the strcache capability I added in the last release, throughout the entire codebase to greatly reduce the amount of memory used by make and hopefully, in some cases at least, increase the efficiency.  The changes you see currently in CVS are minor compared to the ones I have locally in my workspace(s).  The good news is that this work is almost done and as of last weekend my current version passes all the regression tests again!  I'm adding some validation code and checking things with valgrind, etc. but the end appears to be in sight.

The bad news is that since I don't have any Windows systems available to me to test on (ditto for VMS, Amiga, etc.) those ports have definitely "rotted" somewhat.  I fully expect there will be many instances of this sort of problem, where strings were made constant in the POSIX port but now some love has to be applied in the other ports.  I don't think it's really productive to begin this debugging until I've committed all the changes I have locally.

In short, at the moment I will have to simply state that the CVS tree is known not to be stable for non-POSIX operating systems.

Paul D. Smith <psmith>
Group administrator
Wed 13 Dec 2006 04:43:17 PM UTC, original submission:  

in find_directory():

const char *p;

#ifdef WINDOWS32
...
p[-1] = '\0';
#endif

Rafi Einstein <rafi_einstein>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by psmith (Posted a comment)
  • -email is unavailable- added by rafi_einstein (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2006-12-14 psmith StatusNone Later
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code