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 build #38

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

alexBraidwood
Copy link

I put a windows build together for a library I'm working on, thought it might be useful.

  • Update peg to 0.1.18
  • Updated README to include windows dll build instructions
  • Added VS-specific #defines to utility_functions.h
  • Added disable _strdup deprecation compiler bug in VS
  • Added exports.h to export functions with EXPORT define for windows DLL

- Update peg to 0.1.18
- Updated README to include windows dll build instructions
- Added VS-specific #defines to utility_functions.h
- Added disable _strdup deprecation compiler bug in VS
- Added exports.h to export functions with EXPORT define for windows DLL
- Fixed Build configurations
@jgm
Copy link
Owner

jgm commented Jul 26, 2017

This looks good, but it seems to have other changes besides the ones advertised.
E.g. line number tracking, and other substantive changes to the parser.

@alexBraidwood
Copy link
Author

alexBraidwood commented Jul 26, 2017

@jgm Ah! Didn't even see those peg changes. That might explain some weird output. I'll go back and fix it up, then update the request.

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