fix spec for rubygem #836
fix spec for rubygem #836
Conversation
| @@ -10,6 +10,7 @@ | |||
| install -m 0644 --target-directory=%{buildroot}%{_datadir}/dbus-1/agama-services %{buildroot}%{gem_base}/gems/%{mod_full_name}/share/org.opensuse.Agama*.service | |||
| install -D -m 0644 %{buildroot}%{gem_base}/gems/%{mod_full_name}/share/agama.service %{buildroot}%{_unitdir}/agama.service | |||
| install -D -m 0644 %{buildroot}%{gem_base}/gems/%{mod_full_name}/share/agama-proxy-setup.service %{buildroot}%{_unitdir}/agama-proxy-setup.service | |||
| install --directory %{buildroot}/usr/lib/agama.d/ | |||
There was a problem hiding this comment.
Having a look to the Filesystem Hierarchy Standard, I would expect configuration files to be under /usr/share. Something like /usr/share/agama/conf.d. WDYT?
There was a problem hiding this comment.
However, I see modprobe.d, pam.d, etc. under /usr/lib too.
There was a problem hiding this comment.
If I remember correctly we were told to put it under /usr/share already #602 (comment).
There was a problem hiding this comment.
It is possible, but in that case we need to change path also in config file https://github.com/openSUSE/agama/blob/master/service/lib/agama/config_reader.rb#L46
@teclator as you are author of location how do you see it?
There was a problem hiding this comment.
so lets merge this one now to fix build failure and prepare new PR with location change.
| @@ -10,6 +10,7 @@ | |||
| install -m 0644 --target-directory=%{buildroot}%{_datadir}/dbus-1/agama-services %{buildroot}%{gem_base}/gems/%{mod_full_name}/share/org.opensuse.Agama*.service | |||
| install -D -m 0644 %{buildroot}%{gem_base}/gems/%{mod_full_name}/share/agama.service %{buildroot}%{_unitdir}/agama.service | |||
| install -D -m 0644 %{buildroot}%{gem_base}/gems/%{mod_full_name}/share/agama-proxy-setup.service %{buildroot}%{_unitdir}/agama-proxy-setup.service | |||
| install --directory %{buildroot}/usr/lib/agama.d/ | |||
There was a problem hiding this comment.
However, I see modprobe.d, pam.d, etc. under /usr/lib too.
Problem
The change to agama.yaml and move to web.yaml cause build failure of rubygem-agama
Solution
Fix it
Testing