[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug #47880] Allow updates to .INCLUDE_DIRS to change search path
From: |
Steven Simpson |
Subject: |
[bug #47880] Allow updates to .INCLUDE_DIRS to change search path |
Date: |
Mon, 26 Apr 2021 14:51:30 -0400 (EDT) |
User-agent: |
Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:87.0) Gecko/20100101 Firefox/87.0 |
Follow-up Comment #12, bug #47880 (project make):
> > include $(CURDIR)/foo.mk
>
> This indeed suppresses the lookup.
>
> My opinion is that the users should not have to know this detail.
I didn't realize you were wearing your 'user' hat! To me, this is a problem
ideally for the author to solve. However, I'm not against an option to either
reset the include path or disable its initial population, allowing a user to
work around the issue.
> Also, most (as far as i can tell) existing makefiles do not do this.
(Indeed, I only found out about CURDIR yesterday! I had thought prefixing ./
was enough, and I shall be modifying my makefiles accordingly.)
> > this might be better considered a question of disabling the search path on
a per-include basis
>
> Is there a need for such flexibility?
Yes, from the author's perspective. Since you report that using $(CURDIR)
allows the author to resolve the issue, my suggestions such as includelocal or
making a special case for ./-prefixed paths are redundant.
> A command line switch or +I get the job done and relieve the users from
having to change the makefiles.
I'm not against such user-oriented solutions. :-) Sorry if you got that
impression.
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/bugs/?47880>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/
- [bug #47880] Allow updates to .INCLUDE_DIRS to change search path, (continued)
- [bug #47880] Allow updates to .INCLUDE_DIRS to change search path, Paul D. Smith, 2021/04/20
- [bug #47880] Allow updates to .INCLUDE_DIRS to change search path, Dmitry Goncharov, 2021/04/20
- [bug #47880] Allow updates to .INCLUDE_DIRS to change search path, Paul D. Smith, 2021/04/23
- [bug #47880] Allow updates to .INCLUDE_DIRS to change search path, Steven Simpson, 2021/04/24
- [bug #47880] Allow updates to .INCLUDE_DIRS to change search path, Paul D. Smith, 2021/04/24
- [bug #47880] Allow updates to .INCLUDE_DIRS to change search path, Dmitry Goncharov, 2021/04/25
- Re: [bug #47880] Allow updates to .INCLUDE_DIRS to change search path, Jeffrey Walton, 2021/04/25
- Re: [bug #47880] Allow updates to .INCLUDE_DIRS to change search path, Paul Smith, 2021/04/25
- [bug #47880] Allow updates to .INCLUDE_DIRS to change search path, Steven Simpson, 2021/04/25
- [bug #47880] Allow updates to .INCLUDE_DIRS to change search path, Dmitry Goncharov, 2021/04/26
- [bug #47880] Allow updates to .INCLUDE_DIRS to change search path,
Steven Simpson <=
- [bug #47880] Allow updates to .INCLUDE_DIRS to change search path, Dmitry Goncharov, 2021/04/26