Skip to content

Commit e30082f

Browse files
yosifkityosifkit
authored
Merge pull request docker-library#306 from deivid-rodriguez/add_path_scoped_to_ruby_version_to_path_and_gem_path
Stop setting `BUNDLE_PATH`
2 parents defb10a + 82eecb7 commit e30082f

File tree

25 files changed

+75
-175
lines changed

25 files changed

+75
-175
lines changed

‎2.4/alpine3.10/Dockerfile‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -121,16 +121,12 @@ RUN set -eux; \
121121
gem --version; \
122122
bundle --version
123123

124-
# install things globally, for great justice
125-
# and don't create ".bundle" in all our apps
124+
# don't create ".bundle" in all our apps
126125
ENV GEM_HOME /usr/local/bundle
127-
ENV BUNDLE_PATH="$GEM_HOME" \
128-
BUNDLE_SILENCE_ROOT_WARNING=1 \
126+
ENV BUNDLE_SILENCE_ROOT_WARNING=1 \
129127
BUNDLE_APP_CONFIG="$GEM_HOME"
130-
# path recommendation: https://github.com/bundler/bundler/pull/6469#issuecomment-383235438
131-
ENV PATH $GEM_HOME/bin:$BUNDLE_PATH/gems/bin:$PATH
128+
ENV PATH $GEM_HOME/bin:$PATH
132129
# adjust permissions of a few directories for running "gem install" as an arbitrary user
133130
RUN mkdir -p "$GEM_HOME" && chmod 777 "$GEM_HOME"
134-
# (BUNDLE_PATH = GEM_HOME, no need to mkdir/chown both)
135131

136132
CMD [ "irb" ]

‎2.4/alpine3.11/Dockerfile‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -121,16 +121,12 @@ RUN set -eux; \
121121
gem --version; \
122122
bundle --version
123123

124-
# install things globally, for great justice
125-
# and don't create ".bundle" in all our apps
124+
# don't create ".bundle" in all our apps
126125
ENV GEM_HOME /usr/local/bundle
127-
ENV BUNDLE_PATH="$GEM_HOME" \
128-
BUNDLE_SILENCE_ROOT_WARNING=1 \
126+
ENV BUNDLE_SILENCE_ROOT_WARNING=1 \
129127
BUNDLE_APP_CONFIG="$GEM_HOME"
130-
# path recommendation: https://github.com/bundler/bundler/pull/6469#issuecomment-383235438
131-
ENV PATH $GEM_HOME/bin:$BUNDLE_PATH/gems/bin:$PATH
128+
ENV PATH $GEM_HOME/bin:$PATH
132129
# adjust permissions of a few directories for running "gem install" as an arbitrary user
133130
RUN mkdir -p "$GEM_HOME" && chmod 777 "$GEM_HOME"
134-
# (BUNDLE_PATH = GEM_HOME, no need to mkdir/chown both)
135131

136132
CMD [ "irb" ]

‎2.4/buster/Dockerfile‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -80,16 +80,12 @@ RUN set -eux; \
8080
gem --version; \
8181
bundle --version
8282

83-
# install things globally, for great justice
84-
# and don't create ".bundle" in all our apps
83+
# don't create ".bundle" in all our apps
8584
ENV GEM_HOME /usr/local/bundle
86-
ENV BUNDLE_PATH="$GEM_HOME" \
87-
BUNDLE_SILENCE_ROOT_WARNING=1 \
85+
ENV BUNDLE_SILENCE_ROOT_WARNING=1 \
8886
BUNDLE_APP_CONFIG="$GEM_HOME"
89-
# path recommendation: https://github.com/bundler/bundler/pull/6469#issuecomment-383235438
90-
ENV PATH $GEM_HOME/bin:$BUNDLE_PATH/gems/bin:$PATH
87+
ENV PATH $GEM_HOME/bin:$PATH
9188
# adjust permissions of a few directories for running "gem install" as an arbitrary user
9289
RUN mkdir -p "$GEM_HOME" && chmod 777 "$GEM_HOME"
93-
# (BUNDLE_PATH = GEM_HOME, no need to mkdir/chown both)
9490

9591
CMD [ "irb" ]

‎2.4/buster/slim/Dockerfile‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -106,16 +106,12 @@ RUN set -eux; \
106106
gem --version; \
107107
bundle --version
108108

