Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 361502

Summary:	Need a way to create patches from non-committed workspace changes
Product:	[Technology] EGit	Reporter:	Uwe Stieber <uwe.st>
Component:	Core	Assignee:	Project Inbox <egit.core-inbox>
Status:	RESOLVED DUPLICATE	QA Contact:
Severity:	normal
Priority:	P3	CC:	cdtdoug, daniel_megert, markus.kell.r, mober.at+eclipse, nhb_web, remy.suen, tomasz.zarna
Version:	unspecified
Target Milestone:	---
Hardware:	PC
OS:	Windows 7
Whiteboard:
Bug Depends on:
Bug Blocks:	358616

Description Uwe Stieber

2011-10-20 03:40:11 EDT

For CVS and SVN our non-committer team members had been able to _easily_ create patches for their changes living in their workspace against HEAD. Since the move of the projects (CDT, TCF) to Git, creating a patch is a headache. As it appears to be, patches can be created only from a commit in the history. (A) non-committer should not require to commit changes locally to their repository clones and (B) even than, the patch contains content that hasn't been selected or is even part of the selected commit.

What's needed is:
- Create a patch from local workspace changes only (like CVS and SVN offers)
- Allow to select the content of the patch by workspace resource (create a patch only for plug-in A while having other changes in plug-in B, like CVS and SVN offers).

Comment 1 Martin Oberhuber

2011-10-20 07:43:38 EDT

+1

This is important for making it _easy_ to create small community contributions.

The learning curve I have today for forking a repo, creating a branch, committing locally... may make sense for longer projects, but doesn't make sense for quick one-line fixes.

Create Patch / Apply Patch is easy to understand and to use.
It should be supported in egit as well.

Comment 2 Remy Suen

2011-10-20 07:59:06 EDT

Probably a duplicate of bug 341036. The request certainly matches bug 341036 comment 5.

Also see bug 354800.

Comment 3 Dani Megert

2011-10-21 08:37:34 EDT

+1

Comment 4 Tomasz Zarna

2011-12-27 09:31:19 EST

(In reply to comment #2)
> Probably a duplicate of bug 341036.

Yup, looks like a dupe to me.

*** This bug has been marked as a duplicate of bug 341036 ***