git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [EGIT] [PATCH] Make sure that the commit message has Unix format line

From
Ferry Huberts (Pelagic) <ferry.huberts@pelagic.nl>
Date
Feb 11, 2009, 18:14 UTC
Message-ID
<49931568.6000709@pelagic.nl>
In-Reply-To
<20090211161346.GD30949@spearce.org>
Shawn O. Pearce wrote:
Show 57 quoted lines
> "Ferry Huberts (Pelagic)" <ferry.huberts@pelagic.nl> wrote:
>   
>> From 2ad66e099fb35ad9a0d97e553843434b68ec82c9 Mon Sep 17 00:00:00 2001 From: Ferry Huberts <ferry.huberts@pelagic.nl>
>> Date: Wed, 11 Feb 2009 08:29:36 +0100
>> Subject: [PATCH] Make sure that the commit message has Unix format line delimiters
>>
>> Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
>>     
>
> Doesn't this change that I committed yesterday also fix this?
>
>   commit 72cb3bd763f17a7d9894a8443e05965ff5e77157
>   Author: Shawn O. Pearce <spearce@spearce.org>
>   Date:   Tue Feb 10 09:13:06 2009 -0800
>
>     Append Signed-off-by text in the commit message editor
>
>     The user can see and edit the Signed-off-by line in the commit
>     dialog before committing.  Toggling the existing checkbox now
>     updates the message to add or remove the user's Signed-off-by
>     line, displaying the final message.
>
>     Updating the committer name and/or email also updates the
>     Signed-off-by line if it is present.
>
>     For new lines in the commit dialog, use Text.DELIMITER for
>     platform neutrality.
>
>     Signed-off-by: Yann Simon <yann.simon.fr@gmail.com>
>     Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
>
> Speaking of which, #@!*#@@!*@!!@@!#*, I amended the commit with
> something yesterday and it whacked Yann Simon off as the author
> of that patch and switch it to me.
>
> Crap.
>
>   
>>  .../egit/ui/internal/actions/CommitAction.java     |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/actions/CommitAction.java
>> b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/actions/CommitAction.java index 5996596..a14011a 100644 ---
>> a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/actions/CommitAction.java +++
>> b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/actions/CommitAction.java @@ -128,7 +128,7 @@ public void
>> run(IAction act) {
>>  		if (commitDialog.open() != IDialogConstants.OK_ID)
>>  			return;
>>
>> -		String commitMessage = commitDialog.getCommitMessage();
>> +		String commitMessage = commitDialog.getCommitMessage().replaceAll("\n\r|\r\n", "\n");
>>  		amending = commitDialog.isAmending();
>>  		try {
>>  			performCommit(commitDialog, commitMessage);
>>     
>
>   
I'm sorry about that. Appearently I was working with an out of date
repository and plugin.
  I also agree that fixing it in the getter is a better idea.
This was my first patch for egit, looking forward to more. I'll try to
double check better though :-)
Previous: Robin Rosenberg
Message 4 of 4 in “Make sure that the commit message has Unix format line”
  1. Ferry Huberts (Pelagic)Feb 11, 2009
  2. Shawn O. PearceFeb 11, 2009
  3. Robin RosenbergFeb 11, 2009
  4. Ferry Huberts (Pelagic)Feb 11, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.