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

Bugfix/issue#5 #7

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Bugfix/issue#5 #7

wants to merge 5 commits into from

Conversation

mwolf43
Copy link

@mwolf43 mwolf43 commented May 27, 2023

Hello, I'm using 0install for the distribution own developed tools on Linux and Windows for a small and private group of users. I came across the problem that this implementation of 0publish seemed not to be fully compatible with Python 3, which was a blocker on my Ubuntu system. On the other hand it produced errors on Windows (see related commit messages).
Additionally I added support for the usage of GPG passphrases according to the other implementation, Zero Install Publishing Tools for Windows.
The changes have been tested on Linux and Windows so far. Please review them and integrate them into the upstream repository, if possible. I'm not a Python programmer, so please forgive me when my code is not of the required quality. In this case I would happy to see your improvements ;-) Thanks!

@backlabs1
Copy link

Hi @mwolf43, with Git in general, it's better to avoid combining a bugfix and new feature in a single pull request. Thank you, though.

(I am not affiliated with 0install.)

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

Successfully merging this pull request may close these issues.

2 participants