GitHub user NicoK opened a pull request: https://github.com/apache/flink/pull/5788
[FLINK-9110][docs] fix local bundler installation ## What is the purpose of the change The fallback to installing `bundler` locally within `docs/build_docs.sh` did not work for several reasons: - the `-rubygems` parameter to `ruby` is failing with Ruby 2.5: ``` > ruby -rubygems -e 'puts Gem.user_dir' Traceback (most recent call last): 1: from /usr/lib64/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:59:in `require' /usr/lib64/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:59:in `require': cannot load such file -- ubygems (LoadError) ``` - the locally installed bundler formatted executables with the ruby version and was thus not available as `bundle` ## Brief change log - remove `-rubygems` parameter in `ruby` call - do not echo a warning when a local `bundle` is found (adapt the `PATH` to look for that in a second step) - install `bundler` locally with `--no-format-executable` ## Verifying this change This change added tests and can be verified as follows: - run `./docs/build_docs.sh` without globally installed `bundler` ## Does this pull request potentially affect one of the following parts: - Dependencies (does it add or upgrade a dependency): **no** - The public API, i.e., is any changed class annotated with `@Public(Evolving)`: **no** - The serializers: **no** - The runtime per-record code paths (performance sensitive): **no** - Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: **no** - The S3 file system connector: **no** ## Documentation - Does this pull request introduce a new feature? **no** - If yes, how is the feature documented? **not applicable** You can merge this pull request into a Git repository by running: $ git pull https://github.com/NicoK/flink flink-9110 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/5788.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #5788 ---- commit b6258697aabb8688aac19b5b228ddf8926e518f3 Author: Nico Kruber <nico@...> Date: 2018-03-29T13:55:47Z [FLINK-9110][docs] fix local bundler installation commit 540485a5620f6dcdd98e751a086076fb80997f65 Author: Nico Kruber <nico@...> Date: 2018-03-29T13:56:23Z [hotfix][docs] remove duplicate bundle installation ---- ---