Skip to content

Introduce KERNEL_MODULE_OFFLINE_SIGNING#1406

Merged
eric-ch merged 1 commit intoOpenXT:masterfrom
jandryuk:offline-signing
Jun 10, 2021
Merged

Introduce KERNEL_MODULE_OFFLINE_SIGNING#1406
eric-ch merged 1 commit intoOpenXT:masterfrom
jandryuk:offline-signing

Conversation

@jandryuk
Copy link
Copy Markdown
Contributor

This came about because reviewing PRs #1395 and #1401 illustrated the fragility of the offline signing support.

Introduce KERNEL_MODULE_OFFLINE_SIGNING to force the user to make an
explicit choice to perform offline signing. Silently creating
un-bootable images isn't user friendly. This variable forces them to
make the choice.

Add various checks for invalid configurations to enforce using the
variable.

Introduce KERNEL_MODULE_OFFLINE_SIGNING to force the user to make an
explicit choice to perform offline signing.  Silently creating
un-bootable images isn't user friendly.  This variable forces them to
make the choice.

Add various checks for invalid configurations to enforce using the
variable.

Signed-off-by: Jason Andryuk <jandryuk@gmail.com>
@eric-ch
Copy link
Copy Markdown
Contributor

eric-ch commented May 24, 2021

Build: $master-custom-28

@eric-ch
Copy link
Copy Markdown
Contributor

eric-ch commented Jun 9, 2021

Build: $master-custom-31

@eric-ch
Copy link
Copy Markdown
Contributor

eric-ch commented Jun 9, 2021

Will merge soon.

@eric-ch eric-ch merged commit a496f81 into OpenXT:master Jun 10, 2021
@jandryuk jandryuk deleted the offline-signing branch July 29, 2021 15:53
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