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

spawn: autoload_functions better detection #38

Open
icasimpan opened this issue Apr 15, 2023 · 1 comment
Open

spawn: autoload_functions better detection #38

icasimpan opened this issue Apr 15, 2023 · 1 comment

Comments

@icasimpan
Copy link
Collaborator

icasimpan commented Apr 15, 2023

autoload_functions is commonly used as follows:

## list of functions to auto-load
function_to_autoload="
banner
greet
"

autoload_functions "$function_to_autoload"

and it's fine. Spawn understands it.

However, when list of functions is listed this way:

function_to_autoload="banner greet"

it breaks.

Issue is the line function: core/lib/getdeps_indirect.bash.inc

   echo $($CAT_BIN $file | $SED_BIN -n -e "/$deps_varname\=/,/\"/ p" | $GREP_BIN -v $deps_varname | $GREP_BIN -v '"')
@icasimpan
Copy link
Collaborator Author

Keeping this ticket open to refactor the initial fix. Hint: check the TODO list in initial fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant