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

*- Updated the project to VS2013 #12

Closed
wants to merge 13 commits into from
Closed

*- Updated the project to VS2013 #12

wants to merge 13 commits into from

Conversation

Zangetsu38
Copy link
Contributor

  • Updated the project to VS2013
  • Updated git-version-gen.cmd

@AlexAltea
Copy link
Contributor

I see two problems with this pull request:

  • As far as I know, no other developer uses the AZERTY layout so this makes no sense. (A useful change would be for example adding a key mapper to specify the keyboard).
  • Does updating some wxWidgets 3 modules fix your issue? If not, then there is no need to take the 'risk' of mixing two different versions of wxWidgets. We have to stay with one version to avoid unexpected errors.

Otherwise I agree with the updated git-version-gen.cmd. I already have Visual Studio 2013, so if I don't mind about that update. If DH also has VS2013 installed, then I agree with the update of the project files.

@Zangetsu38
Copy link
Contributor Author

yes I understand, I thought there was only the U.S. that had a QWERTY keyboard ^ ^
And wxWidgets 3.0 module I put, its not an issue of incompatibility but I understand what you mean even if it should not be a problem :).

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

2 participants