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

Bump codec2 to 1.0.0. #204

Merged
merged 1 commit into from
Aug 5, 2021
Merged

Bump codec2 to 1.0.0. #204

merged 1 commit into from
Aug 5, 2021

Conversation

tmiw
Copy link
Collaborator

@tmiw tmiw commented Aug 4, 2021

Bumps version number of Codec2 to 1.0.0 and includes "make dist" changes from drowe67/freedv-gui#152.

@tmiw
Copy link
Collaborator Author

tmiw commented Aug 4, 2021

@drowe67, were there any other PRs that should get in prior to merging this one? Also, what should LPCNet be bumped to if anything?

@drowe67
Copy link
Owner

drowe67 commented Aug 4, 2021

Thanks @tmiw, nothing else I'd like to merge at this stage, Hmm, not sure about LPCNet, good question 🤔

@hobbes1069 - I think we have Codec 2 repo ready for the 1.0 release. It's been tested for a while and no further bugs have shown up.

@tmiw
Copy link
Collaborator Author

tmiw commented Aug 5, 2021

Hmm, should the stm32 fixes get in too, or is 1.0 intended to be a release just for regular PCs?

@drowe67
Copy link
Owner

drowe67 commented Aug 5, 2021

I'm inclined to leave them until afterwards. We can also make a new firmware release and update the stm32 README.md

@drowe67
Copy link
Owner

drowe67 commented Aug 5, 2021

@tmiw so I think we are good for a v1.0 release. Do you agree? We've had quite a bit of testing via your recent freedv-gui test releases, and it's been some time since we've found a bug in libcodec2.

If you are happy I'll merge this PR, and add a tag 🙂 ?

@tmiw
Copy link
Collaborator Author

tmiw commented Aug 5, 2021

@drowe67, sounds good. 👍

@drowe67 drowe67 merged commit c57bbf6 into master Aug 5, 2021
@drowe67 drowe67 deleted the ms-packaging branch August 20, 2022 22: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.

2 participants