Skip to content

Commit

Permalink
Change commit_override to git_commit
Browse files Browse the repository at this point in the history
  • Loading branch information
vyasr committed Oct 6, 2023
1 parent b552d3e commit ac28646
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 8 deletions.
4 changes: 2 additions & 2 deletions ci/build_python.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,10 +13,10 @@ package_name="cuml"
package_dir="python"

version=$(rapids-generate-version)
commit_override=$(git rev-parse HEAD)
git_commit=$(git rev-parse HEAD)

sed -i "s/__version__ = .*/__version__ = ${version}/g" ${package_dir}/${package_name}/__init__.py
sed -i "s/__git_commit__ = .*/__git_commit__ = \"${commit_override}\"/g" ${package_dir}/${package_name}/__init__.py
sed -i "s/__git_commit__ = .*/__git_commit__ = \"${git_commit}\"/g" ${package_dir}/${package_name}/__init__.py

rapids-logger "Begin py build"

Expand Down
8 changes: 2 additions & 6 deletions ci/build_wheel.sh
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ source rapids-date-string
# Use gha-tools rapids-pip-wheel-version to generate wheel version then
# update the necessary files
version=$(rapids-generate-version)
commit_override=$(git rev-parse HEAD)
git_commit=$(git rev-parse HEAD)

RAPIDS_PY_CUDA_SUFFIX="$(rapids-wheel-ctk-name-gen ${RAPIDS_CUDA_VERSION})"

Expand All @@ -25,12 +25,8 @@ pyproject_file="${package_dir}/pyproject.toml"
init_file="${package_dir}/${package_name}/__init__.py"

sed -i "s/^name = \"cuml\"/name = \"cuml${PACKAGE_CUDA_SUFFIX}\"/g" ${pyproject_file}
<<<<<<< Updated upstream
sed -i "s/__version__ = .*/__version__ = ${version_override}/g" ${init_file}
=======
sed -i "s/__version__ = .*/__version__ = \"${version}\"/g" ${init_file}
>>>>>>> Stashed changes
sed -i "s/__git_commit__ = .*/__git_commit__ = \"${commit_override}\"/g" ${init_file}
sed -i "s/__git_commit__ = .*/__git_commit__ = \"${git_commit}\"/g" ${init_file}

# For nightlies we want to ensure that we're pulling in alphas as well. The
# easiest way to do so is to augment the spec with a constraint containing a
Expand Down

0 comments on commit ac28646

Please sign in to comment.