unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH] Missing final semi-colon in .desktop's Categories.
@ 2009-11-21  9:13 James Rowe
  2009-11-22  7:07 ` Jeffrey Ollie
  0 siblings, 1 reply; 3+ messages in thread
From: James Rowe @ 2009-11-21  9:13 UTC (permalink / raw)
  To: notmuch

"Those keys which have several values should have a semicolon as the trailing
character."
  -- http://standards.freedesktop.org/desktop-entry-spec/1.0/ar01s03.html

Signed-off-by: James Rowe <jnrowe@gmail.com>
---
 notmuch.desktop |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/notmuch.desktop b/notmuch.desktop
index d29dff6..819cd1e 100644
--- a/notmuch.desktop
+++ b/notmuch.desktop
@@ -4,4 +4,4 @@ Exec=emacs -f notmuch
 Icon=emblem-mail
 Terminal=false
 Type=Application
-Categories=Network;Email
+Categories=Network;Email;
-- 
1.6.4.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] Missing final semi-colon in .desktop's Categories.
  2009-11-21  9:13 [PATCH] Missing final semi-colon in .desktop's Categories James Rowe
@ 2009-11-22  7:07 ` Jeffrey Ollie
  2009-11-22 12:29   ` Carl Worth
  0 siblings, 1 reply; 3+ messages in thread
From: Jeffrey Ollie @ 2009-11-22  7:07 UTC (permalink / raw)
  To: Not Much Mail

Found the same problem on my own, only to minutes later catch up on
the backlog of patches in my email and find this!

On Sat, Nov 21, 2009 at 3:13 AM, James Rowe <jnrowe@gmail.com> wrote:
> "Those keys which have several values should have a semicolon as the trailing
> character."
>  -- http://standards.freedesktop.org/desktop-entry-spec/1.0/ar01s03.html
>
> Signed-off-by: James Rowe <jnrowe@gmail.com>

Reviewed-by: Jeffrey C. Ollie <jeff@ocjtech.us>

> ---
>  notmuch.desktop |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/notmuch.desktop b/notmuch.desktop
> index d29dff6..819cd1e 100644
> --- a/notmuch.desktop
> +++ b/notmuch.desktop
> @@ -4,4 +4,4 @@ Exec=emacs -f notmuch
>  Icon=emblem-mail
>  Terminal=false
>  Type=Application
> -Categories=Network;Email
> +Categories=Network;Email;
> --
> 1.6.4.4
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
>



-- 
Jeff Ollie

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Missing final semi-colon in .desktop's Categories.
  2009-11-22  7:07 ` Jeffrey Ollie
@ 2009-11-22 12:29   ` Carl Worth
  0 siblings, 0 replies; 3+ messages in thread
From: Carl Worth @ 2009-11-22 12:29 UTC (permalink / raw)
  To: Jeffrey Ollie, Not Much Mail

On Sun, 22 Nov 2009 01:07:49 -0600, Jeffrey Ollie <jeff@ocjtech.us> wrote:
> Found the same problem on my own, only to minutes later catch up on
> the backlog of patches in my email and find this!
...
> > Signed-off-by: James Rowe <jnrowe@gmail.com>
> 
> Reviewed-by: Jeffrey C. Ollie <jeff@ocjtech.us>

Much thanks, James and Jeffrey!

I always feel *much* more comfortable pushing a patch I haven't tested
personally if someone else has reviewed it.

-Carl

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-11-22 12:29 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-11-21  9:13 [PATCH] Missing final semi-colon in .desktop's Categories James Rowe
2009-11-22  7:07 ` Jeffrey Ollie
2009-11-22 12:29   ` Carl Worth

Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).