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

Move external packages out of qubes python module #1813

Closed
2 tasks done
marmarek opened this issue Mar 6, 2016 · 0 comments
Closed
2 tasks done

Move external packages out of qubes python module #1813

marmarek opened this issue Mar 6, 2016 · 0 comments
Labels
C: core P: minor Priority: minor. The lowest priority, below "default."
Milestone

Comments

@marmarek
Copy link
Member

marmarek commented Mar 6, 2016

@woju (#1541 (comment))

Combining one python package from different distro packages is generally a bad idea. I wouldn't discount it altogether, however the need is very rare. Note that there are similar solutions like drop-in directories for plugins (vide /srv/salt/_modules), those however aren't proper python packages. In case of qubes I strongly advise against.

I also don't think we need a "naming policy" precisely because the names aren't changing anything (this wouldn't be the case if we put those modules in single package). The gain is purely cosmetic. That said, we could name all our packages as qubes*, i.e. qubessplitgpg, qubesimgconverter, and qubesmanager for that matter.

@marmarek marmarek added C: core P: minor Priority: minor. The lowest priority, below "default." task labels Mar 6, 2016
@marmarek marmarek added this to the Release 4.0 milestone Mar 6, 2016
marmarek added a commit to marmarek/old-qubes-core-agent-linux that referenced this issue Aug 17, 2016
marmarek added a commit to marmarek/qubes-mgmt-salt-base-overrides that referenced this issue May 22, 2017
@marmarek marmarek closed this as completed Jun 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: core P: minor Priority: minor. The lowest priority, below "default."
Projects
None yet
Development

No branches or pull requests

1 participant