[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 5/8] builtins/source: parse the -i option
|
From: |
Greg Wooledge |
|
Subject: |
Re: [PATCH v2 5/8] builtins/source: parse the -i option |
|
Date: |
Tue, 14 May 2024 16:00:17 -0400 |
On Tue, May 14, 2024 at 03:51:10PM -0400, Chet Ramey wrote:
> On 5/13/24 6:37 AM, Matheus Afonso Martins Moreira wrote:
> > Passing the -i option to the source builtin
> > enables isolated sourcing mode which restricts
> > its search path to the directories defined by
> > the BASH_SOURCE_PATH variable. This also has
> > the added benefit of not touching PATH at all.
>
>
> What do folks think about forcing an option to enable using
> BASH_SOURCE_PATH? Should it be required? Is it necessary? (I personally
> think it is not.) We discussed this briefly last week but without any
> conclusion.
I think setting the BASH_SOURCE_PATH variable already expresses the
intent to use it. There's no need for another command to enable the
variable to work.
- Re: [PATCH v2 1/8] findcmd: parameterize path variable in functions, (continued)
[PATCH v2 2/8] findcmd: define find_in_path_var function, Matheus Afonso Martins Moreira, 2024/05/13
[PATCH v2 3/8] builtins/source: extract file executor function, Matheus Afonso Martins Moreira, 2024/05/13
[PATCH v2 4/8] builtins/source: refactor file searching function, Matheus Afonso Martins Moreira, 2024/05/13
[PATCH v2 5/8] builtins/source: parse the -i option, Matheus Afonso Martins Moreira, 2024/05/13
- Re: [PATCH v2 5/8] builtins/source: parse the -i option, Chet Ramey, 2024/05/14
- Re: [PATCH v2 5/8] builtins/source: parse the -i option,
Greg Wooledge <=
- Re: [PATCH v2 5/8] builtins/source: parse the -i option, Lawrence Velázquez, 2024/05/14
- Re: [PATCH v2 5/8] builtins/source: parse the -i option, Koichi Murase, 2024/05/14
- Re: [PATCH v2 5/8] builtins/source: parse the -i option, Koichi Murase, 2024/05/14
- Re: [PATCH v2 5/8] builtins/source: parse the -i option, Chet Ramey, 2024/05/15
- Re: [PATCH v2 5/8] builtins/source: parse the -i option, Koichi Murase, 2024/05/15
- Re: [PATCH v2 5/8] builtins/source: parse the -i option, Chet Ramey, 2024/05/16
- Re: [PATCH v2 5/8] builtins/source: parse the -i option, Greg Wooledge, 2024/05/16
- Re: [PATCH v2 5/8] builtins/source: parse the -i option, Koichi Murase, 2024/05/20
Re: [PATCH v2 5/8] builtins/source: parse the -i option, Matheus Afonso Martins Moreira, 2024/05/16
Re: [PATCH v2 5/8] builtins/source: parse the -i option, Greg Wooledge, 2024/05/16