-
-
Notifications
You must be signed in to change notification settings - Fork 448
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
Fix broken Dockerfile #300
base: master
Are you sure you want to change the base?
Conversation
* Add missing \ * Dockerfile now built from repo root
For my personal use, I am using a different Dockerfile (with latest Debian)
|
It's fixed now with:
I disagree. The imapsync github releases are not tested in the docker context. They can work or they can fail.
I use the local imapsync for testing/fixing purposes. As long as the tests don't pass in a docker context I have to fix them. How? By modifying this local imapsync. Then I update this good imapsync upstream, which is not GitHub in my current workflow.
The github source is not guaranteed to run under the docker context.
I have enough headaches with one Dockerfile to maintain. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't approve the merge because there is a newer Dockerfile and other reasons done in the conversation
Hi Gilles,
I don't understand this - Docker is a build and packaging tool. Just like a Makefile usually uses the local sources, so do Dockerfiles. You wouldn't expect a Makefile to pull in the sources from a remote endpoint and just copy the local one to the side for reference, right? Rather you'd expect the build to use the local source and the local source to be the version you want (usually by checking out a tagged release, not using Typical workflow from a user would be: checkout a tagged release
...Is the version at https://imapsync.lamiral.info/imapsync typically containing changes not present in the git repo source..? So, if I may again, why not:
It is up to the builder to make sure they're using the right source. Possibly by downloading it from your site
One makes sense, I was (and still am btw) a bit confused why one docker image should contain two potentially different versions of the same application.. |
Well, that's not my way for now.
The local one is a temporary one, just for me. The Dockerfile is the same, it's for me first.
I have only one branch.
Is it? really? You know what a typical user workflow is? How do you know?
Yes, that's imapsync latest published. Not Docker guaranteed either, except the one I use when I build the imapsync docker image.
Or possibly using the old local one, or any one.
The imapsync docker image I provide usually contains the same releases, as it is the end of the docker building process I follow: |
Hey, you seem to be either defensive or upset - just leaving feedback here, assumed you're sharing the repo in public in order for it to be useful for others and appreciated some feedback to make it more-so 🤷
From experience building hundreds if not thousands of containerfiles from other projects and experience in the industry and open source community. Look around and you will probably find the same. The way it's done here is highly unusual and as you say, only really usable for you (made by you, for you, after all). But that being said, that doesn't mean it's objectively "better" or "correct" or that you need to follow that - a lot of it is just cultural. It's your project and you do the call on it, after all. I find imapsync a nice and useful tool, and would love for it to be more easy to use for others, without new people being confused and ending up either walking away from it or bothering you about the same thing. |
Hope I'm not too grumpy back - again, want to reinforce I'm super grateful for your making and maintaining this neat software, whatever your workflow is. Keep it real. (and in case it's of use:
to mount current working directory inside the containerfile - that way you get the local copy without having to rebuild even ;)) |
Ok, teacher!
Well, you're unfair. https://hub.docker.com/r/gilleslamiral/imapsync
|
I'm new to docker. I use it only to build the dockerhub imapsync image... |
Dockerfile does not build anymore. Since it now copies a (missing from install dir) local copy of imapsync, switched to building from repo root so proper version can be copied. Also adds missing trailing \ that caused an issue.
@gilleslamiral A separate question I have after addressing this: The image now contains two independent versions of imapsync, one from the local repo and one downloaded release. Commonly I would expect a single Dockerfile to only contain the version built from local source.
I think this can be pretty confusing. Do you see a clear need for a single container to contain both versions? If not I'd propose (and happy to contribute if you agree) one of the following approaches:
This would allow easy building of both versions from the same repo.