Bug 91677 - [Patch] check for libsystemd inseatd of libsystemd-daemon
Summary: [Patch] check for libsystemd inseatd of libsystemd-daemon
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Server/General (show other bugs)
Version: git
Hardware: Other All
: medium enhancement
Assignee: Xorg Project Team
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-08-18 11:25 UTC by Marc-Antoine Perennou
Modified: 2018-06-13 19:18 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
check for libsystemd (2.00 KB, patch)
2015-08-18 11:25 UTC, Marc-Antoine Perennou
no flags Details | Splinter Review

Description Marc-Antoine Perennou 2015-08-18 11:25:16 UTC
They were merged a while ago. Fixes the build when systemd is compiled with --disable-compat-libs
Comment 1 Marc-Antoine Perennou 2015-08-18 11:25:53 UTC
Created attachment 117758 [details] [review]
check for libsystemd
Comment 2 Michel Dänzer 2015-08-19 03:13:36 UTC
Please submit the patch to the xorg-devel@lists.x.org mailing list for review.

Actually, it looks like somebody already submitted a similar patch a while ago and it was reviewed, but maybe not merged yet: https://patchwork.freedesktop.org/patch/45465/
Comment 3 Marc-Antoine Perennou 2015-08-19 07:52:38 UTC
(In reply to Michel Dänzer from comment #2)
> Please submit the patch to the xorg-devel@lists.x.org mailing list for
> review.
> 
> Actually, it looks like somebody already submitted a similar patch a while
> ago and it was reviewed, but maybe not merged yet:
> https://patchwork.freedesktop.org/patch/45465/

Right, I didn't check the list indeed, you can close then.

It would be nice to get the patch from the list merged though
Comment 4 Laurent Bigonville 2015-12-05 12:39:29 UTC
Hi,

The patch in this bug report is more complete that the one that got merged.

The REQUIRED_LIBS variable is still referring to libsystemd-daemon instead of libsystemd
Comment 5 Adam Jackson 2018-06-13 19:18:00 UTC
(In reply to Laurent Bigonville from comment #4)
> Hi,
> 
> The patch in this bug report is more complete that the one that got merged.
> 
> The REQUIRED_LIBS variable is still referring to libsystemd-daemon instead
> of libsystemd

It is, but harmlessly; LIBSYSTEMD_DAEMON is set equal to LIBSYSTEMD if the system has libsystemd but not -daemon.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.