Can not pass `fileobj` together with `path` context to `build`

It seems to be impossible to pass ephemeral Dockerfile to build command together with build context. Here is command line use case:

docker build -t . -f-<<EOF
Lorem ipsum docker file
EOF

According to the documentation https://docker-py.readthedocs.io/en/stable/images.htmlit should be possible to pass Dockerfile as file object, but in current implementation path param with context is ignored.

Here is path that fix part of this issue (it don’t work when path.startswith(('http://', 'https://','git://', 'github.com/', 'git@')))

diff --git a/docker/api/build.py b/docker/api/build.py
index 0486dce..711def2 100644
--- a/docker/api/build.py
+++ b/docker/api/build.py
@@ -133,7 +133,7 @@ class BuildApiMixin(object):
            if not fileobj:
                raise TypeError("You must specify fileobj with custom_context")
            context = fileobj
-        elif fileobj is not None:
+        elif path is None and fileobj is not None:
            context = utils.mkbuildcontext(fileobj)
        elif path.startswith(('http://', 'https://',
                            'git://', 'github.com/', 'git@')):
@@ -149,7 +149,7 @@ class BuildApiMixin(object):
                        lambda x: x != '' and x[0] != '#',
                        [l.strip() for l in f.read().splitlines()]
                    ))
-            dockerfile = process_dockerfile(dockerfile, path)
+            dockerfile = process_dockerfile(dockerfile, path, fileobj)
            context = utils.tar(
                path, exclude=exclude, dockerfile=dockerfile, gzip=gzip
            )
@@ -332,7 +332,13 @@ class BuildApiMixin(object):
            log.debug('No auth config found')


-def process_dockerfile(dockerfile, path):
+def process_dockerfile(dockerfile, path, fileobj):
+    if fileobj is not None:
+        return (
+            '.dockerfile.{0:x}'.format(random.getrandbits(160)),
+            fileobj.read()
+        )
+
    if not dockerfile:
        return (None, None)

[majkrzak]

@danieladams456 I end up with the PR that they don’t want to merge and this simple polyfill that might help you:

import docker.api.build
docker.api.build.process_dockerfile = lambda dockerfile, path: ('Dockerfile', dockerfile)

And then simply:

client.images.build(path='.',dockerfile='FROM ...')

[majkrzak]

If dockerfile it’s not part of context for some reason, it have to be affected to be dockerized. I know that it can be passed as absolute path, but if dockerfile is ephemeral it requires saving is somewhere is filesystem which is little more complicated. Generally it py lib lacs functionality of cli, and arguments could be simplified a bit.