Skip to content

Commit 823cfb9

Browse files
authored
Merge pull request #17434 from martinholters/semver
Put build number in pre-release version instead of build metadata
2 parents fc9036b + 94e0e88 commit 823cfb9

File tree

3 files changed

+14
-10
lines changed

3 files changed

+14
-10
lines changed

base/version.jl

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -208,13 +208,13 @@ A `VersionNumber` object describing which version of Julia is in use. For detail
208208
const VERSION = try
209209
ver = convert(VersionNumber, VERSION_STRING)
210210
if !isempty(ver.prerelease)
211-
build_number = GIT_VERSION_INFO.build_number
212-
if ver == v"0.5.0-pre"
213-
# due to change of reference for counting commits from last tag to last change of VERSION file
214-
build_number += 5578
211+
if GIT_VERSION_INFO.build_number >= 0
212+
ver = VersionNumber(ver.major, ver.minor, ver.patch, (ver.prerelease..., GIT_VERSION_INFO.build_number), ver.build)
213+
else
214+
println("WARNING: no build number found for pre-release version")
215+
ver = VersionNumber(ver.major, ver.minor, ver.patch, (ver.prerelease..., "unknown"), ver.build)
215216
end
216-
ver = VersionNumber(ver.major, ver.minor, ver.patch, ver.prerelease, (build_number,))
217-
elseif GIT_VERSION_INFO.build_number != 0
217+
elseif GIT_VERSION_INFO.build_number > 0
218218
println("WARNING: ignoring non-zero build number for VERSION")
219219
end
220220
ver

base/version_git.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ fork_master_timestamp=$(git show -s $(git merge-base HEAD $(echo $origin)master)
7373

7474
# Check for errrors and emit default value for missing numbers.
7575
if [ -z "$build_number" ]; then
76-
build_number="0"
76+
build_number="-1"
7777
fi
7878
if [ -z "$fork_master_distance" ]; then
7979
fork_master_distance="-1"

contrib/commit-name.sh

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
#!/bin/sh
22
# This file is a part of Julia. License is MIT: http://julialang.org/license
33

4-
# Need to be run from a julia repo clone
5-
# First argument (Optional) is a ref to the commit
4+
# Needs to be run from a julia repo clone
5+
# First argument (optional) is a ref to the commit
66

77
gitref=${1:-HEAD}
88

@@ -32,7 +32,11 @@ else
3232
nb=$(expr $nb + 5578)
3333
fi
3434
if [ -n "$pre" ]; then
35-
echo "$ver+$nb"
35+
if [ $major = 0 -a $minor -le 5 ]; then
36+
echo "$ver+$nb"
37+
else
38+
echo "$ver.$nb"
39+
fi
3640
else
3741
echo $ver
3842
fi

0 commit comments

Comments
 (0)