-
Notifications
You must be signed in to change notification settings - Fork 886
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
Put the release file in a folder inside the .tar.gz #173
Comments
Hey there, my instinct is to not change it at this point because probably people rely on it to not be in a folder and their puppet, ansible, whatever scripts would break if out of a sudden future releases are in a folder. Is this currently causing serious issues for you or is it more of a convenience thing? |
Thanks for the reply and sorry for my delayed reply. |
@oliver006 @grantstephens Can't we use a symlink and this way work around this problem? |
+1 for this, I would like to remove the customisations in https://github.com/lest/prometheus-rpm |
@dkanchev it looks like that would be possible. Something like |
Heads up, PR #256 will fix this and put the binary in a folder. I'm expecting to merge it some time next week. |
As of v1.0.0 the binary is now in a folder named e.g. |
So this might be a weird one, but essentially all the other exporters that I've come across and use put the binary inside another folder in the release .tar.gz- this might be annoying or whatever, but at least it was consistent, except for this exporter.
Would it be possible to adjust the release process to make this possible?
The text was updated successfully, but these errors were encountered: