diff --git a/CMakeLists.txt b/CMakeLists.txt index a9dc38b40b..291a2f7839 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -25,8 +25,8 @@ project(zig C CXX) set(CMAKE_MODULE_PATH ${CMAKE_SOURCE_DIR}/cmake ${CMAKE_MODULE_PATH}) set(ZIG_VERSION_MAJOR 0) -set(ZIG_VERSION_MINOR 7) -set(ZIG_VERSION_PATCH 1) +set(ZIG_VERSION_MINOR 8) +set(ZIG_VERSION_PATCH 0) set(ZIG_VERSION "" CACHE STRING "Override Zig version string. Default is to find out with git.") if("${ZIG_VERSION}" STREQUAL "") @@ -34,18 +34,31 @@ if("${ZIG_VERSION}" STREQUAL "") find_program(GIT_EXE NAMES git) if(GIT_EXE) execute_process( - COMMAND ${GIT_EXE} -C ${CMAKE_SOURCE_DIR} name-rev HEAD --tags --name-only --no-undefined --always + COMMAND ${GIT_EXE} -C ${CMAKE_SOURCE_DIR} describe --match *.*.* --tags RESULT_VARIABLE EXIT_STATUS - OUTPUT_VARIABLE ZIG_GIT_REV + OUTPUT_VARIABLE GIT_DESCRIBE OUTPUT_STRIP_TRAILING_WHITESPACE ERROR_QUIET) if(EXIT_STATUS EQUAL "0") - if(ZIG_GIT_REV MATCHES "\\^0$") - if(NOT("${ZIG_GIT_REV}" STREQUAL "${ZIG_VERSION}^0")) - message("WARNING: Tag does not match configured Zig version") + if(GIT_DESCRIBE MATCHES "^v?([0-9]+\\.[0-9]+\\.[0-9]+)$") + # Tagged release version. + set(GIT_TAG ${CMAKE_MATCH_1}) + if(NOT GIT_TAG VERSION_EQUAL ZIG_VERSION) + message(SEND_ERROR "Zig version (${ZIG_VERSION}) does not match Git tag (${GIT_TAG}).") endif() + elseif(GIT_DESCRIBE MATCHES "^v?([0-9]+\\.[0-9]+\\.[0-9]+)-([0-9]+)-g(.+)$") + # Untagged pre-release. The Zig version is updated to include the number of commits + # since the last tagged version and the commit hash. The version is formatted in + # accordance with the https://semver.org specification. + set(GIT_TAG ${CMAKE_MATCH_1}) + set(GIT_COMMITS_AFTER_TAG ${CMAKE_MATCH_2}) + set(GIT_COMMIT ${CMAKE_MATCH_3}) + if(NOT ZIG_VERSION VERSION_GREATER GIT_TAG) + message(SEND_ERROR "Zig version (${ZIG_VERSION}) must be greater than tagged ancestor (${GIT_TAG}).") + endif() + set(ZIG_VERSION "${ZIG_VERSION}-dev.${GIT_COMMITS_AFTER_TAG}+${GIT_COMMIT}") else() - set(ZIG_VERSION "${ZIG_VERSION}+${ZIG_GIT_REV}") + message(WARNING "Failed to parse version from output of `git describe`.") endif() endif() endif() diff --git a/build.zig b/build.zig index bef3b3b58f..fe588ea2cb 100644 --- a/build.zig +++ b/build.zig @@ -11,7 +11,7 @@ const fs = std.fs; const InstallDirectoryOptions = std.build.InstallDirectoryOptions; const assert = std.debug.assert; -const zig_version = std.builtin.Version{ .major = 0, .minor = 7, .patch = 1 }; +const zig_version = std.builtin.Version{ .major = 0, .minor = 8, .patch = 0 }; pub fn build(b: *Builder) !void { b.setPreferredReleaseMode(.ReleaseFast); @@ -227,24 +227,48 @@ pub fn build(b: *Builder) !void { const version_string = b.fmt("{}.{}.{}", .{ zig_version.major, zig_version.minor, zig_version.patch }); var code: u8 = undefined; - const git_sha_untrimmed = b.execAllowFail(&[_][]const u8{ - "git", "-C", b.build_root, "name-rev", "HEAD", - "--tags", "--name-only", "--no-undefined", "--always", + const git_describe_untrimmed = b.execAllowFail(&[_][]const u8{ + "git", "-C", b.build_root, "describe", "--match", "*.*.*", "--tags", }, &code, .Ignore) catch { break :v version_string; }; - const git_sha_trimmed = mem.trim(u8, git_sha_untrimmed, " \n\r"); + const git_describe = mem.trim(u8, git_describe_untrimmed, " \n\r"); - // This will look like e.g. "0.7.0^0" for a tag commit. - if (mem.endsWith(u8, git_sha_trimmed, "^0")) { - const git_ver_string = git_sha_trimmed[0 .. git_sha_trimmed.len - 2]; - if (!mem.eql(u8, git_ver_string, version_string)) { - std.debug.print("Expected git tag '{}', found '{}'\n", .{ version_string, git_ver_string }); - std.process.exit(1); - } - break :v b.fmt("{}", .{version_string}); - } else { - break :v b.fmt("{}+{}", .{ version_string, git_sha_trimmed }); + switch (mem.count(u8, git_describe, "-")) { + 0 => { + // Tagged release version (e.g. 0.7.0). + if (!mem.eql(u8, git_describe, version_string)) { + std.debug.print("Zig version '{}' does not match Git tag '{}'\n", .{ version_string, git_describe }); + std.process.exit(1); + } + break :v version_string; + }, + 2 => { + // Untagged development build (e.g. 0.7.0-684-gbbe2cca1a). + var it = mem.split(git_describe, "-"); + const tagged_ancestor = it.next() orelse unreachable; + const commit_height = it.next() orelse unreachable; + const commit_id = it.next() orelse unreachable; + + const ancestor_ver = try std.builtin.Version.parse(tagged_ancestor); + if (zig_version.order(ancestor_ver) != .gt) { + std.debug.print("Zig version '{}' must be greater than tagged ancestor '{}'\n", .{ zig_version, ancestor_ver }); + std.process.exit(1); + } + + // Check that the commit hash is prefixed with a 'g' (a Git convention). + if (commit_id.len < 1 or commit_id[0] != 'g') { + std.debug.print("Unexpected `git describe` output: {}\n", .{git_describe}); + break :v version_string; + } + + // The version is reformatted in accordance with the https://semver.org specification. + break :v b.fmt("{}-dev.{}+{}", .{ version_string, commit_height, commit_id[1..] }); + }, + else => { + std.debug.print("Unexpected `git describe` output: {}\n", .{git_describe}); + break :v version_string; + }, } }; exe.addBuildOption([:0]const u8, "version", try b.allocator.dupeZ(u8, version)); diff --git a/ci/azure/linux_script b/ci/azure/linux_script index bf338ce22f..251c3bfaf1 100755 --- a/ci/azure/linux_script +++ b/ci/azure/linux_script @@ -49,6 +49,8 @@ PATH=$PWD/$WASMTIME:$PATH # Make the `zig version` number consistent. # This will affect the cmake command below. git config core.abbrev 9 +git fetch --unshallow || true +git fetch --tags export CC=gcc-7 export CXX=g++-7 diff --git a/ci/azure/macos_script b/ci/azure/macos_script index cb14b7f119..b83f9c940b 100755 --- a/ci/azure/macos_script +++ b/ci/azure/macos_script @@ -28,6 +28,8 @@ cd $ZIGDIR # Make the `zig version` number consistent. # This will affect the cmake command below. git config core.abbrev 9 +git fetch --unshallow || true +git fetch --tags mkdir build cd build diff --git a/ci/azure/windows_msvc_script.bat b/ci/azure/windows_msvc_script.bat index efc68e5d2d..cd5e3d5bca 100644 --- a/ci/azure/windows_msvc_script.bat +++ b/ci/azure/windows_msvc_script.bat @@ -18,6 +18,8 @@ call "C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\VC\Auxiliar REM Make the `zig version` number consistent. REM This will affect the cmake command below. git.exe config core.abbrev 9 +git.exe fetch --unshallow +git.exe fetch --tags mkdir %ZIGBUILDDIR% cd %ZIGBUILDDIR% diff --git a/ci/drone/linux_script b/ci/drone/linux_script index fdc1704fb7..422949e607 100755 --- a/ci/drone/linux_script +++ b/ci/drone/linux_script @@ -14,6 +14,8 @@ pip3 install s3cmd # Make the `zig version` number consistent. # This will affect the cmake command below. git config core.abbrev 9 +git fetch --unshallow || true +git fetch --tags mkdir build cd build diff --git a/ci/srht/freebsd_script b/ci/srht/freebsd_script index 02849da8de..337d715b7c 100755 --- a/ci/srht/freebsd_script +++ b/ci/srht/freebsd_script @@ -20,6 +20,8 @@ cd $ZIGDIR # Make the `zig version` number consistent. # This will affect the cmake command below. git config core.abbrev 9 +git fetch --unshallow || true +git fetch --tags # SourceHut reports that it is a terminal that supports escape codes, but it # is a filthy liar. Here we tell Zig to not try to send any terminal escape diff --git a/src/config.zig.in b/src/config.zig.in index 871c6e9abf..9d16cf3824 100644 --- a/src/config.zig.in +++ b/src/config.zig.in @@ -1,11 +1,6 @@ pub const have_llvm = true; pub const version: [:0]const u8 = "@ZIG_VERSION@"; -pub const semver: @import("std").SemanticVersion = .{ - .major = @ZIG_VERSION_MAJOR@, - .minor = @ZIG_VERSION_MINOR@, - .patch = @ZIG_VERSION_PATCH@, - .build = "@ZIG_GIT_REV@", -}; +pub const semver = try @import("std").SemanticVersion.parse(version); pub const log_scopes: []const []const u8 = &[_][]const u8{}; pub const zir_dumps: []const []const u8 = &[_][]const u8{}; pub const enable_tracy = false;