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

chore: remove deprecated gitian #5756

Merged
merged 6 commits into from
Dec 6, 2023
Merged

Conversation

knst
Copy link
Collaborator

@knst knst commented Dec 6, 2023

Issue being fixed or feature implemented

https://github.com/dashpay/dash-issues/issues/61
Deterministic builds with Gitian is deprecated since v20 release.
New releases are switched to Guix

What was done?

Backporting related bitcoin's changes and cleaning up dash specific files:

How Has This Been Tested?

Run command and go through all items:

find . -iname '*gitian*'
grep -ri gitian  | grep -v '.git/' | grep -v 'doc/release-notes/dash/'

Breaking Changes

Gitian is dropped

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation
  • I have assigned this pull request to a milestone

@knst knst added this to the 20.1 milestone Dec 6, 2023
@knst knst marked this pull request as ready for review December 6, 2023 18:05
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

LGTM, utACK

Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

utACK for merge via merge commit

@UdjinM6
Copy link

UdjinM6 commented Dec 6, 2023

should be merge commit, not squash (because of backports)

fanquake and others added 6 commits December 6, 2023 12:40
…r macOS builds

e4c0cad ci, gitian: Drop unneeded python3-dev package for macOS builds (Hennadii Stepanov)

Pull request description:

ACKs for top commit:
  fanquake:
    ACK e4c0cad - gitian builds match and I checked that this doesn't end up installed as a side-effect of another package.

Tree-SHA512: 520a3909b106a0e005b195c5395691edf62b76ee2df43b6971b7aa193648d68e6dac69cb4f1dc474f594b015a2fc2074061865e571d89365174beb5c1780356f
ab9c342 release: remove gitian (fanquake)

Pull request description:

  Note that this doesn't yet touch any glibc back compat related code.

ACKs for top commit:
  laanwj:
    Code review ACK ab9c342

Tree-SHA512: 8e2fe3ec1097f54bb11ab9136b43818d90eab5dbb0a663ad6a552966ada4bdb49cc12ff4e66f0ec0ec5400bda5c81f3a3ce70a9ebb6fe1e0db612da9f00a51a7
…om configure

3d70c05 build: remove faketime unsetting and comments from configure.ac (fanquake)

Pull request description:

  We no-longer use [`faketime`](https://github.com/wolfcw/libfaketime) (it used to be required in gitian), so as far as I'm aware, there is no need for us to unset `FAKETIME` or mention it in our build docs.

ACKs for top commit:
  laanwj:
    Code review ACK 3d70c05
  prusnak:
    Approach ACK 3d70c05
  hebasto:
    ACK 3d70c05, I have reviewed the code and it looks OK, I agree it can be merged.

Tree-SHA512: 9cf89d63b81119f3d2f02975a66ec0b93e861993fdb0e4f70538e3be6e0047dc09ce87ef2de40cbf877647a21706b39ddf07240c77765278d383d7a7878cc7eb
@PastaPastaPasta PastaPastaPasta merged commit e0fb132 into dashpay:develop Dec 6, 2023
4 of 6 checks passed
@UdjinM6
Copy link

UdjinM6 commented Dec 6, 2023

@knst knst deleted the gitian-removal branch December 12, 2023 10:51
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.

4 participants