bazelbuild / rules_appengine

AppEngine rules for Bazel
Apache License 2.0
30 stars 51 forks source link

missing com_google_cloud_sdk #90

Open thehesiod opened 6 years ago

thehesiod commented 6 years ago

I tried updating google/upvote to use 0.0.8 but then ran into: /monolith_binary.deploy: line 26: ${{TMPDIR:-/tmp}}/war.XXXXXXXX: bad substitution presumably due to double curly braces that seems to be fixed in the latest release.

Using head I get: upvote/gae/BUILD:15:1: no such package '@com_google_cloud_sdk//': The repository could not be resolved and referenced by '//upvote/gae:monolith_binary.deploy'

thehesiod commented 6 years ago

ah, looks like I had to add:

load(
    "@io_bazel_rules_appengine//appengine:sdk.bzl",
    "appengine_repositories",
)

appengine_repositories()

may be worth mentioning in docs for post 0.0.8?

thehesiod commented 6 years ago

found another issue, on this line: https://github.com/bazelbuild/rules_appengine/blob/master/appengine/py/appengine_deploy.sh.template#L26

TMPDIR ends with a slash on my system, so you get a double slash, and a subsequent error because it can't find the directory. I fixed this on my system like so:

tmp_dir=$(echo ${TMPDIR:-/tmp}/war.XXXXXXXX | tr -s /)
tmp_dir="$(mktemp -d "${tmp_dir}")"

still have another error deploying, investigating

thehesiod commented 6 years ago

so now it seems like the requests_toolbelt.adapters is not getting loaded

update: I had to set overwrite_appengine_config = False, worth documenting as well for backwards compatibility

pmbethe09 commented 6 years ago

If this investigation leads to some bug fixes, please send me a PR

thehesiod commented 6 years ago

let me know what you guys think: https://github.com/bazelbuild/rules_appengine/pull/91