Skip to content
This repository has been archived by the owner on Jul 1, 2021. It is now read-only.

0008 packable #2

Closed
wants to merge 5 commits into from
Closed

0008 packable #2

wants to merge 5 commits into from

Conversation

JanKoppe
Copy link

copied Dockerfile to the location that travis is expecting it to be. This should fix the CI builds.

@nuest
Copy link
Member

nuest commented Apr 11, 2016

Right... though that means we're not following your suggested simplification for the structure under /data.
The alternative could have been to put more logic into Travis, e.g. by having more complex sheell scripts execute the container. I don't really like that either.

I'll take a note to reconsider the directory structure!

@nuest
Copy link
Member

nuest commented Apr 11, 2016

And more importantly - how did you solve the issue with adding data to the container if it is in a different part of the directory tree?

@JanKoppe
Copy link
Author

Oh well, that solution is absolutely not a solution, now that I think about it. I'll take a closer look at it. I'm not sure the structure of this repo in general is well layed out - but more about that in a separate issue.

@JanKoppe JanKoppe closed this Apr 11, 2016
@JanKoppe JanKoppe mentioned this pull request Apr 11, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants