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

test and generate release binaries #226

Merged
merged 1 commit into from
Feb 6, 2022
Merged

Conversation

alaviss
Copy link
Contributor

@alaviss alaviss commented Feb 3, 2022

Summary

This change facilitates the generation of source archive as well as
release archives in nimskull's CI. By integrating with main CI, we
test the binaries that will be released directly so they are
guaranteed to work.

Details

There are several changes packed in this:

  • CI now build release compiler and tools, the same kind in a release
    archive. This is to make sure that we test the exact binaries that we
    ship.

    • To help with this, new subcommand all is added to koch which
      builds binaries the same way as a release build done by
      unixrelease would.
  • Source archive and release binaries will be packaged and uploaded to
    artifacts. This allow for testing of these if required and will be
    picked up and published via the continous delivery pipeline.

  • A new test has been added: Verifying that the same release binaries
    can be built from git and source archive. This test is done via
    diffoscope, so in the event that it doesn't pass, a comprehensive
    report will be uploaded to help trace the problem.

There are several changes packed in this:

- CI now build release compiler and tools, the same kind in a release
  archive. This is to make sure that we test the exact binaries that we
  ship.

  - To help with this, new subcommand `all` is added to `koch` which
    builds binaries the same way as a release build done by
    `unixrelease` would.

- Source archive and release binaries will be packaged and uploaded to
  artifacts. This allow for testing of these if required and will be
  picked up and published via the continous delivery pipeline.

- A new test has been added: Verifying that the same release binaries
  can be built from git and source archive. This test is done via
  diffoscope, so in the event that it doesn't pass, a comprehensive
  report will be uploaded to help trace the problem.
@alaviss alaviss requested a review from saem February 3, 2022 21:39
Copy link
Collaborator

@saem saem left a comment

Choose a reason for hiding this comment

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

bors r+

@saem
Copy link
Collaborator

saem commented Feb 5, 2022

bors r+

(what happened buddy?)

@saem
Copy link
Collaborator

saem commented Feb 5, 2022

bors r+

(come on buddy_

bors bot added a commit that referenced this pull request Feb 5, 2022
226: test and generate release binaries r=saem a=alaviss

## Summary

This change facilitates the generation of source archive as well as
release archives in nimskull's CI. By integrating with main CI, we
test the binaries that will be released directly so they are
guaranteed to work.

## Details

There are several changes packed in this:

- CI now build release compiler and tools, the same kind in a release
  archive. This is to make sure that we test the exact binaries that we
  ship.

  - To help with this, new subcommand `all` is added to `koch` which
    builds binaries the same way as a release build done by
    `unixrelease` would.

- Source archive and release binaries will be packaged and uploaded to
  artifacts. This allow for testing of these if required and will be
  picked up and published via the continous delivery pipeline.

- A new test has been added: Verifying that the same release binaries
  can be built from git and source archive. This test is done via
  diffoscope, so in the event that it doesn't pass, a comprehensive
  report will be uploaded to help trace the problem.

Co-authored-by: Leorize <[email protected]>
@bors
Copy link
Contributor

bors bot commented Feb 6, 2022

Already running a review

@bors
Copy link
Contributor

bors bot commented Feb 6, 2022

Timed out.

@bors
Copy link
Contributor

bors bot commented Feb 6, 2022

Build succeeded:

@bors bors bot merged commit ea627e8 into nim-works:devel Feb 6, 2022
@alaviss alaviss deleted the build-release branch February 6, 2022 20:59
@haxscramper haxscramper added the ci Continuous Integration label Nov 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous Integration
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants