unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Matthew Jordan <matthewjordandevops@yandex.com>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: Guix-devel <guix-devel@gnu.org>
Subject: Re: [PATCH] entr-3.5
Date: Wed, 11 May 2016 13:19:04 -0400	[thread overview]
Message-ID: <87y47gy0l3.fsf@mailerver.i-did-not-set--mail-host-address--so-tickle-me> (raw)
In-Reply-To: <87wpn0sh26.fsf@gnu.org>


Thank you for the welcome Ludovic Courtès. I just have a couple of
questions.

>   3. Added the file to gnu/local.mk.

Is this file to be updated when a new source file is added?

>   4. Added a ChangeLog-style commit log.

Is there a specific reference/manual on the "ChangeLog-style" I could review?

>   2. Added a ‘remove-fhs-file-names’ phase to remove hard-coded
>      references to /bin/cat and /usr/bin/clear.

And is ‘remove-fhs-file-names’ a custom phase? If not, which source file
contains the available phases?

Thank you in advance, and sorry for all the noob questions.

Ludovic Courtès writes:

> Hello, and welcome!
>
> Matthew Jordan <matthewjordan85@gmail.com> skribis:
>
>> I'm relatively new to guix, spent the first couple
>> days getting used to running it in my VM.  Loving it so far.  Since I'm
>> not sure what to expect I decided to package a simple program that I
>> find quite handy. Let me know if I should change anything.
>>
>> Also note that I followed "8.5 Submitting Patches" of the "GNU Guix
>> Reference Manual"
>
> Great!  There’s a couple of things that this section doesn’t clearly
> mention, such as copyright notices—there’s room for improvement.  ;-)
>
>> From 5b33ed530e472008bddf85a2f23b3fa9857b365e Mon Sep 17 00:00:00 2001
>> From: Matthew Jordan <matthewjordandevops@yandex.com>
>> Date: Wed, 11 May 2016 09:25:13 -0400
>> Subject: [PATCH] Added package entr.
>
> Pushed as 5d38aa0 with these minor changes:
>
>   1. Added a copyright notice for you (let me know if you think it’s
>      incorrect!).
>
>   2. Added a ‘remove-fhs-file-names’ phase to remove hard-coded
>      references to /bin/cat and /usr/bin/clear.
>
>   3. Added the file to gnu/local.mk.
>
>   4. Added a ChangeLog-style commit log.
>
> Thank you!
>
> Ludo’.

Sincerely,

-- 
Matthew Jordan
Sent with my mu4e

  reply	other threads:[~2016-05-11 17:19 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-11 14:03 [PATCH] entr-3.5 Matthew Jordan
2016-05-11 16:19 ` Ludovic Courtès
2016-05-11 17:19   ` Matthew Jordan [this message]
2016-05-15 19:50     ` 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=87y47gy0l3.fsf@mailerver.i-did-not-set--mail-host-address--so-tickle-me \
    --to=matthewjordandevops@yandex.com \
    --cc=guix-devel@gnu.org \
    --cc=ludo@gnu.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).