Skip to content

Upgraded Mono.Cecil and Mono.Cecil.Pdb.dll to Latest as of 8-29-2014#5

Closed
jguady wants to merge 1 commit into
kzu:masterfrom
jguady:master
Closed

Upgraded Mono.Cecil and Mono.Cecil.Pdb.dll to Latest as of 8-29-2014#5
jguady wants to merge 1 commit into
kzu:masterfrom
jguady:master

Conversation

@jguady
Copy link
Copy Markdown

@jguady jguady commented Aug 29, 2014

Hoping you'll work with me on this. I'm just not sure how to get the csproj to include the assembly version and other pieces.

@einaregilsson
Copy link
Copy Markdown
Contributor

Hi and thanks for the pull request. Is there a particular reason to add a newer version of Mono.Cecil?

@jguady
Copy link
Copy Markdown
Author

jguady commented Sep 2, 2014

A project I'm working is using Squish CoCo http://www.froglogic.com/squish/coco/ to obtain code coverage metrics, however some projects need to use your tool in order to activate the code coverage statistics. Once injected, the tests are ran using Xunit, which also uses Mono.Cecil 9.5 and we were running into version compatibility issues.

@jguady jguady closed this Sep 2, 2014
@jguady
Copy link
Copy Markdown
Author

jguady commented Sep 2, 2014

oops I clicked 'close and comment'

@jguady jguady reopened this Sep 2, 2014
@einaregilsson
Copy link
Copy Markdown
Contributor

I still don't understand why that would conflict. My tool changes an assembly slightly, but it doesn't run in process with the application, and after InjectModuleInitializer is done running there is nothing from it which should cause Mono.Cecil to load at all.

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