-
Notifications
You must be signed in to change notification settings - Fork 26
packages: add utils rpm subpackage #58
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
Conversation
|
Congratulations! One of the builds has completed. 🍾 You can install the built RPMs by following these steps:
Please note that the RPMs should be used only in a testing environment. |
| # https://packit.dev/docs/configuration/ | ||
|
|
||
| specfile_path: packages/fedora/openarm-can.spec | ||
| specfile_path: packages/fedora/openarm_can.spec |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to use _ not -?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The package name is openarm_can. Can get it changed if needed/preferred.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you use openarm-can not openarm_can to align with Debian package, .pc name and so on?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, will rename the package.
| mkdir -p %{buildroot}%{_libexecdir}/openarm_can | ||
| install -pm 0751 setup/configure_socketcan.sh \ | ||
| %{buildroot}%{_libexecdir}/openarm_can/configure_socketcan.sh | ||
| install -pm 0751 setup/set_zero.sh \ | ||
| %{buildroot}%{_libexecdir}/openarm_can/set_zero.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should do this by CMake to avoid duplicated codes in debian/ and .spec.
Cc: @daipom
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Like this? #59
|
@bkmgit Can we close this now? |
Done |
closes #55