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

Windows: Correct RC File #12213

Merged
merged 1 commit into from
Dec 17, 2024
Merged

Windows: Correct RC File #12213

merged 1 commit into from
Dec 17, 2024

Conversation

HTRamsey
Copy link
Collaborator

This is all that is needed in this file

@HTRamsey
Copy link
Collaborator Author

@DonLakeFlyer do you know anything about this?

@DonLakeFlyer
Copy link
Contributor

Not really. Would have to go back and look art what Stable did.

@HTRamsey
Copy link
Collaborator Author

My understanding now, and looking at a bunch of other projects on github, is that this is all that's needed and the other parts are autogenerated now. Everything but this one line for the icon is covered by a APSTUDIO_INVOKED define

@HTRamsey HTRamsey merged commit ac6a4f7 into mavlink:master Dec 17, 2024
@HTRamsey HTRamsey deleted the dev-windows-rc branch December 17, 2024 03:39
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