GNU bug report logs - #28020
[PATCH] gnu: Add emacs-gitpatch.

Previous Next

Package: guix-patches;

Reported by: Oleg Pykhalov <go.wigust <at> gmail.com>

Date: Tue, 8 Aug 2017 18:03:02 UTC

Severity: normal

Tags: patch

Done: Arun Isaac <arunisaac <at> systemreboot.net>

Bug is archived. No further changes may be made.

Full log


Message #25 received at 28020-done <at> debbugs.gnu.org (full text, mbox):

From: ng0 <ng0 <at> infotropique.org>
To: Arun Isaac <arunisaac <at> systemreboot.net>
Cc: go.wigust <at> gmail.com, 28020-done <at> debbugs.gnu.org, ng0 <ng0 <at> infotropique.org>
Subject: Re: bug#28020: [PATCH] gnu: Add emacs-gitpatch.
Date: Thu, 10 Aug 2017 16:55:17 +0000
[Message part 1 (text/plain, inline)]
Arun Isaac transcribed 1.3K bytes:
> 
> ng0 writes:
> 
> >     gnu: Add emacs-gitpatch.
> >
> >     By the way, “gitpatch.el” says different version than release tarball.
> >     Do I need to ask a developer?
> >     --8<---------------cut here---------------start------------->8---
> >     ;; Author: Feng Shu <tumashu <at> 163.com>
> >     ;; Homepage: https://github.com/tumashu/gitpatch
> >     ;; Keywords: convenience
> >     ;; Package-Requires: ((emacs "24.3"))
> >     ;; Version: 0.10
> >     --8<---------------cut here---------------end--------------->8---
> >
> >     >From 9eddf07fa401468738c2f48a25e51ddef2a3ceed Mon Sep 17 00:00:00 2001
> >     From: Oleg Pykhalov <go.wigust <at> gmail.com>
> >     Date: Sat, 29 Jul 2017 14:22:48 +0300
> >     Subject: [PATCH] gnu: Add emacs-gitpatch.
> >
> >     * gnu/packages/emacs.scm (emacs-gitpatch): New variable.
> >
> >     Signed-off-by: Arun Isaac <arunisaac <at> systemreboot.net>
> >
> > I appreciate the concerns, but
> > I don't think the complete paragraph of "By the way…" should be in the
> > commit message. Please place notes like this as "XXX: ..." or "TODO: ..."
> > in the code if possible.
> > Or is there any reason why this landed in the commit message?
> 
> Oops! I didn't notice it was there. Looks like I screwed up big time
> while applying the patch. There's no way to change the commit message
> now, is there?
> 

No, don't worry there are some other misshapenings in the commit log.
Mistakes can happen. Though it would be nice to have a checker for this
somehow...
-- 
ng0
GnuPG: A88C8ADD129828D7EAC02E52E22F9BBFEE348588
GnuPG: https://n0is.noblogs.org/my-keys
https://www.infotropique.org https://krosos.org
[signature.asc (application/pgp-signature, inline)]

This bug report was last modified 7 years and 283 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.