gradle-docker Display docker command line output. This also ensures the process output...

Changes to the DockerNativeClient so that the process buffer does not get full and hang the process.waitFor() blocking call for large docker builds. It also shows the progress of the docker build during its execution.

Aug 27 '14 15:08 skullmaggots

Added another minor change to this pull request to use ant copy task instead of built in copy. This is done so only files that have been modified are copied across from gradle build dir to docker stage if they have changed. Doing this means you will get full use of docker cache on ADD command, which also means subsequent docker commands after that will also get benefit of the cache.

Aug 28 '14 15:08 skullmaggots

Sweet! I've been wanting to see this!

Any reason why you used ant's copy instead of the Sync task in Gradle? (or why you didn't wire up the inputs/outputs for a copy task such that Gradle won't copy the file if it doesn't need to?)

Aug 28 '14 16:08 sgoings

Note: provides a fix for #26...

Aug 28 '14 16:08 sgoings

No technical reason - just looking for something other than gradle copy task which always copies the file whether it has changed or not.

Aug 29 '14 16:08 skullmaggots

Part of the problem is that it was using the copy command in Gradle, not the full-fledged copy task, where the latter doesn't know about inputs/outputs so it doesn't benefit from any up-to-date checks. (see: http://www.gradle.org/docs/current/userguide/working_with_files.html#copy)

Aug 29 '14 16:08 sgoings

I agree that what we probably want to do is fully wire this to Gradle's task inputs/outputs, but my sense right now is that's a bit more work (see #20 for some related issues). For now I think avoiding the copy when not necessary is a good step. I'll do a quick sanity check and then merge to dev.

Sep 04 '14 22:09 sfitts

Unfortunately I'm seeing a failure in one of the tests -- testAddFileWithDir. I think the ant copy task has an issue copying directories. The error I see is:

: Use a resource collection to copy directories.
    at org.apache.tools.ant.taskdefs.Copy.validateAttributes(Copy.java:684)
    at org.apache.tools.ant.taskdefs.Copy.execute(Copy.java:442)

Sep 04 '14 23:09 sfitts

FYI -- I swiped the first part of this (to display the output as we go) and included it in my recent commit to dev.

Sep 11 '14 03:09 sfitts

Any plans to make a release with this issue resolved? We are kind of blocked on issue #37 "docker build task is hanging" and I guess that the code committed to dev as described above solves this issue.

Regards, Magnus.

Jun 26 '15 13:06 magnus-larsson

I am running into this issue in Windows. Any updates into when this fix will be released?

Thank you,

Marco

Jun 29 '15 21:06 marcoivich

We're also having this problem - on Windows systems, the build hangs. We still have a lot of developers working on Windows laptops, and will for the foreseeable future. Any update on when the fix will be included in a release?