search

JamieDixon / GraphViz-C-Sharp-Wrapper

GraphViz C# Wrapper
93 stars 55 forks source link

Create NuGet package #2

Closed JakeGinnivan closed 9 years ago

JakeGinnivan commented 10 years ago

It would be handy to have a nuget package =)

JamieDixon commented 10 years ago

Hi Jake,

Someone emailed me last week about this as it happens.

If I get the time to put together a nuget package in the next couple of weeks I'll do so, otherwise, feel free to send a pull request :)

JakeGinnivan commented 10 years ago

Do you think GraphViz itself should be in the package. I can't think of another way

What is your NuGet username, if you are happy I can submit a PR with the NuSpec etc and also upload it and make you an owner

JamieDixon commented 10 years ago

I'm not hugely satisfied with the way the GraphVizWrapper wraps up the command line utility. When I wrote this I was being pragmatic to meet a deadline and needed it quickly.

I think in an ideal world there would be a graphviz package somewhere that the C# wrapper could depend upon (probably in the form of a dll) and NuGet would resolve that dependency. At the moment I can't see that being feasible. Since the command line utility is currently a hard dependency, and without any time to really look further into it, I think we'd have to include it in the NuGet package.

My NuGet username is JamieDixon.

JakeGinnivan commented 10 years ago

If a better option becomes available, we can deprecate that package. I have worked around my immediate need, but will think about it as well