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

Mark the Nuget Package as "Unofficial" #58

Closed
LeeCampbell opened this issue Apr 4, 2016 · 5 comments
Closed

Mark the Nuget Package as "Unofficial" #58

LeeCampbell opened this issue Apr 4, 2016 · 5 comments

Comments

@LeeCampbell
Copy link
Owner

As this isn't the official HdrHistogram.NET Repo*, we shouldn't be taking the HdrHistogram nuget package title.
It appears to have been taken anyway by the existing thing.
https://www.nuget.org/packages/HdrHistogram.NET/

Also worth removing the build time dependency that has made it into the nuspec file declaration

<?xml version="1.0"?>
<package xmlns="http://schemas.microsoft.com/packaging/2011/10/nuspec.xsd">
  <metadata>
    <!--TODO: rename to HdrHistogram -->
    <id>HdrHistogram</id>
    <version>2.0.75-beta</version>
    <title>HdrHistogram</title>
    ...
    <tags>HdrHistogram Histogram Instrumentation</tags>
    <!--TODO: remove this build time dependency -->
    <dependencies>
      <dependency id="AssemblyVersion.MSBuild" version="0.1.0" />
    </dependencies>
  </metadata>
</package>

*Official one is here - https://github.com/HdrHistogram/HdrHistogram.net

@LeeCampbell
Copy link
Owner Author

Arguably he can keep the HdrHistogram.NET package, and we can just take HdrHistogram? The .NET seem redundant here. We dont get Rx.NET, we get Rx, right?!

@LeeCampbell
Copy link
Owner Author

As part of this fix, it seems that the existing tools have just enough friction that we will need to script a bunch of things.

  • The AssemblyVersioning nuget packages currently are not flagged as DevDeps, so try to get added as NugetDependencies to our package.
  • We want to ship dll, pdb and xml files from release
  • We want to control the versioning of the assembly as Nuget sees it, as .NET sees it and as Windows sees it (all different)
  • We will want more control once DNX/UWA and dotnetStandards start rolling out.

I think it is going to be best to follow the lead of the JSON.NET project.

@peske
Copy link

peske commented Apr 11, 2016

It's great to see that we'll finally have actively developed supported HdrHistogram port. Thanks for that guyz!
Now question: Nuget package is not available yet, right? Any estimation when it will be published?
Thanks!

@LeeCampbell
Copy link
Owner Author

I am doing my best to get it out asap. Was hoping to get it out in the
weekend, but was battling build issues.
Should be a matter of days. Would love to get feedback when it is out.
On 11 Apr 2016 8:22 AM, "peske" notifications@github.com wrote:

It's great to see that we'll finally have actively developed supported
HdrHistogram port. Thanks for that guyz!
Now question: Nuget package is not available yet, right? Any estimation
when it will be published?
Thanks!


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#58 (comment)

@LeeCampbell
Copy link
Owner Author

And we have a nuget package! - https://www.nuget.org/packages/HdrHistogram/

Ok so I didn't go with naming it unofficial, but due to the amount of work I have poured into this project, it feels pretty bloody official to me! ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants