Skip to content

Commit 62f6511

Browse files
committed
Fix loading order for versioning in the Node provider
As required for the upcoming feature of Provider versions on Dockbit: Loading versions from .node-version via avn should override the version used in the Dockbit provider. This fix loads the Node version required in the Provider configuration only if no .node-version exists (hence avn isn't loaded).
1 parent 5e1be95 commit 62f6511

13 files changed

+35
-19
lines changed

CHANGELOG.md

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,10 @@ All changes to the images, whether it's a new image or an update will be added h
33

44
The format is based on [Keep a Changelog](http://keepachangelog.com/)
55

6+
## [Unreleased]
7+
### Fixed
8+
- Fix loading order of Node provider versions.
9+
610
## 2017-03-04
711
### Changed
8-
- Updated [Packer image](packer/) to use [Packer](http://packer.io/) version 0.12.3.
12+
- Updated [Packer image](packer/) to use [Packer](http://packer.io/) version 0.12.3.

node/0.10.46/Dockerfile

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
11
FROM dockbit/base:latest
22
MAINTAINER The Dockbit Team "[email protected]"
33

4-
RUN . $HOME/.bash_profile \
5-
&& nvm use 0.10.46
6-
74
# Execution environment
85
COPY docker-entrypoint.sh /docker-entrypoint.sh
96
ENTRYPOINT ["/docker-entrypoint.sh"]

node/0.10.46/docker-entrypoint.sh

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,11 @@
11
#!/bin/bash
22
source /bin/dockbit_bootstrap.sh
33

4+
# if avn/.node-version isn't involved, load the required version with nvm use
5+
if [ -z "$__avn_active_file" ]; then
6+
nvm use 0.10.46
7+
fi
8+
49
# Compose a string of which node dependencies to track
510
if [ -f package.json ]; then node_dependencies="-f package.json"; fi
611
if [ -f .node-version ]; then node_dependencies="$node_dependencies -f .node-version"; fi

node/0.12.15/Dockerfile

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
11
FROM dockbit/base:latest
22
MAINTAINER The Dockbit Team "[email protected]"
33

4-
RUN . $HOME/.bash_profile \
5-
&& nvm use 0.12.15
6-
74
# Execution environment
85
COPY docker-entrypoint.sh /docker-entrypoint.sh
96
ENTRYPOINT ["/docker-entrypoint.sh"]

node/0.12.15/docker-entrypoint.sh

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,11 @@
11
#!/bin/bash
22
source /bin/dockbit_bootstrap.sh
33

4+
# if avn/.node-version isn't involved, load the required version with nvm use
5+
if [ -z "$__avn_active_file" ]; then
6+
nvm use 0.12.15
7+
fi
8+
49
# Compose a string of which node dependencies to track
510
if [ -f package.json ]; then node_dependencies="-f package.json"; fi
611
if [ -f .node-version ]; then node_dependencies="$node_dependencies -f .node-version"; fi

node/5.12.0/Dockerfile

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
11
FROM dockbit/base:latest
22
MAINTAINER The Dockbit Team "[email protected]"
33

4-
RUN . $HOME/.bash_profile \
5-
&& nvm use 5.12.0
6-
74
# Execution environment
85
COPY docker-entrypoint.sh /docker-entrypoint.sh
96
ENTRYPOINT ["/docker-entrypoint.sh"]

node/5.12.0/docker-entrypoint.sh

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,11 @@
11
#!/bin/bash
22
source /bin/dockbit_bootstrap.sh
33

4+
# if avn/.node-version isn't involved, load the required version with nvm use
5+
if [ -z "$__avn_active_file" ]; then
6+
nvm use 5.12.0
7+
fi
8+
49
# Compose a string of which node dependencies to track
510
if [ -f package.json ]; then node_dependencies="-f package.json"; fi
611
if [ -f .node-version ]; then node_dependencies="$node_dependencies -f .node-version"; fi

node/6.5.0/Dockerfile

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
11
FROM dockbit/base:latest
22
MAINTAINER The Dockbit Team "[email protected]"
33

4-
RUN . $HOME/.bash_profile \
5-
&& nvm use 6.5.0
6-
74
# Execution environment
85
COPY docker-entrypoint.sh /docker-entrypoint.sh
96
ENTRYPOINT ["/docker-entrypoint.sh"]

node/6.5.0/docker-entrypoint.sh

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,11 @@
11
#!/bin/bash
22
source /bin/dockbit_bootstrap.sh
33

4+
# if avn/.node-version isn't involved, load the required version with nvm use
5+
if [ -z "$__avn_active_file" ]; then
6+
nvm use 6.5.0
7+
fi
8+
49
# Compose a string of which node dependencies to track
510
if [ -f package.json ]; then node_dependencies="-f package.json"; fi
611
if [ -f .node-version ]; then node_dependencies="$node_dependencies -f .node-version"; fi

node/6.8.0/Dockerfile

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
11
FROM dockbit/base:latest
22
MAINTAINER The Dockbit Team "[email protected]"
33

4-
RUN . $HOME/.bash_profile \
5-
&& nvm use 6.8.0
6-
74
# Execution environment
85
COPY docker-entrypoint.sh /docker-entrypoint.sh
96
ENTRYPOINT ["/docker-entrypoint.sh"]

0 commit comments

Comments
 (0)