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

Add architecture_file parameter to pkg_deb #390

Merged
merged 1 commit into from
Jul 30, 2021

Conversation

tnarg
Copy link
Contributor

@tnarg tnarg commented Jul 26, 2021

RELNOTES: 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.

The architecture_file parameter provides an alternative to the architecture parameter where the contents of the debian package architecture are provided via a file
@tnarg tnarg requested a review from aiuto as a code owner July 26, 2021 20:50
@aiuto
Copy link
Collaborator

aiuto commented Jul 26, 2021

Is there a compelling need for this and an example of how you would use this?

At first glance, what I would prefer is a way to get the architecture dynamically, from something like --cpu?
Could you start an issue to describe the need, and we can come up with the best design.

@tnarg
Copy link
Contributor Author

tnarg commented Jul 26, 2021

I've opened #391 to provide more context around this change

@aiuto
Copy link
Collaborator

aiuto commented Jul 30, 2021

Thanks for filing the issues. It will provide great context for an example of how Go interacts with Bazel and packaging. If you would care to write one for the examples tree, that would be appreciated.

@aiuto aiuto merged commit e189181 into bazelbuild:main Jul 30, 2021
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.

None yet

2 participants