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

Upgrade to buster and python 3 #72

Merged
merged 3 commits into from
Jan 11, 2020
Merged

Conversation

marciosmiderle
Copy link
Contributor

Python 3
Simplify dependencies
Install warsaw not in startup.sh, but via Dockerfile, and always start warsaw
Now possible to use enviroment variables to set bankfiles and Xauthority with .env file

Python 3
Simplify dependencies
Install warsaw not in startup.sh, but via Dockerfile, and always start warsaw
Now possible to use enviroment variables to set bankfiles and Xauthority with .env file
Copy link
Owner

@farribeiro farribeiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR looks goods... I have some appointments to do

Dockerfile Outdated Show resolved Hide resolved
startup.sh Show resolved Hide resolved
@farribeiro
Copy link
Owner

And this PR corrects #35 ?

@farribeiro farribeiro added the P3 Prioridade Alta label Jan 10, 2020
@marciosmiderle
Copy link
Contributor Author

And this PR corrects #35 ?

I think so, I am using with Banco Sicredi with no problem. Warsaw needs python 3.

@farribeiro
Copy link
Owner

And this PR corrects #35 ?

I think so, I am using with Banco Sicredi with no problem. Warsaw needs python 3.

Its not maintained for a long period, and changed base image many times, if works in alpine image its a great job like this issue

Copy link
Owner

@farribeiro farribeiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great

@farribeiro farribeiro merged commit 69d67a3 into farribeiro:master Jan 11, 2020
@farribeiro farribeiro self-assigned this Jan 11, 2020
@farribeiro
Copy link
Owner

farribeiro commented Jan 11, 2020

Docker hub started build image

@FabioBeneditto
Copy link
Contributor

Working fine with Sicredi

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

Successfully merging this pull request may close these issues.

3 participants