gerrit: commit-msg hook download instructions are incorrect

#181
Opened by tazjin at 2022-05-23T12·47+00

A recent SSH release broke compatibility of the scp command line tool by switching the transfer protocol from the legacy SCP protocol to SFTP.

Gerrit's SSH server (as of the version we are running) does not support SFTP so users get confusing error messages when trying to execute the scp command that Gerrit instructs them to run.

This can be fixed on the client side by adding -O to the scp command line parameters, which switches back to the older protocol. However, either Gerrit should be updated to support the newer protocol OR the instructions for downloading the hook should include -O.

The problem is that an inverse error will probably happen if someone on an older version attempts to use -O ... :|

  1. This issue in upstream Gerrit: https://bugs.chromium.org/p/gerrit/issues/detail?id=15944

    My workaround: https://gerrit-review.googlesource.com/c/gerrit/+/338074

    tazjin at 2022-05-27T10·01+00

  2. Apparently a slightly better workaround (fetching the hook via HTTP) has been merged in upstream Gerrit (https://gerrit-review.googlesource.com/c/plugins/download-commands/+/359823).

    We should either backport, or probably check how many of the patches in third_party/gerrit/default.nix we still need (and check if we can just use the nixpkgs version again).

    flokli at 2023-10-06T12·40+00

  3. Gerrit has been bumped, this can be closed.

    flokli at 2023-11-15T22·28+00

  4. flokli closed this issue at 2023-11-15T22·28+00