Conversation
Finally converted usbutils over to meson from autotools. Odds are I messed something up, so I should do a release soon to see what broke in the distros as I have no idea how they package this thing anymore:
https://git.sr.ht/~gregkh/usbutils/commit/86dcab8964901d15b87b156546559c8ae28bd9fe
4
8
21

@gregkh Only one I notice are the cc = meson.get_compiler('c') in each meson.build which I'm pretty sure are useless as the cc variable isn't used anywhere.

1
0
0

@gregkh don't mind sending a patch or two. Although looking around I see no information how to contribute.

Just finished porting kmod, although my goal s to have 1-2 release overlap (with both meson and autotools) so that distros do not jump on broken glass.

2
0
0
@xexaxo Email works great for patches, or, if you really only like the github workflow, you can send a pull request to https://github.com/gregkh/usbutils as is mentioned in the README
1
0
0
@lanodan Ah, a hold-over for when I was messing around with the cflags. I'll work on adding proper support for more flags and then that variable will be used.

Thanks for taking a look!
0
0
2
@xexaxo And congrats on converting kmod, brave of you to keep two build systems running at once.
0
0
1

@gregkh great to hear!
If I find some time later today I could test your current default branch.

This is how I currently package it for Arch: https://gitlab.archlinux.org/archlinux/packaging/packages/usbutils/-/blob/main/PKGBUILD

It's pretty straight forward with autotools currently. I think the only oddity is installing usbreset manually (someone wanted it).

1
0
0
@dvzrv Cool, let me test that out as you know, that's the distro I use...

And it should be much easier to install usbreset now, perhaps I should just switch to always installing it by default and see if anyone objects...
0
0
1

@gregkh
Aside from `cc`, `pkg` and `config` are also redundant in `usbhid-dump/meson.build`.
Could also use the `files()` function though not required in this use case.

0
0
0

@gregkh fwiw I'm not that tied to GitHub/GitLab/etc, but since I didn't see a public ML... A dozen patch pull request is out.

There is another dozen locally, fixing compiler issues and adding CI, assuming you are ok with the idea.

1
0
0
@xexaxo Thanks for the changes, and the work, much appreciated!
0
0
1