109-
# install things globally, for great justice
110-
# and don't create ".bundle" in all our apps
109+
# don't create ".bundle" in all our apps
111110
ENV GEM_HOME /usr/local/bundle
112-
ENV BUNDLE_PATH="$GEM_HOME" \
113-
BUNDLE_SILENCE_ROOT_WARNING=1 \
111+
ENV BUNDLE_SILENCE_ROOT_WARNING=1 \
114112
BUNDLE_APP_CONFIG="$GEM_HOME"
115-
# path recommendation: https://github.com/bundler/bundler/pull/6469#issuecomment-383235438
116-
ENV PATH $GEM_HOME/bin:$BUNDLE_PATH/gems/bin:$PATH
113+
ENV PATH $GEM_HOME/bin:$PATH
117114
# adjust permissions of a few directories for running "gem install" as an arbitrary user
118115
RUN mkdir -p "$GEM_HOME" && chmod 777 "$GEM_HOME"
119-
# (BUNDLE_PATH = GEM_HOME, no need to mkdir/chown both)
120116

121117
CMD [ "irb" ]

‎2.4/stretch/Dockerfile‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -80,16 +80,12 @@ RUN set -eux; \
8080
gem --version; \
8181
bundle --version
8282

83-
# install things globally, for great justice
84-
# and don't create ".bundle" in all our apps
83+
# don't create ".bundle" in all our apps
8584
ENV GEM_HOME /usr/local/bundle
86-
ENV BUNDLE_PATH="$GEM_HOME" \
87-
BUNDLE_SILENCE_ROOT_WARNING=1 \
85+
ENV BUNDLE_SILENCE_ROOT_WARNING=1 \
8886
BUNDLE_APP_CONFIG="$GEM_HOME"
89-
# path recommendation: https://github.com/bundler/bundler/pull/6469#issuecomment-383235438
90-
ENV PATH $GEM_HOME/bin:$BUNDLE_PATH/gems/bin:$PATH
87+
ENV PATH $GEM_HOME/bin:$PATH
9188
# adjust permissions of a few directories for running "gem install" as an arbitrary user
9289
RUN mkdir -p "$GEM_HOME" && chmod 777 "$GEM_HOME"
93-
# (BUNDLE_PATH = GEM_HOME, no need to mkdir/chown both)
9490

9591
CMD [ "irb" ]

‎2.4/stretch/slim/Dockerfile‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -105,16 +105,12 @@ RUN set -eux; \
105105
gem --version; \
106106
bundle --version
107107

108-
# install things globally, for great justice
109-
# and don't create ".bundle" in all our apps
108+
# don't create ".bundle" in all our apps
110109
ENV GEM_HOME /usr/local/bundle
111-
ENV BUNDLE_PATH="$GEM_HOME" \
112-
BUNDLE_SILENCE_ROOT_WARNING=1 \
110+
ENV BUNDLE_SILENCE_ROOT_WARNING=1 \
113111
BUNDLE_APP_CONFIG="$GEM_HOME"
114-
# path recommendation: https://github.com/bundler/bundler/pull/6469#issuecomment-383235438
115-
ENV PATH $GEM_HOME/bin:$BUNDLE_PATH/gems/bin:$PATH
112+
ENV PATH $GEM_HOME/bin:$PATH
116113
# adjust permissions of a few directories for running "gem install" as an arbitrary user
117114
RUN mkdir -p "$GEM_HOME" && chmod 777 "$GEM_HOME"
118-
# (BUNDLE_PATH = GEM_HOME, no need to mkdir/chown both)
119115

120116
CMD [ "irb" ]

‎2.5/alpine3.10/Dockerfile‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -121,16 +121,12 @@ RUN set -eux; \
121121
gem --version; \
122122
bundle --version
123123

124-
# install things globally, for great justice
125-
# and don't create ".bundle" in all our apps
124+
# don't create ".bundle" in all our apps
126125
ENV GEM_HOME /usr/local/bundle
127-
ENV BUNDLE_PATH="$GEM_HOME" \
128-
BUNDLE_SILENCE_ROOT_WARNING=1 \
126+
ENV BUNDLE_SILENCE_ROOT_WARNING=1 \
129127
BUNDLE_APP_CONFIG="$GEM_HOME"
130-
# path recommendation: https://github.com/bundler/bundler/pull/6469#issuecomment-383235438
131-
ENV PATH $GEM_HOME/bin:$BUNDLE_PATH/gems/bin:$PATH
128+
ENV PATH $GEM_HOME/bin:$PATH
132129
# adjust permissions of a few directories for running "gem install" as an arbitrary user
133130
RUN mkdir -p "$GEM_HOME" && chmod 777 "$GEM_HOME"
134-
# (BUNDLE_PATH = GEM_HOME, no need to mkdir/chown both)
135131

