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

Revert "Merge pull request #1703 from JonathonHall-Purism/purism_coreboot_24.02.01" #1713

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

tlaurion
Copy link
Collaborator

@tlaurion tlaurion commented Jul 9, 2024

This reverts commit 7025031, reversing changes made to 156d2c8 from #1703 diagnosed and fixed.

This is to prevent other users internally upgrade and brick their computers.


This is interim fix to #1712

…rism_coreboot_24.02.01"

This reverts commit 7025031, reversing
changes made to 156d2c8.

Signed-off-by: Thierry Laurion <insurgo@riseup.net>
@JonathonHall-Purism
Copy link
Collaborator

Working on bisecting the actual problem, but I'll test this branch as soon as CI ROMs are up. If it does boot and I haven't found the actual fix by then, we'll merge it.

Copy link
Collaborator

@JonathonHall-Purism JonathonHall-Purism left a comment

Choose a reason for hiding this comment

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

I built this locally (ended up beating CI) and it does fix the issue, so let's merge it while I figure out what's going on.

What's bizarre though is that the same commit was fine in the PureBoot 30 release, I cherry picked it over there since IIRC it wasn't merged yet upstream, and all tests passed. I'll figure it out, let's revert this for now.

@tlaurion tlaurion merged commit c43b6fc into linuxboot:master Jul 9, 2024
40 of 41 checks passed
@tlaurion
Copy link
Collaborator Author

tlaurion commented Jul 9, 2024

Will revert latest docker image to 0.1.9 now.

tlaurion added a commit to tlaurion/heads that referenced this pull request Jul 10, 2024
…712"

This reverts commit c43b6fc, reversing
changes made to fb9c558.

Signed-off-by: Thierry Laurion <insurgo@riseup.net>
JonathonHall-Purism added a commit to JonathonHall-Purism/heads that referenced this pull request Jul 16, 2024
…712"

This reverts commit c43b6fc, reversing
changes made to fb9c558.

Signed-off-by: Jonathon Hall <jonathon.hall@puri.sm>
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.

2 participants