search

graylog-labs / graylog-plugin-metrics-reporter

Graylog Metrics Reporter Plugins
https://www.graylog.org/
GNU General Public License v3.0
80 stars 18 forks source link

Update package names to reflect artifactId #15

Closed patcable closed 7 years ago

patcable commented 7 years ago

When using mvn jdeb:jdeb I noticed that the deb files that were generated had the package name of the module itself (ie. using apt-cache show returns Package: Graylog Metrics Graphite Reporter Plugin) rather than the artifact name (Package: metrics-reporter-graphite).

I think people expect to use apt-get install metrics-reporter-graphite rather than apt-get install 'Graylog Metrics Graphite Reporter Plugin' -- so this patch updates src/deb/control/control to use that instead.

CLAassistant commented 7 years ago

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

dennisoelkers commented 7 years ago

Thank you very much for this contribution, @patcable!

joschi commented 7 years ago

@patcable Please also remember to sign the CLA: https://cla-assistant.io/Graylog2/graylog-plugin-metrics-reporter?pullRequest=15

joschi commented 7 years ago

@patcable Any news on signing the CLA?