Skip to content

NH-3990 - Fixes for upgrade to VS2017 project structure #672

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

Merged
merged 3 commits into from
Aug 11, 2017

Conversation

ngbrown
Copy link
Contributor

@ngbrown ngbrown commented Aug 10, 2017

This corrects a few issues with #605

  • NHibernate.Tool.HbmXsd.csproj wasn't generating an .EXE.
  • Because ABC.hbm.xml was copied to the bin output folders, it was being re-included as an EmbeddedResource. This will cause problems when the VB project is migrated to VS 2017 and depends on NHibernate.Test.csproj. Solution is to exclude the bin folders from being considered when finding files for embedding.
  • Certain references are auto-included, so they can be removed.

@hazzik
Copy link
Member

hazzik commented Aug 10, 2017

Also, the tool needs the default generation path be changed as the tool is being compiled to a different folder now.

@fredericDelaporte
Copy link
Member

And it needs a bit more indeed. It was embedding the mapping schema, but lost it during conversion to new format. Going to add a commit once I have get it running again.

hazzik
hazzik previously approved these changes Aug 11, 2017
@fredericDelaporte fredericDelaporte merged commit 0d70729 into nhibernate:master Aug 11, 2017
@ngbrown ngbrown deleted the NH-3990-HbmXsdExe branch August 12, 2017 02:40
@fredericDelaporte fredericDelaporte added this to the 5.0 milestone Sep 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants