[systemd-devel] [PATCH 1/7] filesystem targets: disable default dependencies

[Lennart Poettering]

On Fri, 23.08.13 15:09, Tom Gundersen (teg at jklm.no) wrote:

> This means we can use default dependencies on mount units without having to get them automatically
> ordered before the filesystem targets.

Looks good. But before we merge this, Harald, can you comment? Any issue
with this?

> 
> Reported-by: Thomas Baechler <thomas at archlinux.org>
> ---
>  units/initrd-fs.target      | 2 ++
>  units/initrd-root-fs.target | 2 ++
>  units/local-fs.target       | 2 ++
>  units/remote-fs.target      | 2 ++
>  4 files changed, 8 insertions(+)
> 
> diff --git a/units/initrd-fs.target b/units/initrd-fs.target
> index 7578b88..866f0d0 100644
> --- a/units/initrd-fs.target
> +++ b/units/initrd-fs.target
> @@ -12,3 +12,5 @@ OnFailure=emergency.target
>  OnFailureIsolate=yes
>  ConditionPathExists=/etc/initrd-release
>  After=initrd-parse-etc.service
> +DefaultDependencies=no
> +Conflicts=shutdown.target
> diff --git a/units/initrd-root-fs.target b/units/initrd-root-fs.target
> index cd189f0..d0b9863 100644
> --- a/units/initrd-root-fs.target
> +++ b/units/initrd-root-fs.target
> @@ -11,3 +11,5 @@ Documentation=man:systemd.special(7)
>  ConditionPathExists=/etc/initrd-release
>  OnFailure=emergency.target
>  OnFailureIsolate=yes
> +DefaultDependencies=no
> +Conflicts=shutdown.target
> diff --git a/units/local-fs.target b/units/local-fs.target
> index 18c3d74..8f06ed6 100644
> --- a/units/local-fs.target
> +++ b/units/local-fs.target
> @@ -9,5 +9,7 @@
>  Description=Local File Systems
>  Documentation=man:systemd.special(7)
>  After=local-fs-pre.target
> +DefaultDependencies=no
> +Conflicts=shutdown.target
>  OnFailure=emergency.target
>  OnFailureIsolate=no
> diff --git a/units/remote-fs.target b/units/remote-fs.target
> index 09213e8..43ffa5c 100644
> --- a/units/remote-fs.target
> +++ b/units/remote-fs.target
> @@ -9,6 +9,8 @@
>  Description=Remote File Systems
>  Documentation=man:systemd.special(7)
>  After=remote-fs-pre.target
> +DefaultDependencies=no
> +Conflicts=shutdown.target
>  
>  [Install]
>  WantedBy=multi-user.target


Lennart

-- 
Lennart Poettering - Red Hat, Inc.