HUION: Add auto generated description files

[JoseExposito]

This PR adds a bunch of HUION tablet description files and SVGs generated with this tool:
https://github.com/JoseExposito/uclogic-libwacom-tablet-descriptions

While incomplete, basic information like the number of buttons, number of touch strips and the device class (bamboo/cintiq) is correct.

[whot]

This LGTM and is obviously a massive boon in tablet files, so thanks heaps for that.

I'm wondering what the overlap is between these new devices and the ones we already have. IOW I think some of these new devices were already matched by existing files and may break/change now but then again that should bring the users out of the woodwork to fix the layout files :)

[JoseExposito]

I'm wondering what the overlap is between these new devices and the ones we already have. IOW I think some of these new devices were already matched by existing files and may break/change now but then again that should bring the users out of the woodwork to fix the layout files :)

I deleted manually the description files already present in libwacom to avoid clashes. Have I override any accidentally?

[whot]

I deleted manually the description files already present in libwacom to avoid clashes. Have I override any accidentally?

I don't see any deleted files in this MR so I'm not sure what you meant there? My comment was related to: huion re-uses the USB IDs and the names so we have .tablet files that match tablets they weren't intended to. With this MR this should be fixed but it may cause a change in a previously overmatched tablet. Again, it may happen but I'm not sure we need to worry about it...

[whot]

To illustrate my comment above: the only .tablet file that is overwritten by the autogenerated results in JoseExposito/uclogic-libwacom-tablet-descriptions#1 is the HS611 and that shows the issue quite nicely:

- DeviceMatch=usb|256c|006d|HUION Huion Tablet_HS611 Pad;usb|256c|006d|HUION Huion Tablet_HS611 Touch Strip;usb|256c|006d|HUION Huion Tablet_HS611 Pen
+ DeviceMatch=usb|256c|006e||HUION_T19c;usb|256c|006d||HUION_T19c;

This device would indeed break if we were to replace the tablet file - anyone with older digimend patches won't get the uniq set and because we have no fallback match we won't match this device anymore.

This means we need to add the autogenerated patches in addition to the existing ones until we can reasonably expect users to carry the updated kernel patches. Our match system goes name+uniq first, then uniq, then name so for tablets where we have both uniq and old-style name match defined we should match correctly either way.

So it's just a bit more messy and on older systems we still get overmatching where the vid/pid/name are shared but overall it Should Just Work^tm :)

[JoseExposito]

To illustrate my comment above: the only .tablet file that is overwritten by the autogenerated results in JoseExposito/uclogic-libwacom-tablet-descriptions#1 is the HS611

Ops, I didn't automate the removal of devices already present in libwacom and instead I manually deleted them... And manual processes tend to have errors. That file shouldn't be there, I removed it in the latest revision.

This PR should contain only new tablets. If the tablet was already present in libwacom, we should use them.

[JoseExposito]

Added an extra commit with 70 tablets (that should be all tablets) from GAOMON.

[whot]

Alrighty, I've merged all the other PRs with the groundwork which means you should be able to rebase and get a) good enough testing and b) useful error messages for the SVG validity tester.

Added an extra commit with 70 tablets (that should be all tablets) from GAOMON.

awesome, thanks. I'm not looking forward to adding all those to the NEWS file :)

[JoseExposito]

Thanks a lot Peter, after rebasing on top of master CI is happy 🎉

[whot]

In that case, let's get this merged! Thanks heaps for all the effort, this is certainly the biggest commit enabling new tablets ever :)