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

Ov50 appveyor #128

Closed
wants to merge 5 commits into from
Closed

Conversation

rgleason
Copy link
Contributor

@rgleason rgleason commented Apr 10, 2019

ov50 - Update appveyor to compile for Opencpn 5.0
Had to add the libaudio/config.h removal of the duplicate define snprintf due to the new libraries for ov50

Tested the appveyor file and it works in ov50 except the second screen of the decoder.

Merge this after stelian's changes to fix Windows Decoder etc. which he will make into a PR
Suggested order for merging:

  1. Merged my PR Xml update 04 10 2019 #126 Xml update
  2. Merge Stelian's changes to fix Windows. He will make a PR. (my PR for win changes is closed now)
  3. Merge my PR Ov50 appveyor #128 Ov50 appveyor

This way Sean's repository will have an updated ov42-ov48 with an appveyor artifact that works better and a new ov50 appeveyor for windows artifact as well.

This was referenced Apr 10, 2019
@rgleason
Copy link
Contributor Author

Closing, will update to a new PR with ov50 appveyor and "ov50" file rename.

@rgleason rgleason closed this Apr 14, 2019
@rgleason rgleason deleted the ov50-appveyor branch April 17, 2019 16:10
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.

None yet

1 participant