Re: [Dnsmasq-discuss] [patch] empty empty lines

2019-10-27 Thread Geert Stappers
On Sun, Sep 08, 2019 at 02:25:18PM +0200, Geert Stappers wrote: > On Sun, Sep 08, 2019 at 02:17:23PM +0200, Geert Stappers wrote: > > Made empty lines that had only spaces or tabs truely empty > > with `sed --in-place -e 's/^[ \t]*$//' src/*.c` > > | $ sed --in-place -e 's/^[ \t]*$//' src/*.c > |

Re: [Dnsmasq-discuss] [patch] empty empty lines

2019-09-30 Thread john doe
On 9/30/2019 4:50 PM, Simon Kelley wrote: > To be clear, I have no objection to this sort of patch/suggestion. > > It should be fairly clear, however, that my available time to work on > dnsmasq at the moment is limited, and stuff like this is not a priority, > and likely to be pushed to the back

Re: [Dnsmasq-discuss] [patch] empty empty lines

2019-09-30 Thread Kurt H Maier
On Mon, Sep 30, 2019 at 12:01:52PM -0400, Paul Smith wrote: > > What you can do is have a hook which will check the content of the > incoming pushed commits and reject the push if any of them have > incorrect formats. Then it will be up to the person pushing to fix > their local copy and re-run

Re: [Dnsmasq-discuss] [patch] empty empty lines

2019-09-30 Thread Paul Smith
On Mon, 2019-09-30 at 15:50 +0100, Simon Kelley wrote: > If we're going to do this, the first stage is probably to add hooks > to git to run expand (for tabs) and this filter on all NEW commits. > Then we won't ever have to do that again. I don't think you can really do this sort of thing with

Re: [Dnsmasq-discuss] [patch] empty empty lines

2019-09-30 Thread Simon Kelley
To be clear, I have no objection to this sort of patch/suggestion. It should be fairly clear, however, that my available time to work on dnsmasq at the moment is limited, and stuff like this is not a priority, and likely to be pushed to the back of the queue, possibly so far that it never emerges

Re: [Dnsmasq-discuss] [patch] empty empty lines

2019-09-30 Thread john doe
Botom-posting. On 9/30/2019 2:11 PM, Petr Mensik wrote: > Hi John, > > dnsmasq is quite interesting piece of software. It is used by lot > different kind of projects. I think noone objected dnsmasq isn't useful > or Simon does not work on it. > > But as with everything, there is always some space

Re: [Dnsmasq-discuss] [patch] empty empty lines

2019-09-30 Thread Petr Mensik
Hi John, dnsmasq is quite interesting piece of software. It is used by lot different kind of projects. I think noone objected dnsmasq isn't useful or Simon does not work on it. But as with everything, there is always some space for improvements. There is no reason why Simon should stop working

Re: [Dnsmasq-discuss] [patch] empty empty lines

2019-09-15 Thread john doe
On 9/15/2019 5:45 PM, Geert Stappers wrote: > On Sun, Sep 08, 2019 at 02:25:18PM +0200, Geert Stappers wrote: >> On Sun, Sep 08, 2019 at 02:17:23PM +0200, Geert Stappers wrote: >>> Made empty lines that had only spaces or tabs truely empty >>> with `sed --in-place -e 's/^[ \t]*$//' src/*.c` >> >>

Re: [Dnsmasq-discuss] [patch] empty empty lines

2019-09-15 Thread Geert Stappers
On Sun, Sep 08, 2019 at 02:25:18PM +0200, Geert Stappers wrote: > On Sun, Sep 08, 2019 at 02:17:23PM +0200, Geert Stappers wrote: > > Made empty lines that had only spaces or tabs truely empty > > with `sed --in-place -e 's/^[ \t]*$//' src/*.c` > > | $ sed --in-place -e 's/^[ \t]*$//' src/*.c > |

Re: [Dnsmasq-discuss] [patch] empty empty lines

2019-09-08 Thread Geert Stappers
On Sun, Sep 08, 2019 at 02:17:23PM +0200, Geert Stappers wrote: > Made empty lines that had only spaces or tabs truely empty > with `sed --in-place -e 's/^[ \t]*$//' src/*.c` | $ sed --in-place -e 's/^[ \t]*$//' src/*.c | $ git diff | wc | 21648 76730 597015 597015 is to big for the current

[Dnsmasq-discuss] [patch] empty empty lines

2019-09-08 Thread Geert Stappers
Made empty lines that had only spaces or tabs truely empty with `sed --in-place -e 's/^[ \t]*$//' src/*.c` ___ Dnsmasq-discuss mailing list Dnsmasq-discuss@lists.thekelleys.org.uk http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss