On 19-11-2015 14:53, Dan Partelly wrote: >> So not submitting a PR for an issue sound real strange to my ears. > > > It is NOT a patch for an issue, bug, anything on those lines at all. > It adds new features to DDB. Specifically it adds relational > operators. > > Since it doesn't solve any problems, I don't see why it should be added > to a problem database which is used for bug reports. > Unlike what you suggest here, I see lots of issues in Bugzilla which exactly what you describe: enhancements on already available tools. I've even submitted a few myself. An alternative for this might be submission to Phabricator if you'd like a more reviewing type of evaluation. --WjW > But then again, it may be just me. > > Dan > > > >> On 19 Nov 2015, at 15:44, Willem Jan Withagen <wjw_at_digiware.nl> wrote: >> >> 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" >>> >> >> _______________________________________________ >> 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" >Received on Thu Nov 19 2015 - 13:10:50 UTC
This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:41:01 UTC