Skip to content

Commit

Permalink
Issue 1022 update readme with details on known issue (#1042)
Browse files Browse the repository at this point in the history
* Add solution and link to details for six import failure

* Make KnownIssues a subsection of Setup
  • Loading branch information
raulAtNines authored and nlopezgi committed Aug 7, 2019
1 parent 25097b5 commit 296ba26
Showing 1 changed file with 17 additions and 5 deletions.
22 changes: 17 additions & 5 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -159,16 +159,28 @@ container_pull(
)
```

Note: Bazel does not deal well with diamond dependencies. If the repositories that
are imported by `container_repositories()` have already been imported (at a
different version) by other rules you called in your `WORKSPACE`, which are
placed above the call to `container_repositories()`, arbitrary errors might
### Known Issues

* Bazel does not deal well with diamond dependencies.


If the repositories that are imported by `container_repositories()` have already been
imported (at a different version) by other rules you called in your `WORKSPACE`, which
are placed above the call to `container_repositories()`, arbitrary errors might
ocurr. If you get errors related to external repositories, you will likely
not be able to use `container_repositories()` and will have to import
directly in your `WORKSPACE` all the required dependencies (see the most up
to date impl of `container_repositories()` for details).

NEW: Starting with Bazel 0.27.0, you also need to add to your .bazelrc
* ImportError: No module named moves.urllib.parse

This is an example of an error due to a diamond dependency. If you get this
error, make sure to import rules_docker before other libraries, so that
_six_ can be patched properly.

See https://github.com/bazelbuild/rules_docker/issues/1022 for more details.

* Starting with Bazel 0.27.0, you also need to add to your .bazelrc
file the following:

```
Expand Down

0 comments on commit 296ba26

Please sign in to comment.