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

Create and deploy a Nuget package #12

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

Conversation

p-softway
Copy link

Hi,
As I needed your library in a project using nugets, I thought it was a good idea to publish it on Nuget.org.
The package is now listed as CrashRpt2.CPP created from my Appveyor project.
This currently works with an API key from my Nuget.org account that I just set up which is encoded for my Appveyor account. If you agree to merge this into your repo I'll add you as an owner of that package so you can replace that key with your own.

@QbProg
Copy link
Collaborator

QbProg commented Dec 4, 2018

Hello! thanks for the contribution! I'll wait a few days if anyone has anything to say about the PR, then I'll merge it! Personally, I don't have interest in becoming the owner of the package; I keep this repo up to date with compiler updates and add some seldom fixes, but unfortunately the project is pretty much abandoned at the moment.

Just a question: how come the nuget version is 1.5.3? Is that related to the 1.5.0.0 used in the master? Or are these unrelated? I mean, if this repo ever gets at 1.5.3.0 what would happen?

@p-softway
Copy link
Author

p-softway commented Dec 9, 2018 via email

@jowr
Copy link
Collaborator

jowr commented Dec 10, 2018

Thanks for working on this.

Would creating a GitHub organization solve some of the issues with the ownership of the package? In that way we could divide the work among the people interested in the project.

Looking at the SVN logs on Sourceforge, it seems that @AndreasSchoenle also contributed earlier. Maybe he could share his views on this as well?

Other things to consider:

  • Add some information stating that the project is looking for a dedicated maintainer.
  • Include the CrashFix server code? What about the license generation?
  • Clean up the wiki and convert some of the pages to static html using GitHub pages.
  • Try to get in contact with the original developer to change the sourceforge project (new tickets and questions keep appearing there).

@AndreasSchoenle
Copy link
Contributor

The major.minor.version.build versioning scheme seems fine to me but we do not use Nuget and at the moment we use binaries compiled from our fork, so I have no real stakes in this discussion.

Whether or not to create a GibHub organization I believe to be entirely up to @QbProg, who has put in the successful effort to collect, align and merge contributions from different forks so far. If he/she is willing to continue and reasonable PRs get accepted upstream from the various forks, I see no problem either way.

As to CrashFix, there seem to be a few imports on GitHub already, e.g. at https://github.com/0um/crashfix,https://github.com/xyzz/CrashFix and forks thereof, that have seen bugfixes and enhancements.

On first glance they do not seem to share a lot of work beyond the initial import. Thus, it seems like many users of CrashRpt largely roll their own report analysis, integrating with their version control system, their specific build and deploy mechanism, server infrastructure and their product versioning scheme and with regression tests specifically tailored to their setup.

I imagine many of them do not use CrashFix at all, as the main task is to analyze a minidump and organize reports in a way that is meaningful to developers. This task is not really tied to the specifics of what CrashRpt does.

Therefore, I would argue to consider CrashFix a separate project with separate repositorie(s). Personally, I am not using most of the CrashProbe API during analysis and would not mind to reduce its scope to just extract the information specific to CrashRpt and a minidump. The Minidump reader itself could then be moved to CrashFix or any other analysis project.

@QbProg
Copy link
Collaborator

QbProg commented Dec 27, 2018

indeed, I don't see any advantange in having the build number in the version, as Patch releases are good enough. We could use the build number to packaging/nuget as @p-softway suggested.
Some modification in crashrpt code would be required though.

There's a lot of work that should be done on this project, as @jowr pointed, but without a proper server , the crashrpt tool is almost useless. I personally thinking of abandoning it as I was not able to put a reporting/analytics solution up quickly, which supports nicely 64bit. In my view, it would be more useful to make it work with a server of a different solution, but could not find any suitable.

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.

4 participants