8000 Dulwich: Initial integration by DaveLak · Pull Request #11900 · google/oss-fuzz · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Dulwich: Initial integration #11900

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

Merged
merged 2 commits into from
May 7, 2024

Conversation

DaveLak
Copy link
Contributor
@DaveLak DaveLak commented May 3, 2024

Dulwich is a Git implementation written in Python with native extensions written with Rust. From the Dulwich README:

It aims to provide an interface to git repos (both local and remote) that doesn't call out to git directly but instead uses pure Python.

Maintainer's Approval

@jelmer is the project's maintainer and approved of this integration proposal here: jelmer/dulwich#1302 (comment). My email is in the auto_ccs list so I can assist with bug triage as discussed in that same thread.

Popularity in the Python Ecosystem

At the time of this PR, pypistats.org lists the following:

  • Downloads last day: 1,432,072
  • Downloads last week: 7,864,024
  • Downloads last month: 35,371,675

Dulwich is mentioned1 as one of the "most popular embeddable Git libraries"2 in the book Pro Git 2nd Edition by Scott Chacon and Ben Straub, and is a dependency of other popular projects such as magic-wormhole/magic-wormhole, openstack/reno, PrefectHQ/prefect, tahoe-lafs/tahoe-lafs, and many more.

Footnotes

  1. Pro Git 2nd Edition, A2.5 Appendix B: Embedding Git in your Applications - Dulwich

  2. Pro Git 2nd Edition, A2.1 Appendix B: Embedding Git in your Applications - Command-line Git

Copy link
github-actions bot commented May 3, 2024

DaveLak is integrating a new project:
- Main repo: https://github.com/jelmer/dulwich
- Criticality score: 0.63489

DaveLak added a commit to DaveLak/dulwich that referenced this pull request May 3, 2024
Introduces an initial fuzzing test and supporting files for
integrating Dulwich into OSS-Fuzz as discussed in:
jelmer#1302

The corresponding PR on the OSS-Fuzz repo is:
google/oss-fuzz#11900
DaveLak added a commit to DaveLak/dulwich that referenced this pull request May 3, 2024
Introduces an initial fuzzing test and supporting files for
integrating Dulwich into OSS-Fuzz as discussed in:
jelmer#1302

The corresponding PR on the OSS-Fuzz repo is:
google/oss-fuzz#11900
DaveLak added a commit to DaveLak/dulwich that referenced this pull request May 3, 2024
Introduces an initial fuzzing test and supporting files for
integrating Dulwich into OSS-Fuzz as discussed in:
jelmer#1302

The corresponding PR on the OSS-Fuzz repo is:
google/oss-fuzz#11900
@DaveLak
Copy link
Contributor Author
DaveLak commented May 3, 2024

Corresponding PR on the Dulwich repo: jelmer/dulwich#1304

CI won't pass here until that is merged, but I thought it better to get the reviews started here and there in parallel.

8000

jelmer added a commit to jelmer/dulwich that referenced this pull request May 4, 2024
As discussed in issue #1302, this PR introduces an initial fuzzing test
and supporting files for integrating Dulwich into OSS-Fuzz. I've also
opened a corresponding PR on the OSS-Fuzz repo proposing the integration
here: google/oss-fuzz#11900

Note that CI will continue to fail on that PR until this one is merged
because it depends on the scripts added here.

I hope that the`fuzzing/README.md` sufficiently describes what is being
introduced here, but please do let me know if there is anything I can
clarify or change to make it fit better into the rest of the repository.

I also ran `ruff` and updated `NEWS` as requested in `CONTRIBUTING.rst`.

Thanks!
@jonathanmetzman jonathanmetzman merged commit 9ed2a55 into google:master May 7, 2024
@DaveLak DaveLak deleted the initial-integration-dulwich branch May 7, 2024 20:12
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
0