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

[WIP] Enable Sourcelink #32

Merged
merged 3 commits into from
Oct 16, 2018
Merged

[WIP] Enable Sourcelink #32

merged 3 commits into from
Oct 16, 2018

Conversation

enricosada
Copy link
Contributor

No description provided.

@7sharp9
Copy link
Contributor

7sharp9 commented Oct 10, 2018

I found an issue with sourcelink in froto, seems Microsoft.SourceLink.GitHub caused the build to fail in both Rider and VS4Mac

@ctaggart
Copy link

The issue is that Rider and VS4Mac are using Mono and libgit2 native library is not loading right. They are working on a fix to trim the dependencies. dotnet/sourcelink#155
If there was a way to conditionally ignore it on Mono, that would be a good workaround.

@enricosada
Copy link
Contributor Author

thx @ctaggart
should i use https://github.com/ctaggart/SourceLink instead?

meanwhile i'll enable it only on windows in Release mode

@enricosada
Copy link
Contributor Author

disabled it if run on mono msbuild.
@7sharp9 can you check?

@7sharp9
Copy link
Contributor

7sharp9 commented Oct 16, 2018

builds fine with Rider, merging.

@7sharp9 7sharp9 merged commit c57709d into jet:master Oct 16, 2018
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.

None yet

3 participants