X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/366a2efdd0ac4630f4381f3b47d70ef155ed2df4..741b677dc5e85f60bc03ef130873e49ac0b75766:/build/run-library.sh diff --git a/build/run-library.sh b/build/run-library.sh index af95cdc601..1971a33d05 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 () { @@ -104,37 +113,40 @@ handle_ruby_gem() { } calculate_go_package_version() { + # $__returnvar has the nameref attribute set, which means it is a reference + # 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 + local oldpwd="$PWD" - 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) - - 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" } -# Usage: package_go_binary services/foo arvados-foo "Compute foo to arbitrary precision" +# Usage: package_go_binary services/foo arvados-foo "Compute foo to arbitrary precision" [apache-2.0.txt] package_go_binary() { local src_path="$1"; shift local prog="$1"; shift @@ -161,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" @@ -176,6 +188,33 @@ package_go_binary() { fpm_build "$GOPATH/bin/${basename}=/usr/bin/${prog}" "${prog}" dir "${go_package_version}" "--url=https://arvados.org" "--license=GNU Affero General Public License, version 3.0" "--description=${description}" "${switches[@]}" } +# Usage: package_go_so lib/foo arvados_foo.so arvados-foo "Arvados foo library" +package_go_so() { + local src_path="$1"; shift + local sofile="$1"; shift + local pkg="$1"; shift + local description="$1"; shift + + debug_echo "package_go_so $src_path as $pkg" + + calculate_go_package_version go_package_version $src_path + cd $WORKSPACE/packages/$TARGET + test_package_presence $pkg $go_package_version go || return 1 + cd $WORKSPACE/$src_path + go build -buildmode=c-shared -o ${GOPATH}/bin/${sofile} + cd $WORKSPACE/packages/$TARGET + local -a fpmargs=( + "--url=https://arvados.org" + "--license=Apache License, Version 2.0" + "--description=${description}" + "$WORKSPACE/apache-2.0.txt=/usr/share/doc/$pkg/apache-2.0.txt" + ) + if [[ -e "$WORKSPACE/$src_path/pam-configs-arvados" ]]; then + fpmargs+=("$WORKSPACE/$src_path/pam-configs-arvados=/usr/share/pam-configs/arvados-go") + fi + fpm_build "$GOPATH/bin/${sofile}=/usr/lib/${sofile}" "${pkg}" dir "${go_package_version}" "${fpmargs[@]}" +} + default_iteration() { if [[ -n "$ARVADOS_BUILDING_VERSION" ]]; then echo "$ARVADOS_BUILDING_ITERATION" @@ -206,6 +245,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 @@ -218,7 +271,7 @@ test_rails_package_presence() { cd $srcdir - local version="$(version_from_git)" + local version="$(rails_package_version "$pkgname" "$srcdir")" cd $tmppwd @@ -226,6 +279,15 @@ test_rails_package_presence() { } get_complete_package_name() { + # if the errexit flag is set, unset it until this function returns + # otherwise, the shift calls below will abort the program if optional arguments are not supplied + if [ -o errexit ]; then + set +e + trap 'set -e' RETURN + fi + # $__returnvar has the nameref attribute set, which means it is a reference + # 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 pkgname="$1"; shift local version="$1"; shift @@ -287,7 +349,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 @@ -300,11 +364,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!" @@ -341,7 +405,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")" && \ ( @@ -439,7 +503,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 @@ -449,9 +513,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 @@ -506,9 +570,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__)'`" @@ -537,7 +601,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 @@ -587,7 +651,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. @@ -595,7 +659,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 @@ -612,7 +676,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 @@ -645,6 +710,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. @@ -886,7 +957,9 @@ timer_reset() { } timer() { - echo -n "$(($SECONDS - $t0))s" + if [[ -n "$t0" ]]; then + echo -n "$(($SECONDS - $t0))s" + fi } report_outcomes() {