136132
CMD [ "irb" ]

‎2.5/alpine3.11/Dockerfile‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -121,16 +121,12 @@ RUN set -eux; \
121121
gem --version; \
122122
bundle --version
123123

124-
# install things globally, for great justice
125-
# and don't create ".bundle" in all our apps
124+
# don't create ".bundle" in all our apps
126125
ENV GEM_HOME /usr/local/bundle
127-
ENV BUNDLE_PATH="$GEM_HOME" \
128-
BUNDLE_SILENCE_ROOT_WARNING=1 \
126+
ENV BUNDLE_SILENCE_ROOT_WARNING=1 \
129127
BUNDLE_APP_CONFIG="$GEM_HOME"
130-
# path recommendation: https://github.com/bundler/bundler/pull/6469#issuecomment-383235438
131-
ENV PATH $GEM_HOME/bin:$BUNDLE_PATH/gems/bin:$PATH
128+
ENV PATH $GEM_HOME/bin:$PATH
132129
# adjust permissions of a few directories for running "gem install" as an arbitrary user
133130
RUN mkdir -p "$GEM_HOME" && chmod 777 "$GEM_HOME"
134-
# (BUNDLE_PATH = GEM_HOME, no need to mkdir/chown both)
135131

136132
CMD [ "irb" ]

‎2.5/buster/Dockerfile‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -80,16 +80,12 @@ RUN set -eux; \
8080
gem --version; \
8181
bundle --version
8282

83-
# install things globally, for great justice
84-
# and don't create ".bundle" in all our apps
83+
# don't create ".bundle" in all our apps
8584
ENV GEM_HOME /usr/local/bundle
86-
ENV BUNDLE_PATH="$GEM_HOME" \
87-
BUNDLE_SILENCE_ROOT_WARNING=1 \
85+
ENV BUNDLE_SILENCE_ROOT_WARNING=1 \
8886
BUNDLE_APP_CONFIG="$GEM_HOME"
89-
# path recommendation: https://github.com/bundler/bundler/pull/6469#issuecomment-383235438
90-
ENV PATH $GEM_HOME/bin:$BUNDLE_PATH/gems/bin:$PATH
87+
ENV PATH $GEM_HOME/bin:$PATH
9188
# adjust permissions of a few directories for running "gem install" as an arbitrary user
9289
RUN mkdir -p "$GEM_HOME" && chmod 777 "$GEM_HOME"
93-
# (BUNDLE_PATH = GEM_HOME, no need to mkdir/chown both)
9490

9591
CMD [ "irb" ]

‎2.5/buster/slim/Dockerfile‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -106,16 +106,12 @@ RUN set -eux; \
106106
gem --version; \
107107
bundle --version
108108

109-
# install things globally, for great justice
110-
# and don't create ".bundle" in all our apps
109+
# don't create ".bundle" in all our apps
111110
ENV GEM_HOME /usr/local/bundle
112-
ENV BUNDLE_PATH="$GEM_HOME" \
113-
BUNDLE_SILENCE_ROOT_WARNING=1 \
111+
ENV BUNDLE_SILENCE_ROOT_WARNING=1 \
114112
BUNDLE_APP_CONFIG="$GEM_HOME"
115-
# path recommendation: https://github.com/bundler/bundler/pull/6469#issuecomment-383235438
116-
ENV PATH $GEM_HOME/bin:$BUNDLE_PATH/gems/bin:$PATH
113+
ENV PATH $GEM_HOME/bin:$PATH
117114
# adjust permissions of a few directories for running "gem install" as an arbitrary user
118115
RUN mkdir -p "$GEM_HOME" && chmod 777 "$GEM_HOME"
119-
# (BUNDLE_PATH = GEM_HOME, no need to mkdir/chown both)
120116

121117
CMD [ "irb" ]

0 commit comments

Comments
 (0)