qmk / qmk_api

QMK API, information on keyboards and keymaps, and a compile service.
http://api.qmk.fm/
Other
34 stars 22 forks source link

[Bug] docker-compose up fails due to github policy changes #54

Open marfrit opened 2 years ago

marfrit commented 2 years ago

Describe the Bug

When running docker-compose up, the step fails:


 ---> Running in 518b6a377240
Collecting git+git://github.com/qmk/qmk_compiler.git@master
  Cloning git://github.com/qmk/qmk_compiler.git (to revision master) to /tmp/pip-req-build-9zduddus
  Running command git clone --filter=blob:none --quiet git://github.com/qmk/qmk_compiler.git /tmp/pip-req-build-9zduddus
  fatal: remote error:
    The unauthenticated git protocol on port 9418 is no longer supported.
  Please see https://github.blog/2021-09-01-improving-git-protocol-security-github/ for more information.
  error: subprocess-exited-with-error

  × git clone --filter=blob:none --quiet git://github.com/qmk/qmk_compiler.git /tmp/pip-req-build-9zduddus did not run successfully.
  │ exit code: 128
  ╰─> See above for output.

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× git clone --filter=blob:none --quiet git://github.com/qmk/qmk_compiler.git /tmp/pip-req-build-9zduddus did not run successfully.
│ exit code: 128
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Service 'qmk_api' failed to build: The command '/bin/sh -c pip3 install -r requirements.txt git+git://github.com/qmk/qmk_compiler.git@master git+git://github.com/skullydazed/kle2xy.git@master' returned a non-zero code: 1

### Additional Context?

_No response_
fauxpark commented 2 years ago

https://github.com/qmk/qmk_api/blob/master/Dockerfile#L7 https://github.com/qmk/qmk_api_tasks/blob/master/Dockerfile#L7 https://github.com/qmk/qmk_bot/blob/2d3281e44dcc34a2efa8265d4217eceac8a218b1/Dockerfile#L7

qmk_bot submodule apparently needs to be updated, the above line is no longer present on master and shouldn't be an issue.