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

[rubberband] add overlayed rubberband v3 #48

Merged
merged 6 commits into from
Aug 6, 2022

Conversation

davidchocholaty
Copy link

This PR is based on the vcpkg implementation for microsoft/vcpkg by @daschuer for RubberBand v2.0.2 (see #24549). The code was used on 19 Jul. 2022 with the current PR version. It was moved into overlayed ports and replaces with the previous code. At the same time, the RubberBand version was changed to v3.0.0.

dreuter and others added 6 commits July 19, 2022 16:20
[rubberband](https://www.breakfastquay.com/rubberband/) is a small
library for stretching sounds.
This is the minimal portfile to make vcpkg happy, but probably still
needs some improvement, i.e. not picking up system libraries.
This commit removes the rubberband 2.0.2 version from ports/rubberband
to /overlay/ports/rubberband. It replaces the previous version
in /overlay/ports/rubberband 1.9.0 and with that,
the last 3.0.0 version is used.
@daschuer
Copy link
Member

All green, Thank you. I will merge this and issue a PR against main after the prerequisite mixxxdj/mixxx#4855 is merged. Although it would be better if one else takes final look. So if one has interest, please go ahead and review or merge it before. It will not effect our build until the main branch references the new commit hash.

@davidchocholaty
Copy link
Author

Okay, perfect. Thank you.

@daschuer daschuer merged commit 56c7437 into mixxxdj:2.4 Aug 6, 2022
@JoergAtGithub
Copy link
Member

microsoft#24549 (Rubberband 2.0.2) is now merged upstream.

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