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

Let Nocco work in Linux (mono) environments #18

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

stephas
Copy link

@stephas stephas commented Jun 21, 2015

Hi dontangg,

I had to remove the nuget.exe reference from the .sln file and let the monodevelop ide resolve the dependencies in addition to the change in this pull request with respect to directory separate characters.

Works great for me! If this compiles and generates docs for you on Windows, than merge with confidence. :)

Thanks for this C# port!

@dontangg
Copy link
Owner

I'm cool with the first change that detects Linux and fixes the path. However, since part of the original point behind this was to be quick and dirty, it would probably be better using the ternary operator.

I have a hard time removing the NuGet reference because Visual Studio won't automatically find the packages, download them, etc. without it. But, I agree that it would be cool if it worked on other platforms.

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