GoXLR-on-Linux / goxlr-on-linux

Documentation and scripts to make the GoXLR and GoXLR Mini useful on Linux.
142 stars 8 forks source link

Implemented new alsa method #38

Closed lm41 closed 2 years ago

FrostyCoolSlug commented 2 years ago

UCM seems to have been somewhat broken since 5.11, while the original script seems to have continued working..

Overwriting something which is normally handled as a package install might be ill-advised (as it may create conflicts and issues with updates)..

We're stuck in a position currently where there are a lot of factors involved in making the GoXLR work, Pulse vs Pipewire, Kernel Versions, UCM vs Script (If UCM is working, you shouldn't need the script), so a potentially deeper understanding of how everything fits together, and what configurations work or are broken may be a necessity at this point.. Manually dumping UCM config into /usr/share/alsa seems like a general bad idea, because tracking versioning may be difficult.

VirtualJunky commented 2 years ago

Agreed with what FrostyCoolSlug says. I think at a certain point once we know every kernal version that works and doesn't and different compatibility issues, we're going to end up with a much more complex version of the current install script that tries to install the best version for the given use case. I don't think we'll ever have a one size fits all though unless we can really push for kernal changes or some kind of standardization of a system that does work on all distros, which seems really unlikely to me.

Even in the proposed changes here on a minor side, we originally used $HOME like you have changed it to, but that didn't work on Pop_OS I believe which is why we had to change it months ago. Compatibility sucks and at this point I think we just keep building onto what we have now and know works, with more versions and fixes to fill in holes we come across all under the same install script.