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

Add the ability for extensions to have requirements on other extension #37

Closed
tfoote opened this issue Feb 25, 2019 · 2 comments · Fixed by #242
Closed

Add the ability for extensions to have requirements on other extension #37

tfoote opened this issue Feb 25, 2019 · 2 comments · Fixed by #242
Labels
enhancement New feature or request

Comments

@tfoote
Copy link
Collaborator

tfoote commented Feb 25, 2019

For example the nvidia extension will need the x11 extension to be useful.

Also the home extension should require the user extension for permissions.

There' s a question of ordering, required before or not. And for each sequential element aka snippets.

Preamble elements are named so order independent, and arguments are order independent as far as I know as long as they're escaped properly.

This will resolve this todo:

desired_plugins = ['x11', 'nvidia', 'user'] #TODO(Tfoote) encode the x11 dependency into the plugin and remove from test here

@tfoote tfoote added the enhancement New feature or request label Feb 25, 2019
@stonier
Copy link
Contributor

stonier commented Mar 31, 2021

Experimenting with this at stonier#31.

Is it in the vicinity of what you were looking for?

@agyoungs
Copy link
Contributor

This looks stale, but would be pretty useful for a few things I'm working on. Is there any progress on this? I'm happy to take a stab at it if the solution provided by @stonier isn't quite complete and they don't have the time to work on it at the moment.

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

Successfully merging a pull request may close this issue.

3 participants