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

Use artifactId to generate debian package name #650

Merged
merged 1 commit into from
Mar 6, 2023

Conversation

Skunnyk
Copy link
Contributor

@Skunnyk Skunnyk commented Nov 8, 2021

  • Since 77a5b8f, "name" is "Prometheus
    JMX Exporter - Http Server", which break debian package generation
  • use artifactId to get a correct name

@Skunnyk
Copy link
Contributor Author

Skunnyk commented Nov 8, 2021

This restore previous behavior, otherwise the package is not installable on debian systems:

dpkg: error processing archive /home/build/jmx_prometheus_httpserver-0.16.1.deb (--install):
 parsing file '/var/lib/dpkg/tmp.ci/control' near line 1:
 invalid package name (character ' ' not allowed (only letters, digits and characters '-+._'))

@ghost
Copy link

ghost commented Oct 6, 2022

It would bee great to get this merged. We prefer installing debian packages over hand crafting installations, but this bug is preventing us from using newer versions of jmx exporter.

- Since 77a5b8f, "name" is "Prometheus
  JMX Exporter - Http Server", which break debian package generation
- use artifactId to get a correct name

Signed-off-by: Romain Bouvier <skunnyk@alteroot.org>
@Skunnyk
Copy link
Contributor Author

Skunnyk commented Feb 20, 2023

Hi :)

Is it possible for this PR to be merged ? This is really a small change affecting debian package build (which is advertised in the README), and it would be super awesome to have it fixed for the next release 🙏

@fstab fstab merged commit 3bdc716 into prometheus:main Mar 6, 2023
@fstab
Copy link
Member

fstab commented Mar 6, 2023

Thanks a lot for the reminder and for the PR, it's merged now.

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.

2 participants