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 - Normal Item Group:  Enhancement
Status:  None Privacy:  Public
Assigned to:  None Open/Closed:  Open
Component Version:  4.2 Operating System:  Any
Fixed Release:  None Triage 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.

Anonymous

 

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

Attach Files:
   
   
Comment:
   

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
  • -email is unavailable- added by psmith (Posted a comment)
  •  

    Do you think this task is very important?
    If so, you can 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 UTC psmith Operating 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 UTC None Attached File- => Added read.c.diff, #37115

    Back to the top


    Powered by Savane 3.3