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
marmarek opened this Issue Mar 6, 2016 · 0 comments

Comments

Projects
None yet
1 participant
@marmarek
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 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

Rename qubes.xdg python module to qubesxdg
Do not interfere with 'qubes' module.

QubesOS/qubes-issues#1813

@marmarek marmarek referenced this issue in QubesOS/qubes-mgmt-salt-base-overrides Aug 31, 2016

Merged

Make it work on R4 #2

marmarek added a commit to marmarek/qubes-mgmt-salt-base-overrides that referenced this issue May 22, 2017

@marmarek marmarek closed this Jun 2, 2017

@qubesos-bot qubesos-bot referenced this issue in QubesOS/updates-status Jun 9, 2017

Closed

core-agent-linux v4.0.0 (r4.0) #68

@qubesos-bot qubesos-bot referenced this issue in QubesOS/updates-status Jul 5, 2017

Closed

mgmt-salt-base-overrides v4.0.1 (r4.0) #112

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