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

Fix docker-compose build errors #106

Closed

Conversation

blooper05
Copy link

Hi! Thank you for a useful tool 😆

I've got 2 errors when I execute docker-compose build command.
So I've added a test to CI for checking that the build always succeeds, and fixed those errors.

  1. matplotlib installation error

  2. pandas installation error

Please check my changes and merge them if you can. Thanks!

`docker-compose build` occurs an error when installing `matplotlib`,
due to lack of dependent packages.
`docker-compose build` occurs a known installation error of `pandas`.
refs. pandas-dev/pandas#20775
@CLAassistant
Copy link

CLAassistant commented Jun 20, 2018

CLA assistant check
All committers have signed the CLA.

@0xdabbad00
Copy link
Collaborator

Thank you for this PR @blooper05 and sorry for taking so long to get back to you. I've been trying to decide what to do for docker support for CloudMapper. I personally don't use Docker with it and as such have been having a hard time trying to figure out what to do for any support for Docker with the tool. I've decided I need to just drop support for Docker as I'm the only person currently maintaining this project, and do so only in my spare time. If you'd like to run a separate project somehow to provide Docker support I can link to that.

@0xdabbad00 0xdabbad00 closed this Jul 9, 2018
@FernandoMiguel
Copy link
Contributor

Since I did one of original Docker PR I may pick this up again next month.
Maybe.

@0xdabbad00 0xdabbad00 mentioned this pull request Jul 25, 2019
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.

4 participants