Re: [dev] [quark] [PATCH] Add the chrootdir configuration variable

From: Ivan Delalande <colona_AT_ycc.fr>
Date: Sat, 29 Nov 2014 19:39:53 +0100

On Sat, Nov 29, 2014 at 12:21:39PM +0100, Hiltjo Posthuma wrote:
> On Sat, Nov 29, 2014 at 9:47 AM, Ivan Delalande <colona_AT_ycc.fr> wrote:
>> On Sat, Nov 29, 2014 at 09:28:24AM +0100, FRIGN wrote:
>>> Hiltjo told me he was almost done with the changes, so as soon as he
>>> finishes this masterpiece, I'll merge it into mainline.
>
> It needs alot more polish and is not ready for the consumers yet :). I
> can't give
> a time when it will be done. I'm also working on other things. I will need to
> rewrite some parts because they are ugly.

How unpolished is it? If it’s working and doesn’t introduce critical
bugs or performance issues and if you are not in the middle of a big
change, I would say that it can be fine to merge.
There will always be a bunch of people around here more than happy to
help with polishing and cleaning the code. ;)

>>> If you like you can check out if the problem still exists in the heavily
>>> modified branch.
>
> Yeah it is reproducable in my version.
>
>> Oh, that’s interesting, he also has other features I also need and was
>> planning to write patches for. :)
>>
> Great, I'm curious which features do you need / use ?

The only one I absolutely need at the moment is the POST method for CGI,
but all your other improvements are nice and always good to take.

-- 
Ivan "Colona" Delalande
Received on Sat Nov 29 2014 - 19:39:53 CET

This archive was generated by hypermail 2.3.0 : Sat Nov 29 2014 - 19:48:08 CET