bugmake - Bugs: bug #47880, Allow updates to .INCLUDE_DIRS to...


bug #47880: Allow updates to .INCLUDE_DIRS to change search path

Submitted by:  None
Submitted on:  Mon 09 May 2016 04:22:45 PM UTC  
Severity: 3 - NormalItem Group: Enhancement
Status: NonePrivacy: Public
Assigned to: NoneOpen/Closed: Open
Component Version: 4.2Operating System: Any
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.


Sat 21 May 2016 09:54:22 PM UTC, comment #1:

There is no need to add a new environment variable for this. The MAKEFLAGS variable already exists, and you can just add -I options into it which will be parsed as if you'd entered it on the command line. Also it's already possible when building GNU make on Windows to set the INCLUDEDIR preprocessor variable to add a new default pathname to GNU make: this just isn't done by default because there's no good choice for what it should be.

I've retargeted this issue as a generic enhancement to allow setting the .INCLUDE_DIRS variable to change the search path.

Paul D. Smith <psmith>
Project Administrator
Mon 09 May 2016 04:22:45 PM UTC, original submission:

Attached a patch (to read.c only) which might serve as a starting point for allowing an additional (hardcoded) default include path given in a Windows environment variable (proposed variable name: 'MAKELIBPATH').

Please note that the patch ignores multiple paths (delimited by ';') given in MAKELIBPATH, i.e. MAKELIBPATH should contain one path only.

Futher enhancement:

The solution above might be considered as an intermediate step to where all paths in .INCLUDEDIRS are searched for makefiles to be included - regardless of where the paths are defined. As of now paths provided by the '-I' switch on the command line and default paths are the only ones which are searched for include makefiles, while any paths added to .INCLUDEDIRS within a makefile are not searched.



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

Attach File(s):

Attached Files
file #37115:  read.c.diff added by None (1KiB - text/x-patch)


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)

    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 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 21 May 2016 09:54:22 PM UTCpsmithOperating SystemMS Windows=>Any
      Summary[Win] added MAKELIBPATH as default include directory=>Allow updates to .INCLUDE_DIRS to change search path
    Mon 09 May 2016 04:22:45 PM UTCNoneAttached File-=>Added read.c.diff, #37115

    Back to the top

    Powered by Savane 3.1-cleanup1