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

(RHEL-26133) efi: alignment of the PE file has to be at least 512 bytes #242

Merged

Conversation

msekletar
Copy link
Member

@msekletar msekletar commented Feb 29, 2024

https://learn.microsoft.com/en-us/windows/win32/debug/pe-format?redirectedfrom=MSDN#optional-header-windows-specific-fields-image-only

Resolves: RHEL-26133

RHEL-only

[msekleta: this is RHEL-only because upstream no longer uses objcopy to create PE files]

@github-actions github-actions bot added pr/needs-ci Formerly needs-ci pr/needs-review Formerly needs-review labels Feb 29, 2024
Copy link

github-actions bot commented Feb 29, 2024

Commit validation

Tracker - RHEL-26133

The following commits meet all requirements

commit upstream
b4a8e4e - efi: alignment of the PE file has to be at least 512 bytes RHEL-only

Tracker validation

Success

🟢 Tracker RHEL-26133 has set desired product: rhel-9.4.0
🟢 Tracker RHEL-26133 has set desired component: systemd
🟢 Tracker RHEL-26133 has been approved


Pull Request validation

Success

🟢 CI - All checks have passed
🟢 Review - Reviewed by a member
🟢 Approval - Changes were approved


Auto Merge

Success

🟢 Pull Request is not marked as draft and it's not blocked by dont-merge label
🟢 Pull Request meet requirements, title has correct form
🟢 Pull Request meet requirements, mergeable is true
🟢 Pull Request meet requirements, mergeable_state is clean
🟢 Pull Request has correct target branch main
🟢 Pull Request was merged

@github-actions github-actions bot removed the pr/needs-ci Formerly needs-ci label Feb 29, 2024
Copy link
Member

@dtardon dtardon left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot removed the pr/needs-review Formerly needs-review label Mar 1, 2024
@github-actions github-actions bot merged commit 4d3b981 into redhat-plumbers:main Mar 1, 2024
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants