Merge branch 'main' into disable-echo-cmd-copy-dir-file-windows

Author: zpervan

CHANGELOG.md

@@ -1,3 +1,26 @@
+Release 1.3.0
+
+**New Features**
+-   Added `dicts.omit` and `dicts.pick` (#304)
+-   Added `expand_template` rule (#330)
+-   Added `subpackages` module (#348)
+-   Added `copy_directory` rule (#366)
+-   Added `bzlmod` support (#385)
+
+**Incompatible Changes**
+-   `native_binary` and `native_test` now use symlinks instead of copying, if
+    the OS supports it (#340)
+-   `native_binary` and `native_test` now build in target configuration instead
+    of host configuration (#341)
+-   `copy_file` no longer adds non-executables to `default_runfiles` (#326)
+
+**Contributors**
+
+Alex Eagle, Alexandre Rostovtsev, Barry McNamara, Derek Cormier, Fabian
+Meumertzheim, Geoffrey Martin-Noble, hchsiao, Kevin Kres, nickgooding,
+Vertexwahn, Vinh Tran
+
+
 Release 1.2.1
 
 Bugfix release: fixes build failure with --incompatible_disallow_empty_glob

MODULE.bazel

@@ -1,7 +1,7 @@
 module(
     name = "bazel_skylib",
     compatibility_level = 1,
-    version = "1.2.1",
+    version = "1.3.0",
 )
 
 register_toolchains(

README.md

@@ -60,8 +60,14 @@ s = shell.quote(p)
 
 * [analysis_test](docs/analysis_test_doc.md)
 * [build_test](docs/build_test_doc.md)
+* [common_settings](docs/common_settings_doc.md)
+* [copy_directory](docs/copy_directory_doc.md)
 * [copy_file](docs/copy_file_doc.md)
+* [diff_test](docs/diff_test_doc.md)
 * [expand_template](docs/expand_template_doc.md)
+* [native_binary and native_test](docs/native_binary_doc.md)
+* [run_binary](docs/run_binary_doc.md)
+* [select_file](docs/select_file_doc.md)
 * [write_file](docs/write_file_doc.md)
 
 ## Writing a new module

docs/BUILD

@@ -3,111 +3,139 @@ load("//docs/private:stardoc_with_diff_test.bzl", "stardoc_with_diff_test", "upd
 licenses(["notice"])
 
 stardoc_with_diff_test(
+    name = "analysis_test",
     bzl_library_target = "//rules:analysis_test",
     out_label = "//docs:analysis_test_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "build_test",
     bzl_library_target = "//rules:build_test",
     out_label = "//docs:build_test_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "collections",
     bzl_library_target = "//lib:collections",
     out_label = "//docs:collections_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "common_settings",
     bzl_library_target = "//rules:common_settings",
     out_label = "//docs:common_settings_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "copy_directory",
     bzl_library_target = "//rules:copy_directory",
     out_label = "//docs:copy_directory_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "copy_file",
     bzl_library_target = "//rules:copy_file",
     out_label = "//docs:copy_file_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "dicts",
     bzl_library_target = "//lib:dicts",
     out_label = "//docs:dicts_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "diff_test",
     bzl_library_target = "//rules:diff_test",
     out_label = "//docs:diff_test_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "expand_template",
     bzl_library_target = "//rules:expand_template",
     out_label = "//docs:expand_template_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "native_binary",
     bzl_library_target = "//rules:native_binary",
     out_label = "//docs:native_binary_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "new_sets",
     bzl_library_target = "//lib:new_sets",
     out_label = "//docs:new_sets_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "partial",
     bzl_library_target = "//lib:partial",
     out_label = "//docs:partial_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "paths",
     bzl_library_target = "//lib:paths",
     out_label = "//docs:paths_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "run_binary",
     bzl_library_target = "//rules:run_binary",
     out_label = "//docs:run_binary_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "selects",
     bzl_library_target = "//lib:selects",
     out_label = "//docs:selects_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "select_file",
+    bzl_library_target = "//rules:select_file",
+    out_label = "//docs:select_file_doc.md",
+)
+
+stardoc_with_diff_test(
+    name = "shell",
     bzl_library_target = "//lib:shell",
     out_label = "//docs:shell_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "structs",
     bzl_library_target = "//lib:structs",
     out_label = "//docs:structs_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "subpackages",
     bzl_library_target = "//lib:subpackages",
     out_label = "//docs:subpackages_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "types",
     bzl_library_target = "//lib:types",
     out_label = "//docs:types_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "unittest",
     bzl_library_target = "//lib:unittest",
     out_label = "//docs:unittest_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "versions",
     bzl_library_target = "//lib:versions",
     out_label = "//docs:versions_doc.md",
 )
 
 stardoc_with_diff_test(
+    name = "write_file",
     bzl_library_target = "//rules:write_file",
     out_label = "//docs:write_file_doc.md",
 )

docs/build_test_doc.md

@@ -36,6 +36,6 @@ Typical usage:
 | :------------- | :------------- | :------------- |
 | <a id="build_test-name"></a>name |  The name of the test rule.   |  none |
 | <a id="build_test-targets"></a>targets |  A list of targets to ensure build.   |  none |
-| <a id="build_test-kwargs"></a>kwargs |  The &lt;a href="https://docs.bazel.build/versions/main/be/common-definitions.html#common-attributes-tests"&gt;common attributes for tests&lt;/a&gt;.   |  none |
+| <a id="build_test-kwargs"></a>kwargs |  The &lt;a href="https://bazel.build/reference/be/common-definitions#common-attributes-tests"&gt;common attributes for tests&lt;/a&gt;.   |  none |
 
 

docs/common_settings_doc.md

@@ -6,7 +6,7 @@ These rules return a BuildSettingInfo with the value of the build setting.
 For label-typed settings, use the native label_flag and label_setting rules.
 
 More documentation on how to use build settings at
-https://docs.bazel.build/versions/main/skylark/config.html#user-defined-build-settings
+https://bazel.build/extending/config#user-defined-build-settings
 
 
 <a id="bool_flag"></a>

docs/diff_test_doc.md

@@ -28,6 +28,6 @@ The test succeeds if the files' contents match.
 | <a id="diff_test-file1"></a>file1 |  Label of the file to compare to &lt;code&gt;file2&lt;/code&gt;.   |  none |
 | <a id="diff_test-file2"></a>file2 |  Label of the file to compare to &lt;code&gt;file1&lt;/code&gt;.   |  none |
 | <a id="diff_test-failure_message"></a>failure_message |  Additional message to log if the files' contents do not match.   |  <code>None</code> |
-| <a id="diff_test-kwargs"></a>kwargs |  The &lt;a href="https://docs.bazel.build/versions/main/be/common-definitions.html#common-attributes-tests"&gt;common attributes for tests&lt;/a&gt;.   |  none |
+| <a id="diff_test-kwargs"></a>kwargs |  The &lt;a href="https://bazel.build/reference/be/common-definitions#common-attributes-tests"&gt;common attributes for tests&lt;/a&gt;.   |  none |
 
 

docs/maintainers_guide.md

@@ -82,7 +82,7 @@ Name 1, Name 2, Name 3 (alphabetically from `git log`)
     can obtain them via `gcloud init`):
 
 ```
-gsutil cp bazel-bin/distro/bazel-skylib-$VERSION.tar.gz gs://bazel-mirror/github.com/bazelbuild/bazel-skylib/releases/download/$VERSION/bazel-skylib-$VERSION.tar.gz
+gsutil cp bazel-bin/distribution/bazel-skylib-$VERSION.tar.gz gs://bazel-mirror/github.com/bazelbuild/bazel-skylib/releases/download/$VERSION/bazel-skylib-$VERSION.tar.gz
 gsutil setmeta -h "Cache-Control: public, max-age=31536000" "gs://bazel-mirror/github.com/bazelbuild/bazel-skylib/releases/download/$VERSION/bazel-skylib-$VERSION.tar.gz"
 ```
 

docs/native_binary_doc.md

@@ -29,7 +29,7 @@ in genrule.tools for example. You can also augment the binary with runfiles.
 | Name  | Description | Type | Mandatory | Default |
 | :------------- | :------------- | :------------- | :------------- | :------------- |
 | <a id="native_binary-name"></a>name |  A unique name for this target.   | <a href="https://bazel.build/docs/build-ref.html#name">Name</a> | required |  |
-| <a id="native_binary-data"></a>data |  data dependencies. See https://docs.bazel.build/versions/main/be/common-definitions.html#typical.data   | <a href="https://bazel.build/docs/build-ref.html#labels">List of labels</a> | optional | [] |
+| <a id="native_binary-data"></a>data |  data dependencies. See https://bazel.build/reference/be/common-definitions#typical.data   | <a href="https://bazel.build/docs/build-ref.html#labels">List of labels</a> | optional | [] |
 | <a id="native_binary-out"></a>out |  An output name for the copy of the binary   | String | required |  |
 | <a id="native_binary-src"></a>src |  path of the pre-built executable   | <a href="https://bazel.build/docs/build-ref.html#labels">Label</a> | required |  |
 
@@ -55,7 +55,7 @@ the binary with runfiles.
 | Name  | Description | Type | Mandatory | Default |
 | :------------- | :------------- | :------------- | :------------- | :------------- |
 | <a id="native_test-name"></a>name |  A unique name for this target.   | <a href="https://bazel.build/docs/build-ref.html#name">Name</a> | required |  |
-| <a id="native_test-data"></a>data |  data dependencies. See https://docs.bazel.build/versions/main/be/common-definitions.html#typical.data   | <a href="https://bazel.build/docs/build-ref.html#labels">List of labels</a> | optional | [] |
+| <a id="native_test-data"></a>data |  data dependencies. See https://bazel.build/reference/be/common-definitions#typical.data   | <a href="https://bazel.build/docs/build-ref.html#labels">List of labels</a> | optional | [] |
 | <a id="native_test-out"></a>out |  An output name for the copy of the binary   | String | required |  |
 | <a id="native_test-src"></a>src |  path of the pre-built executable   | <a href="https://bazel.build/docs/build-ref.html#labels">Label</a> | required |  |
 

docs/private/stardoc_with_diff_test.bzl

@@ -1,3 +1,17 @@
+# Copyright 2022 The Bazel Authors. All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
 """Helpers for keeping stardoc documentation up-to-date.
 
 These are currently a private API in bazel-skylib.
@@ -14,22 +28,23 @@ load("@bazel_skylib//rules:write_file.bzl", "write_file")
 load("@bazel_skylib//rules:diff_test.bzl", "diff_test")
 
 def stardoc_with_diff_test(
+        name,
         bzl_library_target,
         out_label):
     """Creates a stardoc target coupled with a `diff_test` for a given `bzl_library`.
 
     This is helpful for minimizing boilerplate in repos wih lots of stardoc targets.
 
     Args:
+        name: the stardoc target name
         bzl_library_target: the label of the `bzl_library` target to generate documentation for
         out_label: the label of the output MD file
     """
-
     out_file = out_label.replace("//", "").replace(":", "/")
 
     # Generate MD from .bzl
     stardoc(
-        name = out_file.replace("/", "_").replace(".md", "-docgen"),
+        name = name,
         out = out_file.replace(".md", "-docgen.md"),
         input = bzl_library_target + ".bzl",
         deps = [bzl_library_target],