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

WIP: Script to rebuild a package from buildinfo #433

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

legoktm
Copy link
Member

@legoktm legoktm commented Mar 24, 2023

No description provided.

@legoktm
Copy link
Member Author

legoktm commented Mar 24, 2023

rebuilding securedrop-client 0.9.0 with this basically worked: https://gist.github.com/legoktm/3352b54b373bc9f186f555a087a6d3a4 - all the diffs are known issues.

Remaining work:

  • We need to capture the git commit sha1 of the repository being packaged as well as securedrop-builder.
  • Presumably the correct order of operations is something like:
    • parse buildinfo, check out correct version of securedrop-builder (in place? somewhere else?)
    • Run make install-deps to install apt packages and bootstrap
    • Set up snapshot apt config, force install correct versions of packages (hope that python, etc. didn't materially change and the bootstrap doesn't need to be recreated??)
    • Build the package
    • Print diff of the buildinfo (at least build date should be different)
    • Print diffoscope of the debs (ideally no differences)

I don't think we're yet at the point of automating this, but we should try it during release time to verify the buildinfo/package.

In freedomofpress/securedrop#6356 (comment) I explained how we can add in more environment variables. I think we should prefix them with "SD_", so we probably want to rename PKG_GITREF, etc. to use that. And then probably some new SD_BUILDER_GIT_VERSION

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.

1 participant