Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

source: fix variable interpretation #3469

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

source: fix variable interpretation #3469

wants to merge 1 commit into from

Conversation

flatcap
Copy link
Member

@flatcap flatcap commented Jul 11, 2022

Change source to look for NeoMutt variables first.


The source and set commands differed in how they interpreted a
$variable name.

set my_c = `echo $my_dir/test.rc`

set would look for a NeoMutt variable called my_dir and failing that
look for an extern environment variable with that name.

source `echo $my_dir/test.rc`

source only looked for an extern environment variable.

@flatcap flatcap added the type:bug Bug label Jul 11, 2022
@flatcap flatcap self-assigned this Jul 11, 2022
@flatcap flatcap linked an issue Jul 11, 2022 that may be closed by this pull request
@flatcap flatcap force-pushed the devel/source branch 3 times, most recently from 58fcc7f to 41eb8c6 Compare March 23, 2023 08:40
@flatcap flatcap force-pushed the devel/source branch 2 times, most recently from 238ea38 to d4524b9 Compare April 14, 2023 22:30
@flatcap flatcap force-pushed the devel/source branch 2 times, most recently from 8ac6f1a to 7682748 Compare April 29, 2023 18:41
@flatcap flatcap force-pushed the devel/source branch 2 times, most recently from a61ebc4 to 1205a9e Compare May 15, 2023 17:26
@flatcap flatcap force-pushed the devel/source branch 3 times, most recently from 609331b to db9b4f7 Compare June 20, 2023 17:07
@flatcap flatcap force-pushed the devel/source branch 2 times, most recently from 334bed6 to 5a82da5 Compare October 29, 2023 16:23
@flatcap flatcap force-pushed the devel/source branch 2 times, most recently from 93857c9 to 83d1b6c Compare November 16, 2023 19:42
Change `source` to look for NeoMutt variables first.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

source command and/or variable expansion work inconsistently
1 participant