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

Any interest in a Docker image for the build? #23

Open
lpmi-13 opened this issue Mar 13, 2019 · 8 comments
Open

Any interest in a Docker image for the build? #23

lpmi-13 opened this issue Mar 13, 2019 · 8 comments

Comments

@lpmi-13
Copy link

lpmi-13 commented Mar 13, 2019

After having some issues getting this to build (as I suspect others have also encountered), I attempted to put everything into a Dockerfile so that this problem only would need solving once.

I have just now realized that the current setup doesn't allow for a clean build only from the GitHub source and requires some files from the sourceforge repo.

So before I go any further with this, is there any interest in adding in a Dockerfile that grabs the compressed archive from sourceforge and just manually copies the necessary files into the lib folder of openccg prior to building with ant?

@mwhite14850
Copy link
Member

mwhite14850 commented Mar 14, 2019 via email

@chrzyki
Copy link

chrzyki commented Mar 14, 2019

Would love to have something like this as well and would be a great way of preserving the work being done here! Let me know if I can help in any way!

@lpmi-13
Copy link
Author

lpmi-13 commented Mar 14, 2019

alright, great. Then I guess the next step is to clarify if there's any particular reason why the source for building those jar files isn't also in the github repository?

I'm assuming the logic is that asking people to additionally compile those from source felt like a bit too much to handle, though that would be an even better argument for a stable docker image.

I'll start working on a simple Dockerfile that pulls in the sourceforge stuff and see how I get on. Have got it all working locally, so will just try to replicate that. If we're unable to add in the source of the other jars for a full clean build, then we might want to use something like docker-compose to pull from a public dockerhub image (so the sourceforge stuff only needs downloading if the image is updated, which doesn't seem like it will be that frequent, vs downloading each time someone wants to build the docker container).

Ideally we have the complete source in one place, and then a docker build is much easier and straightforward, but I appreciate this might not be possible.

Oh, and I didn't actually get to the stage yet where I needed to do anything with python...was that for using the project to actually generate sentences? It was a bit unclear from the readme so just want to make sure that I've got all the bases covered.

@shoeffner
Copy link
Contributor

shoeffner commented Mar 14, 2019 via email

@mwhite14850
Copy link
Member

mwhite14850 commented Mar 15, 2019 via email

@lpmi-13
Copy link
Author

lpmi-13 commented Mar 15, 2019

ah yes, of course. I should have assumed something like ant-contrib.jar wasn't specific to this project (I'm not really a java user, clearly). I've found almost all of the dependencies via the maven repository and will add in pulling from their into the Dockerfile. Will try to track down the rest so it's a bit closer to a clean build.

Also got the python bit working, since apparently python2.7 is not installed by default anymore with ubuntu16.04 images.

@mwhite14850
Copy link
Member

mwhite14850 commented Mar 15, 2019 via email

@mwhite14850
Copy link
Member

Hi Adam (also Dave)

What is the status of this direction? Šarūnas Navickas has just made a pull request for migrating to Maven; as noted for that request, it would be helpful to figure out whether one or both of these should be incorporated.

#27

Thanks!
Mike

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

No branches or pull requests

4 participants