Fix rails package versions for dev builds.
authorTom Clegg <tclegg@veritasgenetics.com>
Thu, 28 Nov 2019 18:04:11 +0000 (13:04 -0500)
committerTom Clegg <tclegg@veritasgenetics.com>
Thu, 28 Nov 2019 18:04:11 +0000 (13:04 -0500)
[ $goversion > $railsversion ] doesn't compare versions, it just
creates an empty file called $railsversion and returns success.  As a
result, the arvados-api-server and arvados-workbench package
numbers (and the rebuild-or-reuse decisions) were based only on the
last change to cmd/arvados-server, disregarding more recent changes in
the respective Rails projects.

refs #15819
refs #15863

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg@veritasgenetics.com>

build/run-library.sh

index f173504c588b58c8e82ce93ade95ab4a8ce9acf3..8857d5b635df803262d3885942794e10adb83430 100755 (executable)
@@ -219,7 +219,7 @@ rails_package_version() {
        local arvados_server_version
        calculate_go_package_version arvados_server_version cmd/arvados-server
        cd $P
-       if [ $arvados_server_version > $version ] ; then
+       if [ $arvados_server_version -gt $version ] ; then
            version=$arvados_server_version
        fi
     fi