[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add reptyr.
From: |
Alex Griffin |
Subject: |
Re: [PATCH] gnu: Add reptyr. |
Date: |
Tue, 12 Jul 2016 15:17:29 -0500 |
On Sat, Jul 9, 2016, at 01:20 PM, Leo Famulari wrote:
> If we are considering moving these packages around, I think that
> screen.scm and tmux.scm should go into terminals.scm. But, I also think
> that this patch should not be held up by that discussion.
Yes, I didn't necessarily mean just moving tmux into screen.scm. I just
meant that they clearly belong next to each other, wherever that is.
> Why skip the tests? We prefer to run tests whenever possible. Otherwise,
> please add a comment explaining why. For example "No test suite."
There's no tests to run. I added a comment on it.
> This is short and sweet, but we like to use modify-phases for
> consistency.
Okay, done. (btw a bunch of packages in suckless.scm do it that way,
which is where I copied the arguments field from)
> We should capitalize the beginning of the description, and I would also
> wrap 'ssh' and 'screen' in texinfo:
>
> @code{ssh}
> @code{screen}
I wrapped those commands in texinfo, but I don't think that reptyr
should be capitalized even at the beginning of a sentence. Upstream
always seems to keep it lower case, and other package descriptions do it
this way too (like dtach).
I included the updated patch as an attachment. Thanks for the feedback,
--
Alex Griffin
0001-gnu-Add-reptyr.patch
Description: Text Data