[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#27682] [PATCH 2/2] gnu: Add lxc.
From: |
宋文武 |
Subject: |
[bug#27682] [PATCH 2/2] gnu: Add lxc. |
Date: |
Sat, 15 Jul 2017 18:58:30 +0800 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Marius Bakke <address@hidden> writes:
> 宋文武 <address@hidden> writes:
>
>> * gnu/packages/virtualization.scm (lxc): New package.
>
> Thanks!
>
> [...]
>
>> + (arguments
>> + '(#:configure-flags
>> + '("--sysconfdir=/etc"
>> + "--localstatedir=/var")
>> + #:phases
>> + (modify-phases %standard-phases
>> + (replace 'install
>> + (lambda* (#:key outputs #:allow-other-keys)
>> + (let* ((out (assoc-ref outputs "out"))
>> + (bashcompdir (string-append out
>> "/etc/bash_completion.d")))
>> + (zero? (system*
>> + "make" "install"
>> + (string-append "bashcompdir=" bashcompdir)
>> + ;; Thoses files will be handled by system services.
>> + "LXCPATH=/tmp/var/lib/lxc"
>> + "localstatedir=/tmp/var"
>> + "sysconfdir=/tmp/etc"
>> + "sysconfigdir=/tmp/etc/default"))))))))
>
> If these are not important, why not set them to $out/etc, $out/var and
> so on? /tmp seems like a dangerous default to me, since any unprivileged
> user can populate these directories.
Those paths are configured to '/var/lib/lxc', '/var' etc. The install
phase drop them to avoid touching system directories in the builder.
The effective paths are still '/var/lib/lxc', owned by root, which will
be handled by lxc-service-type and alike.