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.

Sign up for GitHub

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

Jump to bottom

plugins/org.rpm.conf does not get installed #2474

Closed
ffesti opened this issue Apr 11, 2023 · 2 comments · Fixed by #2476 or #2530
Closed

plugins/org.rpm.conf does not get installed #2474

ffesti opened this issue Apr 11, 2023 · 2 comments · Fixed by #2476 or #2530
Milestone
4.19.0-beta

Comments

@ffesti
Copy link
Contributor

ffesti commented Apr 11, 2023

it should go to /etc/dbus-1/system.d/org.rpm.conf to make the dbus announce plugin work.
@ffesti ffesti added this to the 4.19.0-alpha milestone Apr 11, 2023
ffesti added a commit to ffesti/rpm that referenced this issue Apr 11, 2023
@ffesti
Install /etc/dbus-1/system.d/org.rpm.conf
56c91fa 
which was accidentially dropped when converting to cmake

Use CMAKE_INSTALL_FULL_SYSCONFDIR as it expands to /etc instead of
/usr/etc

Resolves: rpm-software-management#2474
@ffesti ffesti mentioned this issue Apr 11, 2023
Merged
pmatilai pushed a commit that referenced this issue Apr 11, 2023
@ffesti @pmatilai
Install /etc/dbus-1/system.d/org.rpm.conf
2080896 
which was accidentially dropped when converting to cmake

Use CMAKE_INSTALL_FULL_SYSCONFDIR as it expands to /etc instead of
/usr/etc

Resolves: #2474
@Conan-Kudo
Copy link
Member

Actually, it should be /usr/share/dbus-1/system.d/

@ffesti
Copy link
Contributor Author

ffesti commented Apr 17, 2023

hmpf

@ffesti ffesti reopened this Apr 17, 2023
@ffesti ffesti modified the milestones: 4.19.0-alpha, 4.19.0-beta Apr 17, 2023
ffesti added a commit to ffesti/rpm that referenced this issue Jun 2, 2023
@ffesti
Move dbus announce plugin config in /usr/share
98275fb 
Hard code the prefix to /usr as this needs to match with dbus and not
our own install prefix.

Resolves: rpm-software-management#2474
@ffesti ffesti mentioned this issue Jun 2, 2023
Merged
ffesti added a commit to ffesti/rpm that referenced this issue Jun 2, 2023
@ffesti
Move dbus announce plugin config in /usr/share
ba0647f 
Users are not supposed to meddle with this. So it really should not go
into /etc.

Hard code the prefix to /usr as this needs to match with dbus and not
our own install prefix.

Resolves: rpm-software-management#2474
ffesti added a commit to ffesti/rpm that referenced this issue Jun 2, 2023
@ffesti
Move dbus announce plugin config to DATADIR
d35e9ba 
Users are not supposed to meddle with this. So it really should not go
into /etc.

Use the DATADIR from dbus-1 and not our own install prefix.

Resolves: rpm-software-management#2474
pmatilai pushed a commit that referenced this issue Jun 5, 2023
@ffesti @pmatilai
Move dbus announce plugin config to DATADIR
51c596a 
Users are not supposed to meddle with this. So it really should not go
into /etc.

Use the DATADIR from dbus-1 and not our own install prefix.

Resolves: #2474
dmnks pushed a commit to dmnks/rpm that referenced this issue Jun 9, 2023
@ffesti @dmnks
Move dbus announce plugin config to DATADIR
568469d 
Users are not supposed to meddle with this. So it really should not go
into /etc.

Use the DATADIR from dbus-1 and not our own install prefix.

Resolves: rpm-software-management#2474
(cherry picked from commit 51c596a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
4.19.0-beta
2 participants
@Conan-Kudo @ffesti