Option to use elogind instead of systemd

albfan / miraclecast

Connect external monitors to your system via Wifi-Display specification also known as Miracast

Other

3.68k stars 408 forks source link

Option to use elogind instead of systemd #434

Closed Perlovka closed 1 year ago

Perlovka commented 2 years ago

Hi, this patch is based on https://github.com/albfan/miraclecast/pull/317

CMake option: -DUSE_SYSTEMD=OFF (ON is the default)
Meson option: -Duse-systemd=false (true is the default)

Both CMake and Meson builds ran successful. Binaries compiled with CMake has been tested successfully (screen cast from android to laptop)

albfan commented 2 years ago

Looks good, thanks. See there's an autotools buildsystem too. Can you tweak that too?

Perlovka commented 2 years ago

@albfan I've tried my best, not familiar with autotools ;) Autotools patch has no options, it will detect which package is installed (systemd or elogind) and will build against it. Build tested with elogind - works for me.

albfan commented 2 years ago

Thanks a lot, I postponed for some time to enable CI on PR, will setup this and if this just works will merge it.

Thanks a lot for this

albfan commented 2 years ago

Hmm, maybe you have elogind and systemd at same time installed?

I miss using HAVE_SYSTEMD to exclude some #includes like #441

@Perlovka can you check it?

albfan commented 1 year ago

this is superseed by #441, autotools management is more explicit and dependencies are managed better with DEPS