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

greatly simplify building #38

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mrexodia
Copy link

@mrexodia mrexodia commented Jan 28, 2017

  • no more hardcoded paths in the project
  • more versatile .gitignore (works up to VS2017)
  • include detours.lib directly in the project
  • remove x64 platform (Dark Souls is 32 bit)
  • remove Debug platform (the readme explicitly states this should not be used)
  • add filters file to categorize source files
  • added direction to DirectX SDK in the source (at the place it would error)

This conflicts/replaces pull request #33 I changed the configuration to VS2012 because that's the lowest version I could get this project to compile on.


This change is Reviewable

- no more hardcoded paths in the project
- more versatile .gitignore (works up to VS2017)
- include detours.lib directly in the project
- remove x64 platform (Dark Souls is 32 bit)
- remove Debug platform (the readme explicitly states this should not be used)
- add filters file to categorize source files
- added direction to DirectX SDK in the source (at the place it would error)
@niligulmohar
Copy link
Contributor

@mrexodia This is excellent! I've wanted to make this kind of cleanup, but didn't know how to do it properly. I've rebased my conflicting PR on top of this.

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