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

Run builds with github actions #3

Merged
merged 1 commit into from
Jan 15, 2020
Merged

Conversation

phw
Copy link
Contributor

@phw phw commented Jan 15, 2020

Here we go, something to get this started.

This builds on Github actions, for now only 64bit macOS and Linux builds. Also upgraded to latest ffmpeg 4.2.2.

I likely won't be able to continue working on this before next week, so I thought I'll drop what I have for now. Windows builds will probably requiring fiddling around with msys, see e.g. actions/starter-workflows#95 .

I did not test much except verifying the executable runs. No idea if fpcalc will build with this. I upgraded to ffmpeg 4.2.2, maybe this will break something. Also the update could mean configure flags might need reexamination, e.g. there might be something new we want to disable.

Artifacts are uploaded, but download is only really feasible manually. What I would like to do is hook this up with Github releases and upload artifacts there on tagged builds. That can easily be added, see https://github.com/metabrainz/picard/blob/master/.github/workflows/package-macos.yml#L71

For now only 64bit macOS and Linux builds. Also upgraded to latest ffmpeg 4.2.2.
@phw phw requested a review from lalinsky January 15, 2020 07:50
@lalinsky
Copy link
Member

Awesome, thank you. I'll fix Windows builds and upload the artifacts to releases.

@lalinsky lalinsky merged commit 7271d48 into acoustid:master Jan 15, 2020
@phw phw deleted the github-actions branch January 15, 2020 07:54
@lalinsky
Copy link
Member

https://github.com/acoustid/ffmpeg-build/releases/tag/v4.2.2-1

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