NixOS / nixpkgs

Nix Packages collection & NixOS
MIT License
18.11k stars 14.15k forks source link

syslog.target is deprecated #149721

Open lunik1 opened 2 years ago

lunik1 commented 2 years ago

Issue description

Some NixOS modules refer to syslog.target, which was removed in systemd v198 [1]. It should be safe to remove this as systemd will ensure logging is available before a service is started [2]. Future modules referencing syslog.target should be avoided.

Affected modules (generated with rg -l "syslog.target" nixos/modules | sed "s|nixos/modules/|- [ ] |g"):

Notify maintainers

@pingiun (eternal-terminal) @evils (rasdaemon)

evils commented 2 years ago

your first link no longer points at the right line as they prepended more news shortly after you posted this, you probably want this instead https://github.com/systemd/systemd/blob/6aa8d43ade72e24c9426e604f7fc4b7582b9db7c/NEWS#L61

evils commented 2 years ago

i did some digging @madjar seems to have authored the syslog.target lines in cloud-init and maintains the package @sbruder and @infinisil seem to have co-authored the syslog.target line in invidious and they maintain the package @matejc seems to have authored the syslog.target lines in connman and maintains the package @cstrahan seems to have authored the syslog.target line in mbpfan and maintains the package @erdnaxe seems to have authored the syslog.target line in nitter and maintains the package @spacefrogg seems to have authored the syslog.target line in openafsServer and maintains the package @bachp seems to have authored the syslog.target line in glusterfs @mbrgm seems to have authored the syslog.target lines in keepalived (which has no listed maintainer, do you use it?) @Sohalt seems to have authored the syslog lines in spacenavd and maintains the package

lunik1 commented 2 years ago

I have fixed the link and great work tracking down the maintainers! 🙂

SuperSandro2000 commented 7 months ago

@RaitoBezarius can you take care of keepalived? @AndersonTorres can you take care of connman? @zimbatm can you take care of cloud-init?

lunik1 commented 7 months ago

I have added aesmd to the list, @veehaitch looks to be the main contributor