* 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 external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.