Skip to content

Conversation

@bkmgit
Copy link
Contributor

@bkmgit bkmgit commented Oct 12, 2025

closes #55

@packit-as-a-service
Copy link

Congratulations! One of the builds has completed. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/enactic-openarm_can-58
  • And now you can install the packages.

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
Copy link
Contributor

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 -?

Copy link
Contributor Author

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.

Copy link
Contributor

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?

Copy link
Contributor Author

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.

Comment on lines +57 to +61
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
Copy link
Contributor

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

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like this? #59

@daipom
Copy link
Contributor

daipom commented Oct 16, 2025

@bkmgit Can we close this now?

@bkmgit
Copy link
Contributor Author

bkmgit commented Oct 16, 2025

@bkmgit Can we close this now?

Done

@bkmgit bkmgit closed this Oct 16, 2025
@bkmgit bkmgit deleted the rpm-utils-package branch October 16, 2025 06:53
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.

Add openarm_can-utils rpm subpackage

3 participants