From bd16ab9454e624a26c53a0f3fb892e20fb1c36e6 Mon Sep 17 00:00:00 2001 From: Klaus Aehlig Date: Tue, 15 May 2018 01:42:23 -0700 Subject: http.bzl: drop outdated comment Since cdc99afc1a03ff8fbbbae088d358b7c029e0d232 label arguments are prefetched. So it is no longer necessary to access them early. Change-Id: I6d30bae3ac655387641ffd98f9b9aa95d2f1ece7 PiperOrigin-RevId: 196632853 --- tools/build_defs/repo/http.bzl | 6 ------ 1 file changed, 6 deletions(-) (limited to 'tools/build_defs') diff --git a/tools/build_defs/repo/http.bzl b/tools/build_defs/repo/http.bzl index e39691477f..6659fcbd59 100644 --- a/tools/build_defs/repo/http.bzl +++ b/tools/build_defs/repo/http.bzl @@ -38,12 +38,6 @@ def _http_archive_impl(ctx): if ctx.attr.build_file and ctx.attr.build_file_content: ctx.fail("Only one of build_file and build_file_content can be provided.") - # These print statement is not only for debug, but it also ensures the file - # is referenced before the download is started; this is necessary till a - # proper fix for https://github.com/bazelbuild/bazel/issues/2700 is - # implemented. A proper could, e.g., be to ensure that all ctx.path of - # all the lables provided as arguments are present before the implementation - # function is called the first time. if ctx.attr.build_file: print("ctx.attr.build_file %s, path %s" % (str(ctx.attr.build_file), ctx.path(ctx.attr.build_file))) -- cgit v1.2.3