unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Haskell Platform patchspam
@ 2015-10-19  9:01 Paul van der Walt
  2015-10-19 13:24 ` Ludovic Courtès
  0 siblings, 1 reply; 3+ messages in thread
From: Paul van der Walt @ 2015-10-19  9:01 UTC (permalink / raw)
  To: guix-devel@gnu.org

Hello Guix,

I thought it'd be easier to review my patches if i pushed them to a
feature branch, since after mailing them i have made some minor changes,
including:

* running `guix lint ..` on all of the packages; no more issues
  reported.

* all patches now contain one and only one new variable

So, without further ado, i present the feature branch
`wip-haskell-platform-7.10.2-a` [0]!  As far as i'm concerned, the
patches are ready for prime-time, but i will of course wait for your
collective input before merging into master.

Sorry about the slightly unripe patches with which i flooded your
INBOXes last week!

Ciao,
p.

0. <http://git.savannah.gnu.org/cgit/guix.git/log/?h=wip-haskell-platform-7.10.2-a>

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

* Re: Haskell Platform patchspam
  2015-10-19  9:01 Haskell Platform patchspam Paul van der Walt
@ 2015-10-19 13:24 ` Ludovic Courtès
  2015-10-19 13:53   ` Paul van der Walt
  0 siblings, 1 reply; 3+ messages in thread
From: Ludovic Courtès @ 2015-10-19 13:24 UTC (permalink / raw)
  To: Paul van der Walt; +Cc: guix-devel@gnu.org

Howdy!

Paul van der Walt <paul@denknerd.org> skribis:

> I thought it'd be easier to review my patches if i pushed them to a
> feature branch, since after mailing them i have made some minor changes,
> including:
>
> * running `guix lint ..` on all of the packages; no more issues
>   reported.
>
> * all patches now contain one and only one new variable
>
> So, without further ado, i present the feature branch
> `wip-haskell-platform-7.10.2-a` [0]!  As far as i'm concerned, the
> patches are ready for prime-time, but i will of course wait for your
> collective input before merging into master.

Excellent!

I think we are obviously reaching the limits of our review process here.
So what I would suggest is to leave, say, 4 more days for people to
review the branch and send any suggestions.

On your side, if you haven’t already done that, please pass the packages
through ‘guix lint’ (which has gotten hopefully faster, thanks to your
recent complaint ;-)) and double-check the guidelines in the “Submitting
Patches” section of the manual.

After 4 days have passed, if there are no further comments, feel free to
rebase the branch on ‘master’ and to merge it there.

Thanks again for the herculean work!

Ludo’.

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

* Re: Haskell Platform patchspam
  2015-10-19 13:24 ` Ludovic Courtès
@ 2015-10-19 13:53   ` Paul van der Walt
  0 siblings, 0 replies; 3+ messages in thread
From: Paul van der Walt @ 2015-10-19 13:53 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel@gnu.org

Greetings Ludo,

On 2015-10-19 at 15:24, quoth Ludovic Courtès:
> I think we are obviously reaching the limits of our review process
> here.  So what I would suggest is to leave, say, 4 more days for
> people to review the branch and send any suggestions.

Sounds fine.  I'll see what happens.

> On your side, if you haven’t already done that, please pass the
> packages through ‘guix lint’

Confirmed done, zero warnings.

> (which has gotten hopefully faster, thanks to your recent complaint
> ;-))

It seemed that way indeed, thank you!

> and double-check the guidelines in the “Submitting Patches” section of
> the manual.  After 4 days have passed, if there are no further
> comments, feel free to rebase the branch on ‘master’ and to merge it
> there.

Alright.

> Thanks again for the herculean work!

No problem, it's a pleasure to be thanked for procrastination!

Ciao,
p.

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

end of thread, other threads:[~2015-10-19 13:53 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-19  9:01 Haskell Platform patchspam Paul van der Walt
2015-10-19 13:24 ` Ludovic Courtès
2015-10-19 13:53   ` Paul van der Walt

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).