Skip to content

Scriptable pre ./configure almost works #216

Closed
Closed
Scriptable pre ./configure almost works#216
Labels
type:build/install

Description

@villasv
villasv
opened on Oct 22, 2016

I'm trying to automate the build process of a tensorflow serving docker image, adding the bazel-build for the model server in the provided Dockerfile.
The problem is after cloning the serving repo we have to configure it, and while most configuration steps can be pre-answered with env variables like

RUN export PYTHON_BIN_PATH=/usr/bin/python && \
    export TF_NEED_GCP=0 && \
    export TF_NEED_HDFS=0

The configuration of the python libs controlled by util/python/python_config.sh can only be achieved with normal input.
I know this is actually an issue in tensorflow instead of serving, if I make a PR there I'll reference this one.

Activity

  1. villasv commented on Oct 24, 2016

    @villasv
    villasv
    on Oct 24, 2016
    Author

    The PR just got merged. I could make a PR here just to update the refs, but I guess you guys do it by yourselves regularly so I'll just wait for it and close it right after :-)

  2. chrisolston
    added
    type:build/install
    on Oct 27, 2016

  3. kirilg commented on Nov 3, 2016

    @kirilg
    kirilg
    on Nov 3, 2016
    Contributor

    We updated the TF submodule to point to this commit so your change should be included.

    I'll close this issue since you should be unblocked to automatically configure in a script. Please re-open if anything is broken.

  4. kirilg
    closed this as completedon Nov 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    type:build/install

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions