Skip to content

fix: strip output of `uname -r`

Malte Poll requested to merge malt3/toolchain_utils:fix_uname_r_strip into main

I saw an isue where uname -r contains an extra newline on some machines. This leads to the code trying to parse an int from an invalid string:

Repository rule triplet defined at:
  /.../external/toolchain_utils~/toolchain/local/triplet/repository.bzl:33:26: in <toplevel>
ERROR: /.../external/toolchain_utils~/toolchain/local/triplet/os.bzl:126:82: An error occurred during the fetch of repository 'toolchain_utils~~_repo_rules~local':
   Traceback (most recent call last):
	File "/.../external/toolchain_utils~/toolchain/local/triplet/repository.bzl", line 24, column 25, in implementation
		triplet = detect(rctx).value
	File "/.../external/toolchain_utils~/toolchain/local/triplet/detect.bzl", line 11, column 11, in detect
		os(rctx).value,
	File "/.../external/toolchain_utils~/toolchain/local/triplet/os.bzl", line 181, column 22, in os
		return _uname(rctx, path)
	File "/.../external/toolchain_utils~/toolchain/local/triplet/os.bzl", line 126, column 82, in _uname
		return VersionedInfo("{}.{}.{}.{}".format(kernel, int(major), int(minor), int(patch)))
Error in int: invalid base-10 literal: "114\n"
Edited by Malte Poll

Merge request reports