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

load_commands command #11

Open
sagiegurari opened this issue Dec 31, 2019 · 3 comments
Open

load_commands command #11

sagiegurari opened this issue Dec 31, 2019 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@sagiegurari
Copy link
Owner

Will load native library and extract+load all commands from there.

@sagiegurari sagiegurari added the enhancement New feature or request label Dec 31, 2019
@sagiegurari sagiegurari self-assigned this Dec 31, 2019
@CaptObvious
Copy link

Does this add support for removing standard library commands before running a script? For example, removing the std::env, std::fs, std::net and std::process parts of the stdlib?

@sagiegurari
Copy link
Owner Author

didn't think it should.
if we want to remove commands, it should be a command on its own, like unalias removes aliases.
but... i wonder what is the use case. i mean, why would i remove commands?
if for example its because i don't want to give someone env/fs access, i would probably when i embed duckscript, i would not loads those commands.
but i am not sure what the use case exactly is to give a knowledgeable answer.

@nickheyer
Copy link
Contributor

didn't think it should. if we want to remove commands, it should be a command on its own, like unalias removes aliases. but... i wonder what is the use case. i mean, why would i remove commands? if for example its because i don't want to give someone env/fs access, i would probably when i embed duckscript, i would not loads those commands. but i am not sure what the use case exactly is to give a knowledgeable answer.

embedded duckscript >> duckscript w/ all commands (wrapper) >> duckscript w/ scoped commands

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants