Skip to content

Commit

Permalink
Merge pull request ceph#48546 from rhcs-dashboard/update-node
Browse files Browse the repository at this point in the history
cmake: bump node version to 14

Reviewed-by: Ernesto Puerta <[email protected]>
  • Loading branch information
nizamial09 authored Oct 20, 2022
2 parents 4dc3579 + 5f95db3 commit 8f15d2e
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions doc/dev/developer_guide/dash-devel.rst
Original file line number Diff line number Diff line change
Expand Up @@ -215,8 +215,8 @@ The build process is based on `Node.js <https://nodejs.org/>`_ and requires the
Prerequisites
~~~~~~~~~~~~~

* Node 12.18.2 or higher
* NPM 6.13.4 or higher
* Node 14.15.0 or higher
* NPM 6.14.9 or higher

nodeenv:
During Ceph's build we create a virtualenv with ``node`` and ``npm``
Expand Down
2 changes: 1 addition & 1 deletion make-dist
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,7 @@ build_dashboard_frontend() {

$CURR_DIR/src/tools/setup-virtualenv.sh $TEMP_DIR
$TEMP_DIR/bin/pip install nodeenv
$TEMP_DIR/bin/nodeenv --verbose -p --node=12.18.2
$TEMP_DIR/bin/nodeenv --verbose -p --node=14.15.1
cd src/pybind/mgr/dashboard/frontend

. $TEMP_DIR/bin/activate
Expand Down
2 changes: 1 addition & 1 deletion src/pybind/mgr/dashboard/frontend/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ else(WITH_SYSTEM_NPM)
OUTPUT "${mgr-dashboard-nodeenv-dir}/bin/npm"
COMMAND ${CMAKE_SOURCE_DIR}/src/tools/setup-virtualenv.sh --python=${MGR_PYTHON_EXECUTABLE} ${mgr-dashboard-nodeenv-dir}
COMMAND ${mgr-dashboard-nodeenv-dir}/bin/pip install nodeenv
COMMAND ${mgr-dashboard-nodeenv-dir}/bin/nodeenv --verbose ${node_mirror_opt} -p --node=12.18.2
COMMAND ${mgr-dashboard-nodeenv-dir}/bin/nodeenv --verbose ${node_mirror_opt} -p --node=14.15.1
COMMAND mkdir ${mgr-dashboard-nodeenv-dir}/.npm
WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
COMMENT "dashboard nodeenv is being installed")
Expand Down

0 comments on commit 8f15d2e

Please sign in to comment.