From: Kei Kebreau <kei@openmailbox.org>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: guix-devel@gnu.org, rennes <rennes@openmailbox.org>
Subject: Re: [PATCH] gnu: Add gnome-calculator.
Date: Sat, 24 Sep 2016 10:54:49 -0400 [thread overview]
Message-ID: <87pontmjpi.fsf@openmailbox.org> (raw)
In-Reply-To: <87h996t275.fsf@gnu.org> ("Ludovic \=\?utf-8\?Q\?Court\=C3\=A8s\=22'\?\= \=\?utf-8\?Q\?s\?\= message of "Sat, 24 Sep 2016 12:18:54 +0900")
[-- Attachment #1: Type: text/plain, Size: 1564 bytes --]
ludo@gnu.org (Ludovic Courtès) writes:
> Hello,
>
> Kei Kebreau <kei@openmailbox.org> skribis:
>
>> rennes <rennes@openmailbox.org> writes:
>>
>>> On Thu, 2016-09-22 at 21:23 -0400, Kei Kebreau wrote:
>>>>
>>>> This is a good patch! The only change I recommend is moving the
>>>> defintion to gnome.scm.
>>>
>>> I update the patch,
>>> * Create the definition in gnome.scm file.
>>> * Move 'glib' to native-inputs.
>>> * Delete Copyright duplicate for
>>> ';;; Copyright © 2016 Rene Saavedra <rennes@openmailbox.org>'
>>>
>>> Thank you
>>
>> Looks good and works well.
>>
>> Does somebody want to commit this to master?
>> I'd do it myself, but I'm unfamiliar with the process of commiting
>> someone else's code and I don't want to break anything.
>
> It’s a good occasion to try—we need more reviewers! :-)
>
> The process is essentially to pipe the patch (which includes the commit
> log and Subject: line) through “git am -s”.
>
> If that applies fine, just run ‘guix lint the-package’ and ideally ‘guix
> build --rounds=2 the-package’ to catch any issues.
>
> If nothing comes up, just push!
>
> If minor issues (typos, synopsis, etc.) come up, you might commit them
> on behalf of the author, or ask them what they prefer.
>
> Could you try and report back? :-)
>
> Thanks Kei & rennes!
>
> Ludo’.
I've already pushed it! I guess my last message only went back to
Leo. Thank you and Leo for your help. :-)
rennes, everything looked good. Thanks for your contribution!
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 818 bytes --]
next prev parent reply other threads:[~2016-09-24 14:55 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-23 0:05 [PATCH] gnu: Add gnome-calculator rennes
2016-09-23 1:23 ` Kei Kebreau
2016-09-23 13:07 ` rennes
2016-09-23 16:21 ` Kei Kebreau
2016-09-24 3:18 ` Ludovic Courtès
2016-09-24 14:54 ` Kei Kebreau [this message]
2016-09-27 12:19 ` Ludovic Courtès
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87pontmjpi.fsf@openmailbox.org \
--to=kei@openmailbox.org \
--cc=guix-devel@gnu.org \
--cc=ludo@gnu.org \
--cc=rennes@openmailbox.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.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).