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

fix: added build-base to package list for static build #474

Merged
merged 1 commit into from
Jul 13, 2023

Conversation

ashwing123
Copy link
Contributor

@ashwing123 ashwing123 commented Jul 5, 2023

What type of PR is this?
bug

Which issue does this PR fix:
fixes #473

What does this PR do / Why do we need it:
Enables alpine:edge aarch64 image to finish building (allows it to compile C programs).

If an issue # is not available please add repro steps and logs showing the issue:

Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades?

Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Signed-off-by: Ashwin Gopalan ashwin.gopalan02@gmail.com

@ashwing123 ashwing123 changed the title added build-base to package list for static build fix: added build-base to package list for static build Jul 5, 2023
@hallyn hallyn requested a review from raharper July 6, 2023 14:28
@hallyn
Copy link
Contributor

hallyn commented Jul 6, 2023

Thanks. The commit itself looks good. However, please

  1. Add the justification from the PR description as a patch message, e.g.:

" This enables alpine:edge aarch64 image to finish building (allows it to compile C programs)."

  1. Add your Signed-off-by.
  2. So long as you are editing the message anyway, change the patch subject from past tense to imperative, so 'added build-base...' -> 'add build-base...'

@ashwing123 ashwing123 changed the title fix: added build-base to package list for static build fix: add build-base to package list for static build Jul 6, 2023
@ashwing123 ashwing123 changed the title fix: add build-base to package list for static build fix: add build-base to package list for static build. This enables alpine:edge aarch64 image to finish building (allows it to compile C programs) Jul 6, 2023
@ashwing123 ashwing123 changed the title fix: add build-base to package list for static build. This enables alpine:edge aarch64 image to finish building (allows it to compile C programs) fix: add build-base to package list for static build. Jul 6, 2023
@ashwing123
Copy link
Contributor Author

ashwing123 commented Jul 6, 2023

Added commit message and PR description/subject changes (edit: sorry forgot some other metadata and was having issues with my keyboard copy paste hence the title changes... Fixed below)

@ashwing123 ashwing123 changed the title fix: add build-base to package list for static build. fix: added build-base to package list for static build Jul 6, 2023
@ashwing123 ashwing123 force-pushed the add-build-base branch 3 times, most recently from bc3cf20 to b3a428f Compare July 6, 2023 17:34
Copy link
Contributor

@hallyn hallyn left a comment

Choose a reason for hiding this comment

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

Thanks!

@hallyn
Copy link
Contributor

hallyn commented Jul 13, 2023

Note this is not currently mergeable as "The base branch requires all commits to be signed"

@ashwing123
Copy link
Contributor Author

ashwing123 commented Jul 13, 2023

Let me check on that

@ashwing123 ashwing123 force-pushed the add-build-base branch 2 times, most recently from 63423e0 to 9c85059 Compare July 13, 2023 20:00
This enables alpine:edge aarch64 image to finish building (allows it to compile C programs).
Fixes: project-stacker#473

Signed-off-by: Ashwin Gopalan <ashwin.gopalan02@gmail.com>
@rchincha rchincha merged commit 7550d26 into project-stacker:main Jul 13, 2023
7 checks passed
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.

Bug: alpine:edge aarch64 image doesn't compile c-programs
3 participants