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

Build librdkafka with gssapi support for pip package, fixes #325 #352

Draft
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

timkpaine
Copy link
Member

@timkpaine timkpaine commented Aug 2, 2024

As title, builds librdkafka with GSSAPI support via cyrus-sasl, enabling this feature for our pip packages (currently this only works on our conda packages).

No windows support for now as vcpkg krb5 does not support our static build, will make a separate ticket to track

xref: microsoft/vcpkg#40320

@timkpaine timkpaine force-pushed the tkp/gssapi branch 10 times, most recently from f7d5cf6 to 8f52943 Compare August 8, 2024 02:34
@timkpaine timkpaine added part: dependencies PRs that update a dependency file adapter: kafka Issues and PRs related to the Apache Kafka adapter part: build Issues and PRs related to the build process labels Aug 9, 2024
Signed-off-by: Tim Paine <Timothy.Paine@CubistSystematic.com>
os.environ["ACLOCAL_PATH"] = f"{aclocal_path}:{os.environ.get('ACLOCAL_PATH', '')}"
except subprocess.CalledProcessError as e:
print("Error obtaining Homebrew prefix for autoconf-archive:", e)
aclocal_path = None
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no need to set to none

except subprocess.CalledProcessError as e:
print("Error obtaining Homebrew prefix for autoconf-archive:", e)
aclocal_path = None
elif platform.system() != "Windows":
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will do this no matter the context, i think it probably makes more sense to only do this if we’re doing a manylinux build in the docker image since thats where the problem seems to be

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adapter: kafka Issues and PRs related to the Apache Kafka adapter part: build Issues and PRs related to the build process part: dependencies PRs that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants