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

[git-webkit] Handle ssh protocol with user #11629

Conversation

JonWBedard
Copy link
Member

@JonWBedard JonWBedard commented Mar 16, 2023

bb518ed

[git-webkit] Handle ssh protocol with user
https://bugs.webkit.org/show_bug.cgi?id=254052
rdar://106832995

Reviewed by Aakash Jain.

A remote URL of the format 'ssh://git@...' is valid, but breaks our regex.

* Tools/Scripts/libraries/webkitscmpy/setup.py: Bump version.
* Tools/Scripts/libraries/webkitscmpy/webkitscmpy/__init__.py: Ditto.
* Tools/Scripts/hooks/pre-push: Use a regex defined by our template.
* Tools/Scripts/libraries/webkitscmpy/webkitscmpy/program/install_hooks.py:
(InstallHooks): Update REMOTE_RE to handle ssh protocol with user.
(InstallHooks.main): Pass REMOTE_RE to template.
* Tools/Scripts/libraries/webkitscmpy/webkitscmpy/test/install_hooks_unittest.py: Added.
(TestInstallHooks):
(TestInstallHooks.setUp):
(TestInstallHooks.test_regex):

Canonical link: https://commits.webkit.org/261795@main

5abb8db

Misc iOS, tvOS & watchOS macOS Linux Windows
  πŸ§ͺ style   πŸ›  ios   πŸ›  mac   πŸ›  wpe   πŸ›  wincairo
  πŸ§ͺ bindings   πŸ›  ios-sim   πŸ›  mac-AS-debug   πŸ§ͺ wpe-wk2
  πŸ§ͺ webkitperl   πŸ§ͺ ios-wk2   πŸ§ͺ api-mac   πŸ›  gtk
  πŸ§ͺ webkitpy   πŸ§ͺ api-ios   πŸ§ͺ mac-wk1   πŸ§ͺ gtk-wk2
  πŸ›  tv   πŸ§ͺ mac-wk2   πŸ§ͺ api-gtk
  πŸ›  tv-sim   πŸ§ͺ mac-AS-debug-wk2
  πŸ§ͺ services   πŸ›  watch   πŸ§ͺ mac-wk2-stress
  πŸ›  watch-sim
βœ… πŸ›  πŸ§ͺ unsafe-merge

@JonWBedard JonWBedard self-assigned this Mar 16, 2023
@JonWBedard JonWBedard added the Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases label Mar 16, 2023
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Mar 17, 2023
@JonWBedard JonWBedard removed the merging-blocked Applied to prevent a change from being merged label Mar 17, 2023
@JonWBedard JonWBedard force-pushed the eng/git-webkit-Handle-ssh-protocol-with-user branch from df85b4a to 5abb8db Compare March 17, 2023 15:53
@JonWBedard JonWBedard added skip-ews Applied to prevent a change from being run on EWS unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing labels Mar 17, 2023
https://bugs.webkit.org/show_bug.cgi?id=254052
rdar://106832995

Reviewed by Aakash Jain.

A remote URL of the format 'ssh://git@...' is valid, but breaks our regex.

* Tools/Scripts/libraries/webkitscmpy/setup.py: Bump version.
* Tools/Scripts/libraries/webkitscmpy/webkitscmpy/__init__.py: Ditto.
* Tools/Scripts/hooks/pre-push: Use a regex defined by our template.
* Tools/Scripts/libraries/webkitscmpy/webkitscmpy/program/install_hooks.py:
(InstallHooks): Update REMOTE_RE to handle ssh protocol with user.
(InstallHooks.main): Pass REMOTE_RE to template.
* Tools/Scripts/libraries/webkitscmpy/webkitscmpy/test/install_hooks_unittest.py: Added.
(TestInstallHooks):
(TestInstallHooks.setUp):
(TestInstallHooks.test_regex):

Canonical link: https://commits.webkit.org/261795@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/git-webkit-Handle-ssh-protocol-with-user branch from 5abb8db to bb518ed Compare March 17, 2023 16:01
@webkit-commit-queue webkit-commit-queue merged commit bb518ed into WebKit:main Mar 17, 2023
@webkit-commit-queue
Copy link
Collaborator

Committed 261795@main (bb518ed): https://commits.webkit.org/261795@main

Reviewed commits have been landed. Closing PR #11629 and removing active labels.

@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Mar 17, 2023
@JonWBedard JonWBedard deleted the eng/git-webkit-Handle-ssh-protocol-with-user branch March 20, 2023 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-ews Applied to prevent a change from being run on EWS Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases
Projects
None yet
5 participants