-
Notifications
You must be signed in to change notification settings - Fork 177
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
qualify required gssapi.h during samba 4.16.0 build #984
Comments
This patch is inappropriate for Heimdal for two reasons. First, Samba includes its own fork of Heimdal and that fork should be used when building Samba. Second, the proposed patch does not fix an issue in Heimdal but works around a deficiency in LibreELEC. gssapi.h is a public header that should not be included with a relative path. |
Hi @jaltman - I agree with your summation. Heimdal is building by itself correctly👍. The samba included Heimdal is being used for the samba build. The only reason for the separate build in LE is to allow the cross compile with the needed heimdal_asn1_compile and heimdal_compile_et. For whatever reason - I’m sure to come across the issue in why the samba build in LE is picking up the wrong gssapi.h. Issue raised was more as an fyi - and as a workaround until we can discover why the samba build is inconsistently building. |
It is not safe to assume that the output from the asn1_compiler is stable across Heimdal versions. When building Samba the version shipped as part of Samba must be used. |
Please do not raise issues found in Samba's internal use of Heimdal
here. The Heimdal maintainers are busy enough, and header include path
issues are almost certainly due to Samba's build system, not the
upstream code.
Should such a fix be needed upstream, then we can all come back here,
with the justification and a tested fix that has been though Samba's CI
also.
Andrew Bartlett
|
Ticket raised at https://bugzilla.samba.org/show_bug.cgi?id=15033 @jaltman / @abartlet - firstly - apologies for creating the ticket as an FYI in Heimdal too, and potentially causing additional work - was not my intention. Was as an FYI as I had miss understood from the release notes that the Samba Heimdal was now closely tracking the GitHub. I do agree - a LibreELEC + Samba build issue. @jaltman - thanks for the insight on asn1. The LE build has been this way, and whilst now maintaining the package currency - only came across the compatibility issue when updating to 4.16 from 4.13. Looking at the change logs, and your pointer - I can only guess at why we didn’t use a host build of Samba’s Heimdal for these 2 binaries (something for the LE backlog 🙂) |
I do try not to break the ABI nor API when I make changes to the ASN.1 compiler, but at some point I will have to, and you should assume the ABI for it breaks with every release. (In particular the DER encode/decode functions really need to be able to take an optional alternative outer tag for best supporting IMPLICIT tagging. I could add more entry points, but why bloat things when I can break things? 🙂) |
Whilst this is a build bug with LibreELEC + samba - though I would raise the issue here too.
I am in the process of raising a samba bug for this too.
heimdal/lib/gssapi/gssapi/gssapi_krb5.h
Line 39 in 6b45c35
During a cross compile we are having inconsistent builds of Samba 4.16.0 with the gssapi.h.
a) We separately build heimdal_asn1_compile and heimdal_compile_et and are using heimdal/heimdal:master without issue.
I have put together the following workaround to deal with the build failure. Not really sure the right way to have this fixed.
I have confirmed that the only gssapi.h files in the samba build are:
LibreELEC/LibreELEC.tv@810be4a
== Detail below ==
samba: qualify required gssapi.h
Inconsistent builds with samba 4.16 with the wrong gssapi.h being picked up. This is a workaround, and being raised upstream.
WORKAROUND PATCH
The text was updated successfully, but these errors were encountered: