Overview

Request 1037575 accepted

No description set
Loading...

Dominique Leuenberger's avatar

This needs work on systemd side:

[ 1073s] ERROR: link target doesn't exist (neither in build root nor in installed system):
[ 1073s]   /usr/lib/environment.d/99-environment.conf -> ../../../etc/environment
[ 1073s] Add the package providing the target to BuildRequires and Requires
[ 1073s] INFO: relinking /usr/lib/systemd/tests/testdata/testsuite-08.units/root.mount -> -.mount (was ./-.mount)
[ 1073s] error: Bad exit status from /var/tmp/rpm-tmp.YPb0PF (%install)

https://github.com/systemd/systemd/blob/9f5bf99f1b296aeef3c84527b0a87f4734f1029c/meson.build#L2408

CC @fbui

Build fail log: https://build.opensuse.org/package/live_build_log/openSUSE:Factory:Staging:B/systemd/standard/x86_64


Thorsten Kukuk's avatar
author source maintainer

Whatever systemd tries to solve with linking to /etc/environment: pam_env is reading many files and combine them to one, not only and simply this file. Beside that the used files are additional depending on the pam_env configuration. So this most likely did never work as somebody expected.


Franck Bui's avatar

The arguments for systemd to parse /etc/environment directly (see https://github.com/systemd/systemd/pull/25546) aren't really convincing. The only downside is that modifying /etc/environment will require systemctl --user daemon-reexec instead of systemctl --user daemon-reload but I don't it will make any difference for users in practice.

Given the fact that pam_env is included in common-session, which in its turn is included by systemd-user, removing the symlink /usr/lib/environment.d/99-environment.conf symlink should require no other change and the env built by pam_env should continue to be part of the env of the user managers.

So I'll drop the symlink.

Request History
Thorsten Kukuk's avatar

kukuk created request


Factory Auto's avatar

factory-auto added opensuse-review-team as a reviewer

Please review sources


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


Saul Goodman's avatar

licensedigger accepted review

ok


Dominique Leuenberger's avatar

dimstar accepted review


Dominique Leuenberger's avatar

dimstar_suse set openSUSE:Factory:Staging:B as a staging project

Being evaluated by staging project "openSUSE:Factory:Staging:B"


Dominique Leuenberger's avatar

dimstar_suse accepted review

Picked "openSUSE:Factory:Staging:B"


Dominique Leuenberger's avatar

dimstar_suse accepted review

Staging Project openSUSE:Factory:Staging:B got accepted.


Dominique Leuenberger's avatar

dimstar_suse approved review

Staging Project openSUSE:Factory:Staging:B got accepted.


Dominique Leuenberger's avatar

dimstar_suse accepted request

Staging Project openSUSE:Factory:Staging:B got accepted.

openSUSE Build Service is sponsored by