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

[WIP] lib: nghttp2: upgrade to v1.62.1 #9046

Closed
wants to merge 3 commits into from

lib: nghttp2: try building with shared lib on

6f998f3
Select commit
Loading
Failed to load commit list.
Closed

[WIP] lib: nghttp2: upgrade to v1.62.1 #9046

lib: nghttp2: try building with shared lib on
6f998f3
Select commit
Loading
Failed to load commit list.
DCO / DCO required action Jul 6, 2024 in 1s

DCO

There are 6 commits incorrectly signed off. This means that the author(s) of these commits failed to include a Signed-off-by line in their commit message.

To avoid having PRs blocked in the future, always include Signed-off-by: Author Name <[email protected]> in every commit message. You can also do this automatically by using the -s flag (i.e., git commit -s).

Here is how to fix the problem so that this code can be merged.


Rebase the branch

If you have a local git environment and meet the criteria below, one option is to rebase the branch and add your Signed-off-by lines in the new commits. Please note that if others have already begun work based upon the commits in this branch, this solution will rewrite history and may cause serious issues for collaborators (described in the git documentation under "The Perils of Rebasing").

You should only do this if:

  • You are the only author of the commits in this branch
  • You are absolutely certain nobody else is doing any work based upon this branch
  • There are no empty commits in the branch (for example, a DCO Remediation Commit which was added using --allow-empty)

To add your Signed-off-by line to every commit in this branch:

  1. Ensure you have a local copy of your branch by checking out the pull request locally via command line.
  2. In your local branch, run: git rebase HEAD~81 --signoff
  3. Force push your changes to overwrite the branch: git push --force-with-lease origin nghttp2-1.62.1

Summary

Commit sha: 88afa90, Author: José Lecaros, Committer: GitHub; Expected "José Lecaros [email protected]", but got "lecaros [email protected]".
Commit sha: f6c650e, Author: José Lecaros, Committer: GitHub; Expected "José Lecaros [email protected]", but got "lecaros [email protected]".
Commit sha: 742ec1c, Author: José Lecaros, Committer: GitHub; Can not find "José Lecaros [email protected]", in ["lecaros [email protected]", "lecaros [email protected]"].
Commit sha: 456c305, Author: Meetp369, Committer: GitHub; Expected "Meetp369 [email protected]", but got "Meet [email protected]".
Commit sha: 7c8a4bb, Author: Pat, Committer: GitHub; Expected "Pat [email protected]", but got "Patrick Stephens [email protected]".
Commit sha: fe988b1, Author: protohuf, Committer: GitHub; Expected "protohuf [email protected]", but got "Marcus Hufvudsson [email protected]".