Skip to main content


Eclipse Community Forums
Forum Search:

Search      Help    Register    Login    Home
Home » Eclipse Projects » EGit / JGit » Create patch with EGit
Create patch with EGit [message #500552] Thu, 26 November 2009 16:22 Go to next message
Lars Vogel is currently offline Lars VogelFriend
Messages: 1098
Registered: July 2009
Senior Member
Hi,

I used EGit to checkout the Eclipse Source code. I did for testing a
tiny change you wanted to create a patch via EGit which I could post in
Bugzilla.

Does EGit have the possibility to create patches? I don't see it in the
menu.

Best regards, Lars


--
http://www.vogella.de/ - Eclipse plugin and Eclipse RCP Tutorials
http://www.twitter.com/vogella - vogella on Twitter

Report message to a moderator

Re: Create patch with EGit [message #500559 is a reply to message #500552] Thu, 26 November 2009 17:02 Go to previous messageGo to next message
Christian Halstrick is currently offline Christian HalstrickFriend
Messages: 274
Registered: July 2009
Senior Member
Hi,

currently EGit doesn't have the functionality to create patches. That is because it is still lacking a good diff algorithm. Introducing that diff and the associated work is what I am currently working on.
You would need to use a native git implementation to create such a patch.

Ciao
Chris

Report message to a moderator

Re: Create patch with EGit [message #500877 is a reply to message #500559] Mon, 30 November 2009 01:46 Go to previous messageGo to next message
Lars Vogel is currently offline Lars VogelFriend
Messages: 1098
Registered: July 2009
Senior Member
Hi Christian,

sounds good. You have already a bug report for this so that I can follow
your work?

Best regards, Lars

--
http://www.vogella.de/ - Eclipse plugin and Eclipse RCP Tutorials
http://www.twitter.com/vogella - vogella on Twitter


Chris wrote:
> Hi,
>
> currently EGit doesn't have the functionality to create patches. That is
> because it is still lacking a good diff algorithm. Introducing that diff
> and the associated work is what I am currently working on.
> You would need to use a native git implementation to create such a patch.
>

Report message to a moderator

Re: Create patch with EGit [message #500897 is a reply to message #500877] Mon, 30 November 2009 08:25 Go to previous messageGo to next message
Christian Halstrick is currently offline Christian HalstrickFriend
Messages: 274
Registered: July 2009
Senior Member
Hi,

you can watch it in http://egit.eclipse.org where we propose and discuss code changes. This work is done in the cq-diff branch. E.g http://egit.eclipse.org/r/#change,87 or http://egit.eclipse.org/r/#change,37.

Ciao
Chris

Ciao
Chris

Report message to a moderator

Re: Create patch with EGit [message #501046 is a reply to message #500897] Mon, 30 November 2009 19:32 Go to previous messageGo to next message
Lars Vogel is currently offline Lars VogelFriend
Messages: 1098
Registered: July 2009
Senior Member
Hi Chris,

thank you. Best regards, Lars

Report message to a moderator

Re: Create patch with EGit [message #503336 is a reply to message #500897] Fri, 11 December 2009 23:11 Go to previous message
Lars Vogel is currently offline Lars VogelFriend
Messages: 1098
Registered: July 2009
Senior Member
Hi Chris,

I also opened a Bug report for tis:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=297636

Cheers, Lars



On 30.11.2009 00:25, Chris wrote:
> Hi,
>
> you can watch it in http://egit.eclipse.org where we propose and discuss
> code changes. This work is done in the cq-diff branch. E.g
> http://egit.eclipse.org/r/#change,87 or
> http://egit.eclipse.org/r/#change,37
>
> Ciao
> Chris

Report message to a moderator

Re: Create patch with EGit [message #576046 is a reply to message #500552] Thu, 26 November 2009 17:02 Go to previous message
Christian Halstrick is currently offline Christian HalstrickFriend
Messages: 274
Registered: July 2009
Senior Member
Hi,

currently EGit doesn't have the functionality to create patches. That is because it is still lacking a good diff algorithm. Introducing that diff and the associated work is what I am currently working on.
You would need to use a native git implementation to create such a patch.

Ciao
Chris

Report message to a moderator

Re: Create patch with EGit [message #576203 is a reply to message #576046] Mon, 30 November 2009 01:46 Go to previous message
Lars Vogel is currently offline Lars VogelFriend
Messages: 1098
Registered: July 2009
Senior Member
Hi Christian,

sounds good. You have already a bug report for this so that I can follow
your work?

Best regards, Lars

--
http://www.vogella.de/ - Eclipse plugin and Eclipse RCP Tutorials
http://www.twitter.com/vogella - vogella on Twitter


Chris wrote:
> Hi,
>
> currently EGit doesn't have the functionality to create patches. That is
> because it is still lacking a good diff algorithm. Introducing that diff
> and the associated work is what I am currently working on.
> You would need to use a native git implementation to create such a patch.
>

Report message to a moderator

Re: Create patch with EGit [message #576217 is a reply to message #500877] Mon, 30 November 2009 08:25 Go to previous message
Christian Halstrick is currently offline Christian HalstrickFriend
Messages: 274
Registered: July 2009
Senior Member
Hi,

you can watch it in http://egit.eclipse.org where we propose and discuss code changes. This work is done in the cq-diff branch. E.g http://egit.eclipse.org/r/#change,87 or http://egit.eclipse.org/r/#change,37

Ciao
Chris
--
Ciao
Chris

Ciao
Chris

Report message to a moderator

Re: Create patch with EGit [message #576290 is a reply to message #576217] Mon, 30 November 2009 19:32 Go to previous message
Lars Vogel is currently offline Lars VogelFriend
Messages: 1098
Registered: July 2009
Senior Member
Hi Chris,

thank you. Best regards, Lars

Report message to a moderator

Re: Create patch with EGit [message #576634 is a reply to message #576217] Fri, 11 December 2009 23:11 Go to previous message
Lars Vogel is currently offline Lars VogelFriend
Messages: 1098
Registered: July 2009
Senior Member
Hi Chris,

I also opened a Bug report for tis:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=297636

Cheers, Lars



On 30.11.2009 00:25, Chris wrote:
> Hi,
>
> you can watch it in http://egit.eclipse.org where we propose and discuss
> code changes. This work is done in the cq-diff branch. E.g
> http://egit.eclipse.org/r/#change,87 or
> http://egit.eclipse.org/r/#change,37
>
> Ciao
> Chris

Report message to a moderator

Previous Topic:UI dependencies in egit core
Next Topic:JGit not working correctly
Goto Forum:
  

[ Syndicate this forum (XML) ] [ RSS ]

Current Time: Wed Apr 24 22:52:55 GMT 2024

Powered by FUDForum. Page generated in 0.03172 seconds
.:: Contact :: Home ::.

Powered by: FUDforum 3.0.2.
Copyright ©2001-2010 FUDforum Bulletin Board Software

Back to the top

Sign up to our Newsletter

A fresh new issue delivered monthly