[Webkit-unassigned] [Bug 256887] New: [git-webkit] pr from a detached head leads to surprising errors

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 17 04:44:26 PDT 2023


https://bugs.webkit.org/show_bug.cgi?id=256887

            Bug ID: 256887
           Summary: [git-webkit] pr from a detached head leads to
                    surprising errors
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Tools / Tests
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: gsnedders at apple.com
                CC: jbedard at apple.com, webkit-bug-importer at group.apple.com

Bug 240468 apparently fixed this, but this once again doesn't work. Clearly we need to be careful about where we push to (what do we call it?), but at the very least we should exit with some more meaningful error (like "cannot push from a detached HEAD") or we should come up with some default branch name to push to.

```
gsnedders at gsnedders-marsha OS2 % git status
interactive rebase in progress; onto 4518454316d9
Last commands done (3 commands done):
   squash b256b58e80bd Test W3C import with existing tests-options.json data https://bugs.webkit.org/show_bug.cgi?id=256851
   break
  (see more in file /Volumes/gsnedders/projects/Safari/OpenSource/.git/worktrees/OS2/rebase-merge/done)
No commands remaining.
You are currently editing a commit while rebasing branch 'wpt-importer-extra-unittests-256851' on '4518454316d9'.
  (use "git commit --amend" to amend the current commit)
  (use "git rebase --continue" once you are satisfied with your changes)

nothing to commit, working tree clean
gsnedders at gsnedders-marsha OS2 % ./Tools/Scripts/git-webkit pr
branch 'main' set up to track 'origin/main'.
    Found 1 commit...
Total errors found: 0 in 1 files
error: The destination you provided is not a full refname (i.e.,
starting with "refs/"). We tried to guess what you meant by:

- Looking for a ref that matches 'HEAD' on the remote side.
- Checking if the <src> being pushed ('HEAD')
  is a ref in "refs/{heads,tags}/". If so we add a corresponding
  refs/{heads,tags}/ prefix on the remote side.

Neither worked, so we gave up. You must fully qualify the ref.
hint: The <src> part of the refspec is a commit object.
hint: Did you mean to create a new branch by pushing to
hint: 'HEAD:refs/heads/HEAD'?
error: failed to push some refs to 'github.com:gsnedders/WebKit.git'
Failed to push 'HEAD' to 'fork' (alias of 'git at github.com:gsnedders/WebKit.git')
Your checkout may be mis-configured, try re-running 'git-webkit setup' or
your checkout may not have permission to push to 'git at github.com:gsnedders/WebKit.git'
```

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20230517/f178146c/attachment.htm>


More information about the webkit-unassigned mailing list