search

mystilleef / FedoraZram

Scripts that enable zram for Fedora
72 stars 48 forks source link

zram.service overwritten by Anaconda. #15

Open binarybana opened 9 years ago

binarybana commented 9 years ago

I found this in my (Fedora 21) system today:

$ dnf whatprovides /usr/lib/systemd/system/zram.service
Using metadata from Tue May 19 15:25:37 2015 (9 days, 18:24:36 hours old)
anaconda-core-21.48.22-1.fc21.x86_64 : Core of the Anaconda installer
Repo        : @System

So I think it would be more correct to install zram.service into /etc/systemd/system/ which is where Local Configuration units should be installed (and will mask the anaconda one).

Hubbitus commented 9 years ago

I think it is not common practice to install under /etc/systemd/system/ from packages. It is primarily intended for system administrators customization: https://fedoraproject.org/wiki/Systemd#How_do_I_customize_a_unit_file.2F_add_a_custom_unit_file.3F

binarybana commented 9 years ago

I can see that point. I guess when I installed this package by cloning from github and running the makefile, I felt like I was customizing my install, but I forgot that the makefile can also generate an rpm.

But there still is the issue of overlap with the Anaconda service file.

Hubbitus commented 9 years ago

May be just rename file is enough?

binarybana commented 9 years ago

Agreed.

On Wed, Jun 3, 2015, 3:26 AM Pavel Alexeev aka Pahan-Hubbitus < notifications@github.com> wrote:

May be just rename file is enough?

— Reply to this email directly or view it on GitHub https://github.com/mystilleef/FedoraZram/issues/15#issuecomment-108290686 .