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

Add release workflow and complementary files #1

Merged
merged 21 commits into from
Feb 18, 2022
Merged

Conversation

umbynos
Copy link
Contributor

@umbynos umbynos commented Feb 16, 2022

With this PR I'm adding:

  • release.yml which is the release workflow to publish compiled imgtool version 1.8.0 to the release section here on GitHub and on the Arduino download servers. The workflow uses pyinstaller to generate an executable from a python project. The workflow leverage native compilation for win, macos, linux64. The binaries for the other platforms (linux32, linux-armv6, linux-armv7, linux-arm64) are built using crosscompilation, qemu, and docker containers.
  • patches/ it's a folder containing all the patches generated by us here
  • README.md with general info regarding the repo and imgtool

@umbynos umbynos self-assigned this Feb 16, 2022
@umbynos
Copy link
Contributor Author

umbynos commented Feb 16, 2022

Release Example
image

@umbynos umbynos requested a review from a team February 16, 2022 11:19
@umbynos umbynos force-pushed the umbynos/add_workflow branch from dc6552f to 4155b72 Compare February 16, 2022 11:34
umbynos and others added 5 commits February 16, 2022 12:38
0009 is used to patch cryptography version used because of: healthchecks/healthchecks#565
[TODO] remove hardcoded version and use ${GITHUB_REF/refs\/tags\//}

add bash as default shell to find zip on win

use 7zip on win to archive, zip is not installed by default

remove ${{ github.workspace }} from win, it does not get expanded correctly
tar in macos is not working correctly on the ghactions hosted runners actions/runner-images#2619
@umbynos umbynos force-pushed the umbynos/add_workflow branch from 4155b72 to c5a94da Compare February 16, 2022 11:39
@umbynos umbynos force-pushed the umbynos/add_workflow branch from c5a94da to 4483ec0 Compare February 16, 2022 14:34
@umbynos umbynos requested a review from a team February 16, 2022 14:34
@per1234 per1234 self-requested a review February 16, 2022 15:00
Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

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

I have one other suggestion, but it is not well suited for the PR review suggestions system, so I'll submit that as a PR against your PR branch once the suggestions here have been settled.

.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
build.sh Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
build.sh Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
@umbynos umbynos force-pushed the umbynos/add_workflow branch from 640fb48 to 584c2ab Compare February 17, 2022 11:35
@umbynos umbynos force-pushed the umbynos/add_workflow branch from 1ce15ed to 2961b14 Compare February 18, 2022 09:37
@umbynos umbynos requested a review from per1234 February 18, 2022 13:39
Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

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

Great work Umberto!

The other idea I had was to use the build script in the build job to make the system easier to maintain by avoiding code duplication between the two jobs. However, I found that it was difficult to achieve compatibility of the script in all platforms, so I gave up on the idea.

@umbynos umbynos merged commit bfffe92 into master Feb 18, 2022
@umbynos umbynos deleted the umbynos/add_workflow branch February 18, 2022 17:41
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