Skip to content

Conversation

@KOLANICH
Copy link

No description provided.

@Cropi
Copy link
Owner

Cropi commented Sep 6, 2021

Hi,

Thank you for your PR. We appreciate it.
Just one quick question, have you experienced any trouble using autotools ?

@Cropi Cropi requested a review from ZoltanFridrich September 6, 2021 05:36
@KOLANICH
Copy link
Author

KOLANICH commented Sep 6, 2021

have you experienced any trouble using autotools ?

Both yes and no.

No: I know the generic workflow of autotools: autoreconf -i; ./configure --help; examine available flags; ./configure --wanted-flags; make; DESTDIR=./destDir make install; package from the dest dir manually (I have written (but now it is inoperational, it has been broken somewhen, I have no time to fix it currently) a tool wrapping multiple build systems and generating the binary packages).
Yes: autotools is always a pain to deal with. It is hard to edit the scripts, it is slow, too tied to bash, it doesn't support smarter build backens like ninja. I am also not enough familar to it, so my estimation were biased.

Also I should mention that I had to manually edit autotools scripts to disable building of the docs, because to build docs large deps are needed.

Also I should mention that I got a big problem with the lack of libusbguard-dev Debian package, so I had to download usbguard sources and create a temporary replacement for one. The proper long-term solution is of course to make distros to shio this package too.

I also should mention that neither of the built binaries worked well for me. My desktop environment is KDE and the OS (currently, I guess quite soon it will be 21.10) is Kubuntu 21.04. First I noticed that the notifier systemd service has been started but I got no notifications on USB devices attachment. After a few restarts the service stopped starting too. Currently I have no time to investigate this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants