prepare for a 0.5.1 release (bazelbuild#411)

* prepare for a 0.5.1 release
* use a Label to make print_rel_notes path resolution work
* linty fresh

Author: aiuto

pkg/CHANGELOG.md

@@ -1,3 +1,16 @@
+# Release 0.5.1
+
+## New features
+- Add `pkg_deb(architecture_file)` to provide a way to set the Debian package architecture from the content of a file created at build time.  (#390)
+- Provide `pkg_install` for a "make install"-like experience in Bazel (#380)
+  - Note: This feature is still in development. Read the PR for more information.
+- Initial support for the `pkg_filegroup` framework in `pkg_zip` (#373)
+
+## Closed bugs
+
+- Change the Debian example to reflect standard Debian naming. (#389)
+- pkg_tar(include_runfiles) now works again (#392)
+
 # Release 0.5.0
 
 ## New Features

pkg/private/pkg_files.bzl

@@ -69,6 +69,7 @@ def _check_dest(content_map, dest, src, origin):
     # people specify the owner in one place, but another overly broad glob
     # brings in the file with a different owner.
     if old_entry and old_entry.src != src:
+        # buildifier: disable=print
         print("Duplicate output path: <%s>, declared in %s and %s" % (
             dest,
             origin,

pkg/releasing/defs.bzl

@@ -10,14 +10,17 @@ def print_rel_notes(
         changelog = None,
         mirror_host = None):
     tarball_name = ":%s-%s.tar.gz" % (repo, version)
+    # Must use Label to get a path relative to the rules_pkg repository,
+    # instead of the calling BUILD file.
+    print_rel_notes_helper = Label("//releasing:print_rel_notes")
+    tools = [print_rel_notes_helper]
     cmd = [
-        "$(location //releasing:print_rel_notes)",
+        "$(location %s)" % str(print_rel_notes_helper),
         "--org=%s" % org,
         "--repo=%s" % repo,
         "--version=%s" % version,
         "--tarball=$(location %s)" % tarball_name,
     ]
-    tools = ["//releasing:print_rel_notes"]
     if setup_file:
         cmd.append("--setup_file=%s" % setup_file)
     if deps_method:

pkg/tests/mappings/manifest_test_lib.py

@@ -38,4 +38,4 @@ def assertManifestsMatch(self, expected, got):
         'rules_pkg/tests/mappings/' + got)
     with open(g_file, mode='rb') as g_fp:
       got = json.load(g_fp)
-    self.assertEquals(expected, got)
+    self.assertEqual(expected, got)

pkg/version.bzl

@@ -13,4 +13,4 @@
 # limitations under the License.
 """The version of rules_pkg."""
 
-version = "0.5.0"
+version = "0.5.1"