Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

//tools/build_defs/pkg:incompatible_no_build_defs_pkg_deb Remove pkg_deb from Bazel #11217

Closed
aiuto opened this issue Apr 24, 2020 · 7 comments
Closed
Assignees
Labels
incompatible-change Incompatible/breaking change P2 We'll consider working on this in future. (Assignee optional) team-Configurability platforms, toolchains, cquery, select(), config transitions type: feature request

Comments

@aiuto
Copy link
Contributor

aiuto commented Apr 24, 2020

//tools/build_defs/pkg:pkg_deb has moved to https://github.com/bazelbuild/rules_pkg/tree/master/pkg and is no longer maintained as part of Bazel nor used in its build or tests. Users should migrate to the new versions.

Migration:

  1. Add WORKSPACE stanza as specified in a rule_pkg release
    https://github.com/bazelbuild/rules_pkg/releases
  2. Replace instances of
load("//tools/build_defs/pkg:pkg.bzl", "pkg_deb")

with

load("@rules_pkg//:pkg.bzl", "pkg_deb")

One line solution (for some users)

find . -name 'BUILD' -o -name '*.bzl' | xargs sed -i -e '/^load/s#//tools/build_defs/pkg:#@rules_pkg//:#'

Expected timeline:

  • Flag flip at next major Bazel release
  • Code removal by next minor release after that.
@aiuto aiuto self-assigned this Apr 24, 2020
@laurentlb laurentlb added the incompatible-change Incompatible/breaking change label Apr 28, 2020
@jin jin added the team-Configurability platforms, toolchains, cquery, select(), config transitions label May 6, 2020
@siddharthab
Copy link
Contributor

This flag is not yet available in bazel. Can we remove the migration-3.y labels from the issue because bazelisk uses it to check for migration readiness?

@aiuto
Copy link
Contributor Author

aiuto commented Aug 5, 2020

Done. I'm not sure what happened there.

@siddharthab
Copy link
Contributor

Thanks. Can you also remove the labels from #11218?

@aiuto aiuto added P2 We'll consider working on this in future. (Assignee optional) and removed breaking-change-4.0 labels Oct 30, 2020
@aiuto
Copy link
Contributor Author

aiuto commented Oct 30, 2020

This probably won't make the 4.0 cut. In any case, we're just going to drop the code, rather than flag it. People have had deprecation warnings for about a year now and the migration is trivial.

@aiuto
Copy link
Contributor Author

aiuto commented Jan 22, 2021

Status update: Planned removal in March 2021

@aiuto
Copy link
Contributor Author

aiuto commented Feb 10, 2021

Isn't this a "defeature request"? :-)

@aiuto
Copy link
Contributor Author

aiuto commented Feb 10, 2021

Status update:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
incompatible-change Incompatible/breaking change P2 We'll consider working on this in future. (Assignee optional) team-Configurability platforms, toolchains, cquery, select(), config transitions type: feature request
Projects
None yet
Development

No branches or pull requests

7 participants