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

patch: etcher-util is corrupted in RPM package #4333

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

aethernet
Copy link
Contributor

Cherry-picking #4304


rpmbuild strips executables by default when generating an rpm packge. This was causing the JavaScript code bundled in the etcher-util file to be removed, causing "Pkg: Error reading from file." whenever etcher-util was called.

This in turn caused balena-etcher to generate the error message Error: (0, h.requestMetadata) is not a function when attempting to write an SD card.

This fixes the issue for RPM builds by replacing the strip command with true so that rpmbuild no longer strips the executables and the embeded code stays intact.

See: #4150

Copy link

github-actions bot commented Oct 8, 2024

Website deployed to CF Pages, 👀 preview link https://e33cf96b.etcher.pages.dev

Copy link
Member

@dfunckt dfunckt left a comment

Choose a reason for hiding this comment

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

👍

@aethernet aethernet marked this pull request as ready for review October 9, 2024 09:23
rpmbuild strips executables by default when generating an rpm packge.
This was causing the JavaScript code bundled in the etcher-util file
to be removed, causing "Pkg: Error reading from file." whenever
etcher-util was called.

This in turn caused balena-etcher to generate the error message
`Error: (0, h.requestMetadata) is not a function` when attempting
to write an SD card.

This fixes the issue for RPM builds by replacing the `strip` command
with `true` so that rpmbuild no longer strips the executables and
the embeded code stays intact.

See: #4150

Signed-off-by: Richard Glidden <richard@glidden.org>
@aethernet aethernet force-pushed the rglidden/rpm-fix-etcher-util branch from 9915818 to e43ee78 Compare October 9, 2024 13:54
@flowzone-app flowzone-app bot merged commit 87fb4df into master Oct 9, 2024
55 checks passed
@flowzone-app flowzone-app bot deleted the rglidden/rpm-fix-etcher-util branch October 9, 2024 14:21
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.

3 participants