-
Notifications
You must be signed in to change notification settings - Fork 42
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
Problem loading XML documentation on OS X #152
Comments
Started working on a fix here: https://github.com/tor-stangeland-bouvet/Namotion.Reflection Need to test a bit more on Linux and OS X before submitting a PR. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We have a custom tool that runs as part of our build process that uses
Namotion.Reflection
to extract the XML documentation. This runs fine on Windows and Linux, but on OS X the XML documentation file is not found.Looking into the code I have found two bugs that should be handled:
Assembly.Location
folder is not done, only check for NuGet package is done. This is inXmlDocsExtensions.GetPathByOs()
(Namotion.Reflection/src/Namotion.Reflection/XmlDocsExtensions.cs
Line 946 in a4354e2
Assembly.CodeBase
is handled differently on Windows where removingfile:///
still results in an absolute path. While on Linux and OS X the first/
(slash) is removed so that it becomes a relative address.Namotion.Reflection/src/Namotion.Reflection/XmlDocsExtensions.cs
Lines 751 to 765 in a4354e2
The text was updated successfully, but these errors were encountered: