Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Add development package to build#97

Merged
mmitche merged 1 commit into
dotnet:masterfrom
mmitche:dev-package
Feb 9, 2015
Merged

Add development package to build#97
mmitche merged 1 commit into
dotnet:masterfrom
mmitche:dev-package

Conversation

@mmitche
Copy link
Copy Markdown
Member

@mmitche mmitche commented Feb 5, 2015

(Note once the other PDB changes go in I'll rebase this onto master to remove some of the noisy diffs from that change)

Adds a "development" package to the build. This builds a nuget package that contains the PDBs as well as headers that can be used for development of other components.

@mmitche
Copy link
Copy Markdown
Member Author

mmitche commented Feb 5, 2015

Waiting on #96 to finish before this goes in

Comment thread src/inc/CMakeLists.txt Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FWIW, this can be one call:

install(FILES core.h corhdr.h corinfo.h corjit.h DESTINATION inc)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks will update

@mmitche
Copy link
Copy Markdown
Member Author

mmitche commented Feb 9, 2015

Did anyone take a look at this?

@mmitche
Copy link
Copy Markdown
Member Author

mmitche commented Feb 9, 2015

@ellismg maybe you? :D :D

@ellismg
Copy link
Copy Markdown

ellismg commented Feb 9, 2015

LGTM.

Add new nuget package that can be developed against.  Also modify the cmake lists of a few directories to add in missing PDBs.
mmitche added a commit that referenced this pull request Feb 9, 2015
Add development package to build
@mmitche mmitche merged commit b6da550 into dotnet:master Feb 9, 2015
@mmitche mmitche deleted the dev-package branch February 10, 2015 00:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants