search

jwrdegoede / rtl8189ES_linux

78 stars 90 forks source link

Change kernel compile tests to latest kernels #31

Closed jerbob92 closed 3 years ago

jerbob92 commented 4 years ago

This changes the tests to the kernels currently listed on https://www.kernel.org/

I had to change the gcc for 5.4 and 5.5 to gcc-8 and gcc-9, not sure if that's what we want, but that was the fix to make it compile again.

CGarces commented 4 years ago

Hi @jerbob92

Thanks for your changes. I have tried also to fix the compilation issues with CGG < 7.5 without any success. The issue is caused by this commit: https://github.com/torvalds/linux/commit/40576e5e63ea5eeab814de7af83ad09dd64134ee https://github.com/torvalds/linux/blob/v5.4-rc2/include/linux/compiler_types.h#L209-L213

Please consider update the build environment to dist: bionic. It made the build more compact removing some hacks. Use this file as template https://github.com/aircrack-ng/rtl8812au/blob/v5.6.4.2/.travis.yml

For me its ok build only with GCC 8 & 9 only. Note that GCC 7.5 also should work but not is part (not yet) of bionic.

CGarces commented 4 years ago

Can you consider use this? Maybe last mainline and last LTS is enough for the build matrix.

jerbob92 commented 4 years ago

Why limit the build matrix? I think it's good to make sure at least the supported kernel versions work.

CGarces commented 4 years ago

was just a suggestion. Can you please update the build todist: bionic ?

CGarces commented 3 years ago

Supersede by #46