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

 
 

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

Submitted by:  Rafi Einstein <rafi_einstein>
Submitted on:  Wed 13 Dec 2006 04:43:17 PM UTC  
 
Severity: 3 - NormalItem Group: Build/Install
Status: LaterPrivacy: Public
Assigned to: NoneOpen/Closed: Closed
Component Version: 4.0Operating System: MS Windows
Fixed Release: NoneTriage Status: None

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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>
Project 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 File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 2 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 14 Dec 2006 05:57:47 AM UTCpsmithStatusNone=>Later
      Open/ClosedOpen=>Closed

    Back to the top


    Powered by Savane 3.1-cleanup1