diff mbox series

[2/4,v3] package/freerdp: optional systemd's journald support

Message ID 7a50d6a13cc101e92bbcb3af45310421415af9a5.1510474387.git.yann.morin.1998@free.fr
State Accepted
Commit 57c0ac59a54a82ccec7473bd7e126daf15ae71a2
Headers show
Series [1/4,v3] package/freerdp: drop dependency on host-pkgconf | expand

Commit Message

Yann E. MORIN Nov. 12, 2017, 8:13 a.m. UTC
FreeRDP now has support for writing to the journal, so we must ensure
proper dependency and correct options.

Partially fixes:
    http://autobuild.buildroot.org/results/81aa66ddd88919295ccb5f34b527b737627263a7/

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Arnout Vandecappelle <arnout@mind.be>
Reviewed-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

---
Changes v1 -> v2:
  - fix typo in BR2_PACAKGE_SYSTEMD  (Arnout)
---
 package/freerdp/freerdp.mk | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Samuel Martin Nov. 12, 2017, 9:18 a.m. UTC | #1
On Sun, Nov 12, 2017 at 9:13 AM, Yann E. MORIN <yann.morin.1998@free.fr> wrote:
> FreeRDP now has support for writing to the journal, so we must ensure
> proper dependency and correct options.
>
> Partially fixes:
>     http://autobuild.buildroot.org/results/81aa66ddd88919295ccb5f34b527b737627263a7/
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Arnout Vandecappelle <arnout@mind.be>
> Reviewed-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Reviewed-by: Samuel Martin <s.martin49@gmail.com>

Regards,
Peter Korsgaard Nov. 13, 2017, 8:25 p.m. UTC | #2
>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 > FreeRDP now has support for writing to the journal, so we must ensure
 > proper dependency and correct options.

 > Partially fixes:
 >     http://autobuild.buildroot.org/results/81aa66ddd88919295ccb5f34b527b737627263a7/

 > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
 > Cc: Arnout Vandecappelle <arnout@mind.be>
 > Reviewed-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

 > ---
 > Changes v1 -> v2:
 >   - fix typo in BR2_PACAKGE_SYSTEMD  (Arnout)

Committed, thanks.
diff mbox series

Patch

diff --git a/package/freerdp/freerdp.mk b/package/freerdp/freerdp.mk
index 6d1664b86e..458bd5eea7 100644
--- a/package/freerdp/freerdp.mk
+++ b/package/freerdp/freerdp.mk
@@ -56,6 +56,14 @@  else
 FREERDP_CONF_OPTS += -DWITH_PULSEAUDIO=OFF
 endif
 
+# For the systemd journal
+ifeq ($(BR2_PACKAGE_SYSTEMD),y)
+FREERDP_CONF_OPTS += -DWITH_LIBSYSTEMD=ON
+FREERDP_DEPENDENCIES += systemd
+else
+FREERDP_CONF_OPTS += -DWITH_LIBSYSTEMD=OFF
+endif
+
 ifeq ($(BR2_ARM_CPU_HAS_NEON),y)
 FREERDP_CONF_OPTS += -DWITH_NEON=ON
 else