| Summary: | Advanced synchronization a no-op with certain destination references | ||
|---|---|---|---|
| Product: | [Technology] EGit | Reporter: | R Shapiro <rshapiro> |
| Component: | Core | Assignee: | Dariusz Luksza <dariusz.luksza> |
| Status: | ASSIGNED --- | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | avantnoisebot, eclipse |
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | |||
|
Description
R Shapiro
Note also that the egit 'Compare With' operation works with bridge references. For example if I right-click a file, select 'Compare With -> Branch, Tag or Reference...', the bridge references appear under 'Remote Tracking', as expected, and selecting any of these entries does the diff, as expected. So in general egit/jigit seem fine with using SVN bridge references like any other Git reference. It turns out that synchronize assumes that all remote branches belongs to remote repository (eg. refs/remotes/origin/master). Looks like this assumption isn't true for repositories created by git-svn bridge. In couple of minutes I'll push change set for review that should fix this bug. Change set 4582[1] is fixing this issue [1] http://egit.eclipse.org/r/4582 As Dariusz notes, the bug (and it really is a bug) is in the assumption that any reference under refs/remotes must be a remote-reference: Git does not require references under .git/refs/remotes to have this form. Testing his fix is very simple: just add a file to .git/refs/remotes called "advance-sync-test" whose contents is a valid SHA, then run advanced synchronization and select that reference. Assuming this works, run advanced synchronization on a true remote-reference to ensure that still works. That's it. Let me emphasize again just how trivial it is to test this: 1) cp .git/refs/remotes/origin/master .git/refs/remotes/test-363173 2) Run Team - Advanced -> Synchronize -> refs/remotes/test-363173. That's it. The whole process should take under a minute. *** Bug 350804 has been marked as a duplicate of this bug. *** |