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

Remove debian package information #169

Closed
wants to merge 1 commit into from
Closed

Remove debian package information #169

wants to merge 1 commit into from

Conversation

josegonzalez
Copy link

This is not a StatsD concern. Refs #161

This is not a StatsD concern. Refs #161
@mrtazz
Copy link
Member

mrtazz commented Oct 20, 2012

I agree it's not a StatsD main concern, but it still serves as a useful example I think. And just because there is no documentation included, doesn't mean it should be removed completely.

@mrtazz mrtazz closed this Oct 20, 2012
@josegonzalez
Copy link
Author

All or no documentation. Don't be a wiki troll.
On Oct 19, 2012 11:12 PM, "Daniel Schauenberg" notifications@github.com
wrote:

I agree it's not a StatsD main concern, but it still serves as a useful
example I think. And just because there is no documentation included,
doesn't mean it should be removed completely.


Reply to this email directly or view it on GitHubhttps://github.com//pull/169#issuecomment-9627591.

@loganfuller
Copy link

Just to put in my two cents - it's comments like josegonzalez's that make people avoid open source. The entire point of a package is to allow people unable or unwilling to dedicate their time to learning such a specific workflow as creating a Debian package to still derive use from the software. I strongly agree that the package info should remain.

@josegonzalez
Copy link
Author

@loganfuller Thanks for commenting on a 5 month old, closed pull request.

My original pull request was meant to be a tongue-in-cheek remark about the fact that, while we do have some of the requirements for a debian package, there is no usage documentation for that.

I intended on replacing our own statsd version - which broke due to a ruby upgrade - with etsy's statsd, but the installation documentation left something to be desired. Should I install an old tag? Should I just clone the repo as the docs say? How do I go about updating (if at all)? That is extremely unfriendly to someone who wants to use the software.

Yes, I get the point of having the package info there, but there are a few problems with the etsy developer's stance on it:

  • Does this mean we ignore all other packaging attempts? Should we include Redhat or Arch packaging information?
  • Given that at some point the resources were deemed good enough to add to the repo, can whoever made that commit provide a small - not even necessarily complete - guide as to how to use the package info to create a package?

The entire point of a package is to allow people unable or unwilling to dedicate their time to learning such a specific workflow as creating a Debian package to still derive use from the software.

I can still derive use from the software without debian packaging info - which might even be unmaintained given the fact that there is no one actively stating "here is how we package it, find and report bugs everyone!" in the readme - so what is the point of it?

At least one etsy developer has said it wasn't their responsibility to maintain a debian package, and this I agree with. Given that they do not want to maintain a debian package - which is totally their choice - I decided that maybe they should just remove the debian info from the repo, so as not to cause issues when new developers stumble upon it, hoping that there is a package they can just wget and install.

In any case, the pull request is closed, and there is no point to either of our comments, except to waste time and annoy anyone watching this repository via email.

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.

3 participants