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

Use v3 working directory for go-ldap #12884

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

johnweldon
Copy link

No description provided.

Copy link

johnweldon (verified) is either the primary contact or is in the CCs list of projects/go-ldap.

@DavidKorczynski DavidKorczynski enabled auto-merge (squash) December 27, 2024 18:28
@DonggeLiu
Copy link
Contributor

Strange that CI results did not load.
Let's wait a bit longer just in case.

@DavidKorczynski
Copy link
Collaborator

@johnweldon it looks like the build is failing. Did you confirm the changes work? You can run the following locally to replicate the issue:

python3 infra/helper.py build_fuzzers go-ldap

@AdamKorcz
Copy link
Collaborator

AdamKorcz commented Jan 2, 2025

@johnweldon You have removed the fuzz tests in https://github.com/go-ldap/ldap/pull/542/files#diff-4a0954693dd52efba878ef592132c50fa85c54acfb61641e79fe080df92a072b without moving them to the v3 subdir: https://github.com/go-ldap/ldap/tree/master/v3. OSS-Fuzz is trying to build the fuzzers but can't find them, so the build fails.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants