Skip to content

Add p4c 1.2.5.11#7761

Merged
keith merged 1 commit into
bazelbuild:mainfrom
smolkaj:add/p4c-1.2.5.11
Mar 3, 2026
Merged

Add p4c 1.2.5.11#7761
keith merged 1 commit into
bazelbuild:mainfrom
smolkaj:add/p4c-1.2.5.11

Conversation

@smolkaj

@smolkaj smolkaj commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

Description

Adds p4c v1.2.5.11 — the P4_16 reference compiler — to the BCR. This is the first p4c submission, made possible by the Bzlmod migration completed in this release (p4lang/p4c#5454, p4lang/p4c#5497).

Module

Notes

  • The inja dependency is not yet in the BCR and is loaded via a module extension defined in p4c's own bazel/repositories.bzl. This is self-contained and requires no BCR changes.
  • The p4c_ir_extensions module extension provides an opt-in mechanism for downstream users to register custom IR extensions; it has no effect unless explicitly used.
  • Presubmit builds @p4c//... on Ubuntu 22.04 and 24.04 against Bazel 7.x, 8.x, and 9.x.

Checklist

  • Module added under modules/p4c/
  • metadata.json with homepage, maintainers, and repository
  • source.json pointing to the release tarball with verified SHA-256 integrity
  • MODULE.bazel with correct version
  • presubmit.yml covering Ubuntu 22.04/24.04 × Bazel 7/8/9

@bazel-io

bazel-io commented Mar 2, 2026

Copy link
Copy Markdown
Member

Hello @matthewtlam, @tsnl, modules you maintain (z3) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io

bazel-io commented Mar 2, 2026

Copy link
Copy Markdown
Member

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (p4c) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@smolkaj smolkaj force-pushed the add/p4c-1.2.5.11 branch 3 times, most recently from d45cf0f to 9e2c1b1 Compare March 2, 2026 04:10
@smolkaj smolkaj marked this pull request as ready for review March 2, 2026 04:12
@smolkaj smolkaj force-pushed the add/p4c-1.2.5.11 branch from 9e2c1b1 to f6186a4 Compare March 2, 2026 04:27
@smolkaj

smolkaj commented Mar 2, 2026

Copy link
Copy Markdown
Contributor Author

Would be grateful for a review

@keith

keith commented Mar 2, 2026

Copy link
Copy Markdown
Member

I started CI, looks like you might need a --cxxopt flag to set the min C++ version

matthewtlam
matthewtlam previously approved these changes Mar 2, 2026

@matthewtlam matthewtlam left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @smolkaj

Comment thread modules/p4c/1.2.5.11/MODULE.bazel
@smolkaj smolkaj force-pushed the add/p4c-1.2.5.11 branch from f6186a4 to a80364a Compare March 3, 2026 03:26
@bazel-io bazel-io dismissed matthewtlam’s stale review March 3, 2026 03:26

Require module maintainers' approval for newly pushed changes.

p4c (https://github.com/p4lang/p4c) is the P4_16 reference compiler.
This is the first submission of p4c to the BCR, made possible by the
Bzlmod migration in v1.2.5.11 (p4lang/p4c#5454, p4lang/p4c#5497).

Signed-off-by: Steffen Smolka <steffen.smolka@gmail.com>
@smolkaj smolkaj force-pushed the add/p4c-1.2.5.11 branch from a80364a to de44d4f Compare March 3, 2026 03:26
@smolkaj

smolkaj commented Mar 3, 2026

Copy link
Copy Markdown
Contributor Author

I started CI, looks like you might need a --cxxopt flag to set the min C++ version

Thanks @keith, added.

@keith keith merged commit f332008 into bazelbuild:main Mar 3, 2026
17 checks passed
ileitch pushed a commit to ileitch/bazel-central-registry that referenced this pull request Apr 7, 2026
## Description

Adds [p4c](https://github.com/p4lang/p4c) v1.2.5.11 — the P4_16
reference compiler — to the BCR. This is the first p4c submission, made
possible by the Bzlmod migration completed in this release
([p4lang/p4c#5454](p4lang/p4c#5454),
[p4lang/p4c#5497](p4lang/p4c#5497)).

## Module
- **Name:** `p4c`
- **Version:** `1.2.5.11`
- **Source:** https://github.com/p4lang/p4c/releases/tag/v1.2.5.11

## Notes

- The `inja` dependency is not yet in the BCR and is loaded via a module
extension defined in p4c's own `bazel/repositories.bzl`. This is
self-contained and requires no BCR changes.
- The `p4c_ir_extensions` module extension provides an opt-in mechanism
for downstream users to register custom IR extensions; it has no effect
unless explicitly used.
- Presubmit builds `@p4c//...` on Ubuntu 22.04 and 24.04 against Bazel
7.x, 8.x, and 9.x.

## Checklist

- [x] Module added under `modules/p4c/`
- [x] `metadata.json` with homepage, maintainers, and repository
- [x] `source.json` pointing to the release tarball with verified
SHA-256 integrity
- [x] `MODULE.bazel` with correct version
- [x] `presubmit.yml` covering Ubuntu 22.04/24.04 × Bazel 7/8/9

Signed-off-by: Steffen Smolka <steffen.smolka@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants