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

Docker #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Docker #2

wants to merge 1 commit into from

Conversation

senare
Copy link

@senare senare commented Dec 7, 2020

Not really working out for me, bad dependencies ?

@brother
Copy link
Owner

brother commented Dec 9, 2020

A very basic check for dependencies is in bin/gleshys-dep-check - I recorded the exact versions of the dependencies needed when I introduced them. There are no checks for lower or upper versions and thus fails to be very usable other than documentation at this moment.
For instance the dependency check expects bash to be version 5.0.18 but my machine uses 5.1.0 now (and gleshys works fine. And in #1 we learned about the lower version of bash being some 4.0.x where mapfile was introduced (and also that this makes the code unusable at mac os).

I can see some points in having a docker image for using in some automated validation probably. For every day use it seems a bit pointless, the tools are meant to move interaction from web towards the command line. Having docker between the user and them commands probably makes it a tad bit harder to use. Could add some magic gleshys docker [command + flags] that executes everything inside a container or such.

First thing is to untangle the versions that alpine is using and see what can be done to move that forward. Or use a minimal debian image as base as I am using debian my self for the development stuff.

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