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

package/nwipe: switch from static tarball to Git commit (051e1aa) #322

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Knogle
Copy link
Contributor

@Knogle Knogle commented Jan 25, 2025

This commit updates the nwipe package to build from a specific Git commit instead of using the previously referenced tarball release. The changes include:

  • Setting NWIPE_SITE to the GitHub repository and NWIPE_SITE_METHOD to git.
  • Specifying the commit hash (051e1aa) in NWIPE_VERSION.
  • Removing or emptying the nwipe.hash file, as we no longer verify a tarball and instead rely on the Git checkout.

This approach allows ShredOS/Buildroot to clone the nwipe repository at commit 051e1aa, apply the banner patch, and build the nwipe binary directly from Git.

You can specify the desired commit in the nwipe.mk file.

This commit updates the nwipe package to build from a specific Git commit
instead of using the previously referenced tarball release. The changes
include:

- Setting NWIPE_SITE to the GitHub repository and NWIPE_SITE_METHOD to git.
- Specifying the commit hash (051e1aa) in NWIPE_VERSION.
- Removing or emptying the nwipe.hash file, as we no longer verify a tarball
  and instead rely on the Git checkout.

This approach allows ShredOS/Buildroot to clone the nwipe repository at
commit 051e1aa, apply the banner patch, and build the nwipe binary directly
from Git.
@PartialVolume
Copy link
Owner

Thanks, Looks good, it will save us some time messing about calculating hashes.

I'll give it a try over the next couple of days and then commit.

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