X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/4031e38a48822b543e95f166003ecb01906e49dc..7ff8f285eb1ff374091e2d8e8e23f36d8b15a775:/build/run-library.sh diff --git a/build/run-library.sh b/build/run-library.sh index cb5549df71..f8e5129dae 100755 --- a/build/run-library.sh +++ b/build/run-library.sh @@ -40,27 +40,23 @@ EOF format_last_commit_here() { local format="$1"; shift - TZ=UTC git log -n1 --first-parent "--format=format:$format" . + local dir="${1:-.}"; shift + TZ=UTC git log -n1 --first-parent "--format=format:$format" "$dir" } version_from_git() { # Output the version being built, or if we're building a # dev/prerelease, output a version number based on the git log for - # the current working directory. + # the given $subdir. + local subdir="$1"; shift if [[ -n "$ARVADOS_BUILDING_VERSION" ]]; then echo "$ARVADOS_BUILDING_VERSION" return fi - local git_ts git_hash prefix - if [[ -n "$1" ]] ; then - prefix="$1" - else - prefix="0.1" - fi - - declare $(format_last_commit_here "git_ts=%ct git_hash=%h") - ARVADOS_BUILDING_VERSION="$(git tag -l |sort -V -r |head -n1).$(date -ud "@$git_ts" +%Y%m%d%H%M%S)" + local git_ts git_hash + declare $(format_last_commit_here "git_ts=%ct git_hash=%h" "$subdir") + ARVADOS_BUILDING_VERSION="$($WORKSPACE/build/version-at-commit.sh $git_hash)" echo "$ARVADOS_BUILDING_VERSION" } @@ -69,11 +65,24 @@ nohash_version_from_git() { echo "$ARVADOS_BUILDING_VERSION" return fi - version_from_git $1 | cut -d. -f1-4 + version_from_git | cut -d. -f1-4 } timestamp_from_git() { - format_last_commit_here "%ct" + local subdir="$1"; shift + format_last_commit_here "%ct" "$subdir" +} + +calculate_python_sdk_cwl_package_versions() { + python_sdk_ts=$(cd sdk/python && timestamp_from_git) + cwl_runner_ts=$(cd sdk/cwl && timestamp_from_git) + + python_sdk_version=$(cd sdk/python && nohash_version_from_git) + cwl_runner_version=$(cd sdk/cwl && nohash_version_from_git) + + if [[ $python_sdk_ts -gt $cwl_runner_ts ]]; then + cwl_runner_version=$python_sdk_version + fi } handle_python_package () { @@ -108,32 +117,32 @@ calculate_go_package_version() { # to another variable that is passed in as the first argument to this function. # see https://www.gnu.org/software/bash/manual/html_node/Shell-Parameters.html local -n __returnvar="$1"; shift - local src_path="$1"; shift - - mkdir -p "$GOPATH/src/git.curoverse.com" - ln -sfn "$WORKSPACE" "$GOPATH/src/git.curoverse.com/arvados.git" - (cd "$GOPATH/src/git.curoverse.com/arvados.git" && "$GOPATH/bin/govendor" sync -v) + local oldpwd="$PWD" - cd "$GOPATH/src/git.curoverse.com/arvados.git/$src_path" - local version="$(version_from_git)" - local timestamp="$(timestamp_from_git)" + cd "$WORKSPACE" + go mod download # Update the version number and build a new package if the vendor # bundle has changed, or the command imports anything from the # Arvados SDK and the SDK has changed. - declare -a checkdirs=(vendor) - if grep -qr git.curoverse.com/arvados .; then + declare -a checkdirs=(go.mod go.sum) + while [ -n "$1" ]; do + checkdirs+=("$1") + shift + done + if grep -qr git.arvados.org/arvados .; then checkdirs+=(sdk/go lib) fi + local timestamp=0 for dir in ${checkdirs[@]}; do - cd "$GOPATH/src/git.curoverse.com/arvados.git/$dir" - ts="$(timestamp_from_git)" + cd "$WORKSPACE" + ts="$(timestamp_from_git "$dir")" if [[ "$ts" -gt "$timestamp" ]]; then - version=$(version_from_git) + version=$(version_from_git "$dir") timestamp="$ts" fi done - + cd "$oldpwd" __returnvar="$version" } @@ -164,7 +173,7 @@ package_go_binary() { return 1 fi - go get -ldflags "-X main.version=${go_package_version}" "git.curoverse.com/arvados.git/$src_path" + go get -ldflags "-X git.arvados.org/arvados.git/lib/cmd.version=${go_package_version} -X main.version=${go_package_version}" "git.arvados.org/arvados.git/$src_path" local -a switches=() systemd_unit="$WORKSPACE/${src_path}/${prog}.service" @@ -209,6 +218,20 @@ _build_rails_package_scripts() { done } +rails_package_version() { + local pkgname="$1"; shift + local srcdir="$1"; shift + if [[ -n "$ARVADOS_BUILDING_VERSION" ]]; then + echo "$ARVADOS_BUILDING_VERSION" + return + fi + local version="$(version_from_git)" + if [ $pkgname = "arvados-api-server" -o $pkgname = "arvados-workbench" ] ; then + calculate_go_package_version version cmd/arvados-server "$srcdir" + fi + echo $version +} + test_rails_package_presence() { local pkgname="$1"; shift local srcdir="$1"; shift @@ -221,7 +244,7 @@ test_rails_package_presence() { cd $srcdir - local version="$(version_from_git)" + local version="$(rails_package_version "$pkgname" "$srcdir")" cd $tmppwd @@ -299,7 +322,9 @@ test_package_presence() { # sure it gets picked up by the test and/or upload steps. # Get the list of packages from the repos - if [[ "$FORMAT" == "deb" ]]; then + if [[ "$FORCE_BUILD" == "1" ]]; then + echo "Package $full_pkgname build forced with --force-build, building" + elif [[ "$FORMAT" == "deb" ]]; then declare -A dd dd[debian9]=stretch dd[debian10]=buster @@ -312,11 +337,11 @@ test_package_presence() { repo_subdir=${pkgname:0:1} fi - repo_pkg_list=$(curl -s -o - http://apt.arvados.org/pool/${D}/main/${repo_subdir}/) - echo ${repo_pkg_list} |grep -q ${full_pkgname} + repo_pkg_list=$(curl -s -o - http://apt.arvados.org/pool/${D}-dev/main/${repo_subdir}/${pkgname}/) + echo "${repo_pkg_list}" |grep -q ${full_pkgname} if [ $? -eq 0 ] ; then echo "Package $full_pkgname exists upstream, not rebuilding, downloading instead!" - curl -s -o "$WORKSPACE/packages/$TARGET/${full_pkgname}" http://apt.arvados.org/pool/${D}/main/${repo_subdir}/${full_pkgname} + curl -s -o "$WORKSPACE/packages/$TARGET/${full_pkgname}" http://apt.arvados.org/pool/${D}-dev/main/${repo_subdir}/${pkgname}/${full_pkgname} return 1 elif test -f "$WORKSPACE/packages/$TARGET/processed/${full_pkgname}" ; then echo "Package $full_pkgname exists, not rebuilding!" @@ -353,7 +378,7 @@ handle_rails_package() { local srcdir="$1"; shift cd "$srcdir" local license_path="$1"; shift - local version="$(version_from_git)" + local version="$(rails_package_version "$pkgname" "$srcdir")" echo "$version" >package-build.version local scripts_dir="$(mktemp --tmpdir -d "$pkgname-XXXXXXXX.scripts")" && \ ( @@ -451,7 +476,7 @@ fpm_build_virtualenv () { fi # arvados-python-client sdist should always be built, to be available - # for other dependant packages. + # for other dependent packages. if [[ -n "$ONLY_BUILD" ]] && [[ "arvados-python-client" != "$PKG" ]] && [[ "$PYTHON_PKG" != "$ONLY_BUILD" ]] && [[ "$PKG" != "$ONLY_BUILD" ]]; then return 0 fi @@ -461,9 +486,9 @@ fpm_build_virtualenv () { rm -rf dist/* # Get the latest setuptools - if ! $pip install $DASHQ_UNLESS_DEBUG $CACHE_FLAG -U setuptools; then + if ! $pip install $DASHQ_UNLESS_DEBUG $CACHE_FLAG -U 'setuptools<45'; then echo "Error, unable to upgrade setuptools with" - echo " $pip install $DASHQ_UNLESS_DEBUG $CACHE_FLAG -U setuptools" + echo " $pip install $DASHQ_UNLESS_DEBUG $CACHE_FLAG -U 'setuptools<45'" exit 1 fi # filter a useless warning (when building the cwltest package) from the stderr output @@ -518,9 +543,9 @@ fpm_build_virtualenv () { fi echo "pip version: `build/usr/share/$python/dist/$PYTHON_PKG/bin/$pip --version`" - if ! build/usr/share/$python/dist/$PYTHON_PKG/bin/$pip install $DASHQ_UNLESS_DEBUG $CACHE_FLAG -U setuptools; then + if ! build/usr/share/$python/dist/$PYTHON_PKG/bin/$pip install $DASHQ_UNLESS_DEBUG $CACHE_FLAG -U 'setuptools<45'; then echo "Error, unable to upgrade setuptools with" - echo " build/usr/share/$python/dist/$PYTHON_PKG/bin/$pip install $DASHQ_UNLESS_DEBUG $CACHE_FLAG -U setuptools" + echo " build/usr/share/$python/dist/$PYTHON_PKG/bin/$pip install $DASHQ_UNLESS_DEBUG $CACHE_FLAG -U 'setuptools<45'" exit 1 fi echo "setuptools version: `build/usr/share/$python/dist/$PYTHON_PKG/bin/$python -c 'import setuptools; print(setuptools.__version__)'`" @@ -549,7 +574,7 @@ fpm_build_virtualenv () { cd build/usr/share/$python/dist/$PYTHON_PKG/ # Replace the shebang lines in all python scripts, and handle the activate - # scripts too This is a functional replacement of the 237 line + # scripts too. This is a functional replacement of the 237 line # virtualenv_tools.py script that doesn't work in python3 without serious # patching, minus the parts we don't need (modifying pyc files, etc). for binfile in `ls bin/`; do @@ -599,7 +624,7 @@ fpm_build_virtualenv () { # 12271 - As FPM-generated packages don't include scripts by default, the # packages cleanup on upgrade depends on files being listed on the %files # section in the generated SPEC files. To remove DIRECTORIES, they need to - # be listed in that sectiontoo, so we need to add this parameter to properly + # be listed in that section too, so we need to add this parameter to properly # remove lingering dirs. But this only works for python2: if used on # python33, it includes dirs like /opt/rh/python33 that belong to # other packages. @@ -607,7 +632,7 @@ fpm_build_virtualenv () { COMMAND_ARR+=('--rpm-auto-add-directories') fi - if [[ "$PKG" == "arvados-python-client" ]]; then + if [[ "$PKG" == "arvados-python-client" ]] || [[ "$PKG" == "arvados-fuse" ]]; then if [[ "$python" == "python2.7" ]]; then COMMAND_ARR+=('--conflicts' "$PYTHON3_PKG_PREFIX-$PKG") else @@ -624,7 +649,8 @@ fpm_build_virtualenv () { COMMAND_ARR+=('-n' "$PYTHON_PKG") COMMAND_ARR+=('-C' "build") - if [[ -e "$WORKSPACE/$PKG_DIR/$PKG.service" ]]; then + systemd_unit="$WORKSPACE/$PKG_DIR/$PKG.service" + if [[ -e "${systemd_unit}" ]]; then COMMAND_ARR+=('--after-install' "${WORKSPACE}/build/go-python-package-scripts/postinst") COMMAND_ARR+=('--before-remove' "${WORKSPACE}/build/go-python-package-scripts/prerm") fi @@ -657,6 +683,12 @@ fpm_build_virtualenv () { COMMAND_ARR+=('--depends' "$i") done + # make sure the systemd service file ends up in the right place + # used by arvados-docker-cleaner and arvados-node-manager + if [[ -e "${systemd_unit}" ]]; then + COMMAND_ARR+=("usr/share/$python/dist/$PKG/share/doc/$PKG/$PKG.service=/lib/systemd/system/$PKG.service") + fi + COMMAND_ARR+=("${fpm_args[@]}") # Make sure to install all our package binaries in /usr/bin. @@ -898,7 +930,9 @@ timer_reset() { } timer() { - echo -n "$(($SECONDS - $t0))s" + if [[ -n "$t0" ]]; then + echo -n "$(($SECONDS - $t0))s" + fi } report_outcomes() {