Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix build for >= 5.18 #275

Merged
merged 5 commits into from
Jun 1, 2022
Merged

Fix build for >= 5.18 #275

merged 5 commits into from
Jun 1, 2022

Conversation

tomaspinho
Copy link
Owner

No description provided.

@witchent
Copy link

The fix does not seem to work for me. The last few lines of my error are

ld: /var/lib/dkms/rtl8821ce/v5.5.2_34066.20200325/build/core/rtw_mp.o: in function `is_zero_mac_addr':
/var/lib/dkms/rtl8821ce/v5.5.2_34066.20200325/build/include/ieee80211.h:1599: multiple definition of `is_zero_mac_addr'; /var/lib/dkms/rtl8821ce/v5.5.2_34066.20200325/build/core/rtw_cmd.o:/var/lib/dkms/rtl8821ce/v5.5.2_34066.20200325/build/include/ieee80211.h:1599: first defined here
make[2]: *** [scripts/Makefile.build:308: /var/lib/dkms/rtl8821ce/v5.5.2_34066.20200325/build/8821ce.prelink.o] Error 1
make[1]: *** [Makefile:1834: /var/lib/dkms/rtl8821ce/v5.5.2_34066.20200325/build] Error 2
make[1]: Leaving directory '/usr/lib/modules/5.18.1-arch1-1/build'
make: *** [Makefile:2244: modules] Error 2

Please tell me if you need a full log.

@zalokin
Copy link

zalokin commented May 31, 2022

No working for my with linux-headers-5.18.1.arch-1 :(

core/linux-headers 5.18.1.arch1-1 [installed]
Headers and scripts for building modules for the Linux kernel

--- updated ---

https://kernel.ubuntu.com/~kernel-ppa/mainline/v5.18.1/

the debs folder it's created yesterday for header 5.18.1, it only remains to wait and change URL in dockerfile.

@tomaspinho
Copy link
Owner Author

@witchent pushed a fix for those multiple definitions

@zalokin seems like the mainline kernel is failing to build for them as no .debs are available yet

@witchent
Copy link

witchent commented Jun 1, 2022

That did the trick

@tomaspinho
Copy link
Owner Author

Merging

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.

None yet

3 participants