Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 349453 - Paste remote repository URL into Git Repositories View
Summary: Paste remote repository URL into Git Repositories View
Status: VERIFIED FIXED
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Matthias Sohn CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-15 11:00 EDT by Wayne Beaton CLA
Modified: 2012-01-19 04:19 EST (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Wayne Beaton CLA 2011-06-15 11:00:25 EDT 
I copied a Git URL from a GitHub repository and attempted to paste it into the Git Repositories view, resulting in a  "Clipboard content is not a directory path error". Intuitively, this seems like a reasonable enough thing to want to do.

At a minimum, the error should probably be reported in a slightly less intimidating manner. The error dialog, combined with the entry in the error log, makes it look like a bug.
Comment 1 Dani Megert CLA 2011-08-25 05:36:14 EDT 
The feature seems to be broken

I tried to paste:
/gitroot/jdt/eclipse.jdt.ui.git
ssh://git.eclipse.org/gitroot/jdt/eclipse.jdt.ui.git
and what I got from 'Copy Path to Clipboard'

None of this works. I get an error dialog, saying it's not a directory path.
Comment 2 Matthias Sohn CLA 2011-08-25 05:43:49 EDT 
pushed patches for review: 

"Paste Repository Path" should accept parent folder of git directory
http://egit.eclipse.org/r/#change,4058 (already submitted)

"[repoView] Enable pasting a URI to initiate cloning"
http://egit.eclipse.org/r/#change,4061
Comment 3 Dani Megert CLA 2011-08-25 05:45:04 EDT 
That's fast :-)
Comment 4 Kevin Sawicki CLA 2011-11-08 20:31:27 EST 
Matthias, can this be closed?  Looks like both patches were merged.
Comment 5 Dani Megert CLA 2011-11-09 05:15:03 EST 
(In reply to comment #4)
> Matthias, can this be closed?  Looks like both patches were merged.

I can confirm that it now works as expected.
Comment 6 Kevin Sawicki CLA 2011-11-09 11:07:47 EST 
Thanks Dani, resolving as fixed.