commit 00b5dadc485f968bced9dc152d9f391a3dc1992b
author szakmary <43033064+szakmary@users.noreply.github.com> Fri May 31 08:43:49 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Fri May 31 08:45:47 2019 -0700
tree 251487bccc5628fa6fd8b3b7414485faf732ea9c
parent 4f133090e80baa9c8e0d7e7e029d3ce0fb7f1f3f

Fix pkg_deb depends_file bug

At version 0.26.0 when depends_file attribute were to used, it generated an error:

file ".../tools/build_defs/pkg/pkg.bzl", line 188, in _pkg_deb_impl
		ctx.file.depends
No attribute 'depends' in file. Make sure there is a label type attribute marked as 'allow_single_file' with this name

This should fix the issue.

Closes #8518.

PiperOrigin-RevId: 250892006

tools/build_defs/pkg/pkg.bzl

diff --git a/tools/build_defs/pkg/pkg.bzl b/tools/build_defs/pkg/pkg.bzl
index 57e24a4..ca12b76 100644
--- a/tools/build_defs/pkg/pkg.bzl
+++ b/tools/build_defs/pkg/pkg.bzl
@@ -191,9 +191,9 @@
         args += ["--built_using=" + ctx.attr.built_using]
 
     if ctx.attr.depends_file:
-        if ctx.file.depends:
+        if ctx.attr.depends:
             fail("Both depends and depends_file attributes were specified")
-        args += ["--depends=@" + ctx.attr.depends_file.path]
+        args += ["--depends=@" + ctx.file.depends_file.path]
         files += [ctx.file.depends_file]
     elif ctx.attr.depends:
         args += ["--depends=" + d for d in ctx.attr.depends]