DDB patches
Willem Jan Withagen
wjw at digiware.nl
Thu Nov 19 13:45:09 UTC 2015
On 19-11-2015 10:57, Dan Partelly wrote:
> Hey Pedro,
>
> Thanks a lot , mate.
>
> I’m reluctant to put it up as a PR, since some PR are outstanding for
> years.
What a strange argument....
Some PR's are fixed within hours/days....
Letting it linger in your mailbox after mental evaluation is not going
to do anybody any good.
As far as I understand is the PR database also the collective memory of
things on/for/by/near/around the FreeBSD source code. People are
explicitly asked to fill a PR so the issue at hand is not forgotten.
So not submitting a PR for an issue sound real strange to my ears.
But then again that's me.
--WjW
> Adrian,
>
> since Pedro has issue with hardware, could you try the patch and give
> a resolution on it ? I reviewed it mentally (no FreeBSD atm machine
> on which I could actually patch the kernel) and apart style changes
> it looks OK . Physically i can test it again fro a couple of days.
> Getting this reviewed & tested / committed or rejected would give me
> an idea on how things actually work around here. This is actual code
> which you can commit or reject not commentaries only like in the
> thread regarding the binary code reuse.
>
>
> [qute from libxo thread ]
>>> It's all fine and good making technical decisions based on
>>> drawings and handwaving and philosophizing, but at some point
>>> someone has to do the code. The reason is simple - someone
>>> offered to do the work and push it through. This isn't a
>>> commercial thing where we get to make project >>decisions and
>>> allocate resources - the juniper folk came up with a solution
>>> that
>
> Once I see how things work around here once someone wrote the code,
> and get this done one way or another , we could proceed to the
> libification of ifconfig, should you so desire, and you believe we
> can all benefit from it.
>
>
> Dan
>
>
>
>> On 19 Nov 2015, at 11:17, Pedro Giffuni <pfg at freebsd.org> wrote:
>
>>
>> Hello;
>>
>>> Il giorno 19/nov/2015, alle ore 02:34, Dan Partelly
>>> <dan_partelly at rdsor.ro> ha scritto:
>>>
>>> Hey Pedro,
>>>
>>> some times ago you got some DDB patches from me in which I added
>>> relational ops support from it. The patch was a bit clobbered,
>>> but last I know you cleaned it up and put it somewhere on
>>> freebsd.org (prolly your page) up for review.
>>>
>>
>> It’s here: https://people.freebsd.org/~pfg/patches/ddb.patch
>>
>> I haven’t tested it though.
>>
>>> Could you or Adrian review the patch set , and if it is OK
>>> potentially proceed with a commit ? Or if it is not ok for a
>>> commit , please advice on a follow up.
>>>
>>
>> I am having hardware issues so I won’t be able to do much in a
>> while. Perhaps you should review it and submit it as a PR.
>>
>> Pedro.
>>
>
> _______________________________________________
> freebsd-current at freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/freebsd-current To
> unsubscribe, send any mail to
> "freebsd-current-unsubscribe at freebsd.org"
>
More information about the freebsd-current
mailing list