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

Win changes #127

Closed
wants to merge 4 commits into from
Closed

Win changes #127

wants to merge 4 commits into from

Conversation

rgleason
Copy link
Contributor

@rgleason rgleason commented Apr 10, 2019

To get the plugin working in Windows again after changes for Android:

weatherfaxUI.cpp
add two instances of #ifdef _OCPN_ANDRIOD and #endif line 172, 175, 224, 227
This file is normally created by wxFormBuilder, but we know of no other way to separate the Android format commands.

src/libaudio/config.h (LATER: did this too early, applies to ov50 compile, reverted it)
Remove (comment out) #1define snprintf at line 4 (reverted this)

Tested the result in ov488 and it works except for the second screen of the decoder where coordinates are obscured.

@rgleason
Copy link
Contributor Author

Stelian has changes that he is going to make into a PR which

  1. Should be merged after my PR Xml update 04 10 2019 #126 Xml update
  2. Before my PR Ov50 appveyor #128 Ov50 appveyor

I am going to close this PR #127 Win Changes because Stelian's PR will be better.

@rgleason rgleason closed this Apr 10, 2019
@rgleason rgleason deleted the win-changes branch April 10, 2019 23:18
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