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

Clarify build requirements for NTSync5 in .cfg files #1268

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

kangtastic
Copy link
Contributor

ntsync is part of the kernel UAPI, so the presence of the ntsync module on the build host is not required to build Wine with ntsync support. A complete ntsync header file at /usr/include/linux/ntsync.h (replacing, if applicable, the incomplete stub mainlined in kernel 6.10) is sufficient.

Note also that the proposed ntsync patchset targets kernels >= 6.8, but builds still work if the build host is running an earlier kernel. This is most immediately relevant for CI, as the ubuntu-latest GitHub Actions runner has kernel 6.5.

…n-tkg.cfg

ntsync is part of the kernel UAPI, so the presence of the
ntsync module on the build host is not required to build
Wine with ntsync support. A complete ntsync header file at
/usr/include/linux/ntsync.h (replacing, if applicable, the
incomplete stub mainlined in kernel 6.10) is sufficient.

Note also that the proposed ntsync patchset targets kernels
>= 6.8, but builds still work if the build host is running
an earlier kernel. This is most immediately relevant for CI,
as the ubuntu-latest GitHub Actions runner has kernel 6.5.
@Tk-Glitch
Copy link
Member

Yup!
Thanks again kangtastic 🐸 ❤️

@Tk-Glitch Tk-Glitch merged commit 987fcff into Frogging-Family:master Sep 26, 2024
@kangtastic
Copy link
Contributor Author

Thanks for accepting these small improvements.

From another Galaxy S kernel hobbyist, see you around 😅

@kangtastic kangtastic deleted the ntsync-cfg-fixup branch September 27, 2024 01:37
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.

2 participants