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

Update Riak rpm naming to match global standards #1076

Merged
merged 1 commit into from
Jul 15, 2021

Conversation

Bob-The-Marauder
Copy link
Contributor

Currently, when building rpm packages, riak-2.x.x names correctly e.g. riak-2.9.8-1.el8.x86_64.rpm but the riak-3.x.x rpms are missing a dot before the basearch i.e. riak-3.0.6-1.el8x86_64.rpm. This incredibly tiny fix re-adds the missing dot in the package name.

I have already tested this successfully on one of the regular package builders we use to publish the binaries by creating a phony tag of riak-3.0.61 and building it accordingly:

[root@localhost riak]# ls rel/pkg/out/packages/
riak-3.0.61.OTP20.3-1.el8.src.rpm      riak-3.0.61.OTP20.3-1.el8.x86_64.rpm 
riak-3.0.61.OTP20.3-1.el8.src.rpm.sha  riak-3.0.61.OTP20.3-1.el8.x86_64.rpm.sha

Currently, when building rpm packages, riak-2.x.x names correctly e.g. `riak-2.9.8-1.el8.x86_64.rpm` but the riak-3.x.x rpms are missing a dot before the basearch i.e. `riak-3.0.6-1.el8x86_64.rpm`. This incredibly tiny fix re-adds the missing dot in the package name.

I have already tested this successfully on one of the regular package builders we use to publish the binaries by creating a phony tag of riak-3.0.61 and building it accordingly:
```
[root@localhost riak]# ls rel/pkg/out/packages/
riak-3.0.61.OTP20.3-1.el8.src.rpm      riak-3.0.61.OTP20.3-1.el8.x86_64.rpm 
riak-3.0.61.OTP20.3-1.el8.src.rpm.sha  riak-3.0.61.OTP20.3-1.el8.x86_64.rpm.sha
```
@Bob-The-Marauder
Copy link
Contributor Author

Please feel free to merge and delete when ready.

@martinsumner martinsumner merged commit 6062a00 into basho:develop-3.0 Jul 15, 2021
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