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

build: Always pass architecture to actions #261

Merged
merged 2 commits into from
Feb 13, 2024

Conversation

chantra
Copy link
Contributor

@chantra chantra commented Feb 12, 2024

In preparation of cross-compiling, ensure that we always pass arch parameter to the action.
At a later stage, this parameter will be made required on libbpf-ci.

In a previous push, I forgot to run `black` on a file I modified.
Thje linter has been unhappy since... Fix this.

Signed-off-by: Manu Bretelle <chantr4@gmail.com>
In preparation of cross-compiling, ensure that we always pass arch parameter
to the action.
At a later stage, this parameter will be made required on libbpf-ci.

Signed-off-by: Manu Bretelle <chantr4@gmail.com>
@chantra
Copy link
Contributor Author

chantra commented Feb 13, 2024

https://github.com/kernel-patches/vmtest/actions/runs/7879475334/job/21499804418?pr=261#step:11:4

shows that we get a warning for the new parameter, but other than that, libbpf/ci happily ignore the parameter.

@chantra chantra merged commit 95bdc48 into kernel-patches:master Feb 13, 2024
13 of 30 checks passed
@chantra chantra deleted the pass_arch branch February 13, 2024 00:17
chantra added a commit to chantra/kernel-patches-vmtest that referenced this pull request Feb 13, 2024
We will use this to find out whether or not to install cross-compilation toolchain.
Similar to kernel-patches#261

Signed-off-by: Manu Bretelle <chantr4@gmail.com>
chantra added a commit that referenced this pull request Feb 13, 2024
We will use this to find out whether or not to install cross-compilation toolchain.
Similar to #261

Signed-off-by: Manu Bretelle <chantr4@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.

1 participant