Re: [Dnsmasq-discuss] dnsmasq won't run on Debian 10.1 buster

2019-10-13 Thread john doe 
Please post through the list.

On 10/13/2019 10:17 PM, Guillaume B. wrote:
> Hi John,
>
> Sorry for the late reply I got caught up with multiple reinstals of Debian
> images.
>
> I need dnsmasq for its min-cache-ttl option and other options like
> clear-on-reload, stop-dns-rebind and dns-loop-detect, that can't be
> configured with solely dnsmasq-base.
>
> I tried installing resolvconf and kept it virgin but it didn't change the
> fact that with dnsmasq installed I still couldn't get a DHCP lease from my
> router. Maybe you have some configuration options for resolvconf to suggest
> me for it to work.
>
> Same as number one, I want to use dnsmasq for its extra options.
>

I don't see how I can help you further other then:
- Have you looked in the log when it is not working
- The internet (the one connected to your upstream dhcp server)
interface is 'excluded' ('except-interface=')

If my understanding is correct you only need the DNS capability of
dnsmasq, consider disabling the internal DHCP server.


If I may, I would sort things out without NM then I would bring it back
into the mix! :)

P.S.

Posting on 'debian-user' maling list might be useful because it does
look to me like it is a Dnsmasq issue.

--
John Doe

___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss

[Dnsmasq-discuss] [PATCH 3/3] Tuned .clang-format for us

2019-10-13 Thread Geert Stappers 
From: Geert Stappers 

the C coding style in dnsmasq is the GNU style,
except without the space preceding (
---
 src/.clang-format | 14 +++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/src/.clang-format b/src/.clang-format
index 08f7020..9acbeb6 100644
--- a/src/.clang-format
+++ b/src/.clang-format
@@ -1,4 +1,5 @@
 # Copyright (C) 2015 Free Software Foundation, Inc.
+# Copyright (C) 2019 Dnsmasq mailinglist
 #
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -13,6 +14,9 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see .
 
+# the C coding style in dnsmasq is the GNU style,
+# except without the space preceding (
+
 # clang-format 7.0.1 is required
 #
 # To utilize the tool to lines just touched by a patch, use
@@ -23,11 +27,14 @@
 # (here the tool will generate a patch)
 # git diff -U0 --no-color | clang-format-diff -p1 -i
 # (modifications are applied)
+#
+# See also
+# https://releases.llvm.org/8.0.0/tools/clang/docs/ClangFormatStyleOptions.html
 
 ---
 Language: Cpp
 AccessModifierOffset: -2
-AlwaysBreakAfterReturnType: TopLevel
+AlwaysBreakAfterReturnType: None
 BinPackArguments: true
 BinPackParameters: true
 BraceWrapping:
@@ -53,10 +60,11 @@ ForEachMacros: [ ]
 IndentCaseLabels: false
 NamespaceIndentation: None
 PenaltyBreakBeforeFirstCallParameter: 100
+PenaltyReturnTypeOnItsOwnLine: 100
 PointerAlignment: Right
 SortIncludes: false
-SpaceAfterCStyleCast: true
-SpaceBeforeParens: Always
+SpaceAfterCStyleCast: false
+SpaceBeforeParens: ControlStatements
 SpacesBeforeTrailingComments: 1
 UseTab: Always
 AlignEscapedNewlines: Right
-- 
2.1.4


___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss

[Dnsmasq-discuss] [PATCH 1/3] New file src/.clang-format, for coding style

2019-10-13 Thread Geert Stappers 
From: Geert Stappers 

Fetched with
  wget 
https://raw.githubusercontent.com/gcc-mirror/gcc/master/contrib/clang-format
---
 src/.clang-format | 150 ++
 1 file changed, 150 insertions(+)
 create mode 100644 src/.clang-format

diff --git a/src/.clang-format b/src/.clang-format
new file mode 100644
index 000..7a4e96f
--- /dev/null
+++ b/src/.clang-format
@@ -0,0 +1,150 @@
+# Copyright (C) 2015 Free Software Foundation, Inc.
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see .
+
+# clang-format 7.0.1 is required
+#
+# To utilize the tool to lines just touched by a patch, use
+# clang-format-diff script that is usually also packaged with clang-format.
+#
+# Example of usage:
+# git diff -U0 --no-color | clang-format-diff -p1
+# (here the tool will generate a patch)
+# git diff -U0 --no-color | clang-format-diff -p1 -i
+# (modifications are applied)
+
+---
+Language: Cpp
+AccessModifierOffset: -2
+AlwaysBreakAfterReturnType: TopLevel
+BinPackArguments: true
+BinPackParameters: true
+BraceWrapping:
+  AfterClass: true
+  AfterControlStatement: true
+  AfterEnum: true
+  AfterFunction: true
+  AfterNamespace: false
+  AfterObjCDeclaration: true
+  AfterStruct: true
+  AfterUnion: true
+  BeforeCatch: true
+  BeforeElse: true
+  IndentBraces: true
+  SplitEmptyFunction: false
+BreakBeforeBinaryOperators: All
+BreakBeforeBraces: Custom
+BreakBeforeTernaryOperators: true
+ColumnLimit: 80
+ConstructorInitializerIndentWidth: 2
+ContinuationIndentWidth: 2
+ForEachMacros: [
+'FOR_ALL_BB_FN',
+'FOR_ALL_EH_REGION',
+'FOR_ALL_EH_REGION_AT',
+'FOR_ALL_EH_REGION_FN',
+'FOR_ALL_INHERITED_FIELDS',
+'FOR_ALL_PREDICATES',
+'FOR_BB_BETWEEN',
+'FOR_BB_INSNS',
+'FOR_BB_INSNS_REVERSE',
+'FOR_BB_INSNS_REVERSE_SAFE',
+'FOR_BB_INSNS_SAFE',
+'FOR_BODY',
+'FOR_COND',
+'FOR_EACH_AGGR_INIT_EXPR_ARG',
+'FOR_EACH_ALIAS',
+'FOR_EACH_ALLOCNO',
+'FOR_EACH_ALLOCNO_OBJECT',
+'FOR_EACH_ARTIFICIAL_DEF',
+'FOR_EACH_ARTIFICIAL_USE',
+'FOR_EACH_BB_FN',
+'FOR_EACH_BB_REVERSE_FN',
+'FOR_EACH_BIT_IN_MINMAX_SET',
+'FOR_EACH_CALL_EXPR_ARG',
+'FOR_EACH_CLONE',
+'FOR_EACH_CONST_CALL_EXPR_ARG',
+'FOR_EACH_CONSTRUCTOR_ELT',
+'FOR_EACH_CONSTRUCTOR_VALUE',
+'FOR_EACH_COPY',
+'FOR_EACH_DEF',
+'FOR_EACH_DEFINED_FUNCTION',
+'FOR_EACH_DEFINED_SYMBOL',
+'FOR_EACH_DEFINED_VARIABLE',
+'FOR_EACH_DEP',
+'FOR_EACH_EDGE',
+'FOR_EACH_EXPR',
+'FOR_EACH_EXPR_1',
+'FOR_EACH_FUNCTION',
+'FOREACH_FUNCTION_ARGS',
+'FOREACH_FUNCTION_ARGS_PTR',
+'FOR_EACH_FUNCTION_WITH_GIMPLE_BODY',
+'FOR_EACH_HASH_TABLE_ELEMENT',
+'FOR_EACH_IMM_USE_FAST',
+'FOR_EACH_IMM_USE_ON_STMT',
+'FOR_EACH_IMM_USE_STMT',
+'FOR_EACH_INSN',
+'FOR_EACH_INSN_1',
+'FOR_EACH_INSN_DEF',
+'FOR_EACH_INSN_EQ_USE',
+'FOR_EACH_INSN_INFO_DEF',
+'FOR_EACH_INSN_INFO_EQ_USE',
+'FOR_EACH_INSN_INFO_MW',
+'FOR_EACH_INSN_INFO_USE',
+'FOR_EACH_INSN_USE',
+'FOR_EACH_LOCAL_DECL',
+'FOR_EACH_LOOP',
+'FOR_EACH_LOOP_FN',
+'FOR_EACH_OBJECT',
+'FOR_EACH_OBJECT_CONFLICT',
+'FOR_EACH_PHI_ARG',
+'FOR_EACH_PHI_OR_STMT_DEF',
+'FOR_EACH_PHI_OR_STMT_USE',
+'FOR_EACH_PREF',
+'FOR_EACH_SCALAR',
+'FOR_EACH_SSA_DEF_OPERAND',
+'FOR_EACH_SSA_TREE_OPERAND',
+'FOR_EACH_SSA_USE_OPERAND',
+'FOR_EACH_STATIC_INITIALIZER',
+'FOR_EACH_SUBRTX',
+'FOR_EACH_SUBRTX_PTR',
+'FOR_EACH_SUBRTX_VAR',
+'FOR_EACH_SUCC',
+'FOR_EACH_SUCC_1',
+'FOR_EACH_SYMBOL',
+'FOR_EACH_VARIABLE',
+'FOR_EACH_VEC_ELT',
+'FOR_EACH_VEC_ELT_FROM',
+'FOR_EACH_VEC_ELT_REVERSE',
+'FOR_EACH_VEC_SAFE_ELT',
+'FOR_EACH_VEC_SAFE_ELT_REVERSE',
+'FOR_EXPR',
+'FOR_INIT_STMT',
+'FOR_SCOPE'
+]
+IndentCaseLabels: false
+NamespaceIndentation: None
+PenaltyBreakBeforeFirstCallParameter: 100
+PointerAlignment: Right
+SortIncludes: false
+SpaceAfterCStyleCast: true
+SpaceBeforeParens: Always
+SpacesBeforeTrailingComments: 1
+UseTab: Always
+AlignEscapedNewlines: Right
+AlignTrailingComments: true
+AllowShortFunctionsOnASingleLine: All
+AlwaysBreakTemplateDeclarations: MultiLine
+KeepEmptyLinesAtTheStartOfBlocks: false
+Standard: Cpp03
-- 
2.1.4


___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss

[Dnsmasq-discuss] [PATCH 2/3] No ForEachMacros in .clang-format

2019-10-13 Thread Geert Stappers 
From: Geert Stappers 

There are no "For Each Macros" in Dnsmasq.
---
 src/.clang-format | 85 +--
 1 file changed, 1 insertion(+), 84 deletions(-)

diff --git a/src/.clang-format b/src/.clang-format
index 7a4e96f..08f7020 100644
--- a/src/.clang-format
+++ b/src/.clang-format
@@ -49,90 +49,7 @@ BreakBeforeTernaryOperators: true
 ColumnLimit: 80
 ConstructorInitializerIndentWidth: 2
 ContinuationIndentWidth: 2
-ForEachMacros: [
-'FOR_ALL_BB_FN',
-'FOR_ALL_EH_REGION',
-'FOR_ALL_EH_REGION_AT',
-'FOR_ALL_EH_REGION_FN',
-'FOR_ALL_INHERITED_FIELDS',
-'FOR_ALL_PREDICATES',
-'FOR_BB_BETWEEN',
-'FOR_BB_INSNS',
-'FOR_BB_INSNS_REVERSE',
-'FOR_BB_INSNS_REVERSE_SAFE',
-'FOR_BB_INSNS_SAFE',
-'FOR_BODY',
-'FOR_COND',
-'FOR_EACH_AGGR_INIT_EXPR_ARG',
-'FOR_EACH_ALIAS',
-'FOR_EACH_ALLOCNO',
-'FOR_EACH_ALLOCNO_OBJECT',
-'FOR_EACH_ARTIFICIAL_DEF',
-'FOR_EACH_ARTIFICIAL_USE',
-'FOR_EACH_BB_FN',
-'FOR_EACH_BB_REVERSE_FN',
-'FOR_EACH_BIT_IN_MINMAX_SET',
-'FOR_EACH_CALL_EXPR_ARG',
-'FOR_EACH_CLONE',
-'FOR_EACH_CONST_CALL_EXPR_ARG',
-'FOR_EACH_CONSTRUCTOR_ELT',
-'FOR_EACH_CONSTRUCTOR_VALUE',
-'FOR_EACH_COPY',
-'FOR_EACH_DEF',
-'FOR_EACH_DEFINED_FUNCTION',
-'FOR_EACH_DEFINED_SYMBOL',
-'FOR_EACH_DEFINED_VARIABLE',
-'FOR_EACH_DEP',
-'FOR_EACH_EDGE',
-'FOR_EACH_EXPR',
-'FOR_EACH_EXPR_1',
-'FOR_EACH_FUNCTION',
-'FOREACH_FUNCTION_ARGS',
-'FOREACH_FUNCTION_ARGS_PTR',
-'FOR_EACH_FUNCTION_WITH_GIMPLE_BODY',
-'FOR_EACH_HASH_TABLE_ELEMENT',
-'FOR_EACH_IMM_USE_FAST',
-'FOR_EACH_IMM_USE_ON_STMT',
-'FOR_EACH_IMM_USE_STMT',
-'FOR_EACH_INSN',
-'FOR_EACH_INSN_1',
-'FOR_EACH_INSN_DEF',
-'FOR_EACH_INSN_EQ_USE',
-'FOR_EACH_INSN_INFO_DEF',
-'FOR_EACH_INSN_INFO_EQ_USE',
-'FOR_EACH_INSN_INFO_MW',
-'FOR_EACH_INSN_INFO_USE',
-'FOR_EACH_INSN_USE',
-'FOR_EACH_LOCAL_DECL',
-'FOR_EACH_LOOP',
-'FOR_EACH_LOOP_FN',
-'FOR_EACH_OBJECT',
-'FOR_EACH_OBJECT_CONFLICT',
-'FOR_EACH_PHI_ARG',
-'FOR_EACH_PHI_OR_STMT_DEF',
-'FOR_EACH_PHI_OR_STMT_USE',
-'FOR_EACH_PREF',
-'FOR_EACH_SCALAR',
-'FOR_EACH_SSA_DEF_OPERAND',
-'FOR_EACH_SSA_TREE_OPERAND',
-'FOR_EACH_SSA_USE_OPERAND',
-'FOR_EACH_STATIC_INITIALIZER',
-'FOR_EACH_SUBRTX',
-'FOR_EACH_SUBRTX_PTR',
-'FOR_EACH_SUBRTX_VAR',
-'FOR_EACH_SUCC',
-'FOR_EACH_SUCC_1',
-'FOR_EACH_SYMBOL',
-'FOR_EACH_VARIABLE',
-'FOR_EACH_VEC_ELT',
-'FOR_EACH_VEC_ELT_FROM',
-'FOR_EACH_VEC_ELT_REVERSE',
-'FOR_EACH_VEC_SAFE_ELT',
-'FOR_EACH_VEC_SAFE_ELT_REVERSE',
-'FOR_EXPR',
-'FOR_INIT_STMT',
-'FOR_SCOPE'
-]
+ForEachMacros: [ ]
 IndentCaseLabels: false
 NamespaceIndentation: None
 PenaltyBreakBeforeFirstCallParameter: 100
-- 
2.1.4


___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss

[Dnsmasq-discuss] [PATCH] Remove empty tailing lines

2019-10-13 Thread Geert Stappers 
From: Geert Stappers 

Several .c files have empty lines at their end.
Because there is no need to keep them, are they now removed.
---
 src/arp.c   | 2 --
 src/auth.c  | 3 ---
 src/blockdata.c | 1 -
 src/bpf.c   | 2 --
 src/cache.c | 2 --
 src/conntrack.c | 3 ---
 src/dhcp6.c | 2 --
 src/dnsmasq.c   | 2 --
 src/forward.c   | 5 -
 src/helper.c| 3 ---
 src/inotify.c   | 1 -
 src/lease.c | 4 
 src/netlink.c   | 2 --
 src/network.c   | 5 -
 src/rfc2131.c   | 7 ---
 15 files changed, 44 deletions(-)

diff --git a/src/arp.c b/src/arp.c
index 6cfe014..66ecfb5 100644
--- a/src/arp.c
+++ b/src/arp.c
@@ -230,5 +230,3 @@ int do_arp_script_run(void)
 
   return 0;
 }
-
-
diff --git a/src/auth.c b/src/auth.c
index 854af0d..f12ce4d 100644
--- a/src/auth.c
+++ b/src/auth.c
@@ -863,6 +863,3 @@ size_t answer_auth(struct dns_header *header, char *limit, 
size_t qlen, time_t n
 }
   
 #endif  
-  
-
-
diff --git a/src/blockdata.c b/src/blockdata.c
index e6e625f..89a049d 100644
--- a/src/blockdata.c
+++ b/src/blockdata.c
@@ -174,4 +174,3 @@ struct blockdata *blockdata_read(int fd, size_t len)
 {
   return blockdata_alloc_real(fd, NULL, len);
 }
-
diff --git a/src/bpf.c b/src/bpf.c
index 982318d..6561f1a 100644
--- a/src/bpf.c
+++ b/src/bpf.c
@@ -440,5 +440,3 @@ void route_sock(void)
 }
 
 #endif /* HAVE_BSD_NETWORK */
-
-
diff --git a/src/cache.c b/src/cache.c
index b3ed50b..205b2cf 100644
--- a/src/cache.c
+++ b/src/cache.c
@@ -1994,5 +1994,3 @@ void log_query(unsigned int flags, char *name, union 
all_addr *addr, char *arg)
   else
 my_syslog(LOG_INFO, "%s %s %s %s", source, name, verb, dest);
 }
-
- 
diff --git a/src/conntrack.c b/src/conntrack.c
index d41de54..a11fedd 100644
--- a/src/conntrack.c
+++ b/src/conntrack.c
@@ -83,6 +83,3 @@ static int callback(enum nf_conntrack_msg_type type, struct 
nf_conntrack *ct, vo
 }
 
 #endif
-  
-
-
diff --git a/src/dhcp6.c b/src/dhcp6.c
index 5badc46..4849180 100644
--- a/src/dhcp6.c
+++ b/src/dhcp6.c
@@ -827,5 +827,3 @@ void dhcp_construct_contexts(time_t now)
 }
 
 #endif
-
-
diff --git a/src/dnsmasq.c b/src/dnsmasq.c
index 7842538..b635413 100644
--- a/src/dnsmasq.c
+++ b/src/dnsmasq.c
@@ -2082,5 +2082,3 @@ int delay_dhcp(time_t start, int sec, int fd, uint32_t 
addr, unsigned short id)
   return 0;
 }
 #endif
-
- 
diff --git a/src/forward.c b/src/forward.c
index e4745a3..f488b90 100644
--- a/src/forward.c
+++ b/src/forward.c
@@ -2405,8 +2405,3 @@ static unsigned short get_id(void)
   
   return ret;
 }
-
-
-
-
-
diff --git a/src/helper.c b/src/helper.c
index c392eec..5292e74 100644
--- a/src/helper.c
+++ b/src/helper.c
@@ -886,6 +886,3 @@ void helper_write(void)
 }
 
 #endif
-
-
-
diff --git a/src/inotify.c b/src/inotify.c
index 7107833..7d9c56b 100644
--- a/src/inotify.c
+++ b/src/inotify.c
@@ -295,4 +295,3 @@ int inotify_check(time_t now)
 }
 
 #endif  /* INOTIFY */
-  
diff --git a/src/lease.c b/src/lease.c
index f14d128..f0c93b9 100644
--- a/src/lease.c
+++ b/src/lease.c
@@ -1198,7 +1198,3 @@ void lease_add_extradata(struct dhcp_lease *lease, 
unsigned char *data, unsigned
 #endif
 
 #endif
- 
-
-  
-
diff --git a/src/netlink.c b/src/netlink.c
index eaa772d..91913ac 100644
--- a/src/netlink.c
+++ b/src/netlink.c
@@ -367,5 +367,3 @@ static void nl_async(struct nlmsghdr *h)
 queue_event(EVENT_NEWADDR);
 }
 #endif
-
-  
diff --git a/src/network.c b/src/network.c
index c1077b4..aed820d 100644
--- a/src/network.c
+++ b/src/network.c
@@ -1683,8 +1683,3 @@ void newaddress(time_t now)
 lease_find_interfaces(now);
 #endif
 }
-
-
-
-
-
diff --git a/src/rfc2131.c b/src/rfc2131.c
index ec2650a..1e402d9 100644
--- a/src/rfc2131.c
+++ b/src/rfc2131.c
@@ -2740,10 +2740,3 @@ static void apply_delay(u32 xid, time_t recvtime, struct 
dhcp_netid *netid)
 }
 
 #endif
-  
-
-  
-  
-
-
-  
-- 
2.1.4


___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss

Re: [Dnsmasq-discuss] clang format

2019-10-13 Thread Geert Stappers 
On Sat, Oct 12, 2019 at 11:56:58PM +0100, Simon Kelley wrote:
> On 10/10/2019 22:00, Geert Stappers wrote:
> > Previous-Subject: Re: [Dnsmasq-discuss] [patch] empty empty lines
> > In-Reply-To: <55adb604-91a9-77a8-ed41-500363f4c...@mail.com>
> > On Mon, Sep 30, 2019 at 07:18:12PM +0200, john doe wrote:
> >> On 9/30/2019 4:50 PM, Simon Kelley wrote:
> >>>
> >>>  ...
> >>>
> >>> So, let's come to a consensus if a one-time clean up commit across the
> >>> codebase is a price worth paying to fix the formatting issues, and if it
> >>> is, work out how to add automatic filters to git to keep things clean
> >>> afterwards. If anyone has experience of that, I'd like to hear.
> >>>
> >>
> >> The way I see things which is up for debate:
> >> - Everyone that is committing to the project would need to use 
> > 
> > Where it is OK to get tooling to bend our source code in shape,
> > is it more important to decide in which shape it should be bend.
> > 
> >  ... Linux kernel (C language, several contributors)
> >  have code style documented and have machine reable version ...
> >  
> > Tooling they use is `clangformat`
> >  https://clang.llvm.org/docs/ClangFormat.html
> > 
> > 
> > Please take the above information as input for moving Dnsmasq from
> > a project without a coding style to a project with a coding style.
> > 
> 
> 
> This seems to have escalated from inconsistent use of spaces and tabs to
> "without a coding style". For information, the C coding style in dnsmasq
> is the GNU style, except without the space preceding (
> 
> I like that style, I chose that style and it's not going to change.
> 

Having that choice machine readable in the Dnsmasq source tree
is what I'm aiming for.

Manual page of clang-format (version 8.0.1) says

-style=   - Coding style, currently supports:
  LLVM, Google, Chromium, Mozilla, WebKit.

I don't know which one is closest to the GNU style. A websearch
on "clang format gnu style" did yield a mailinglist posting
 https://gcc.gnu.org/ml/gcc-patches/2015-11/msg02214.html
It starts with
 Following patch adds a clang-format config file that should respect the
 GNU coding standards.

Recent version is 
https://gcc.gnu.org/git/?p=gcc.git;a=blob_plain;f=contrib/clang-format



Groeten
Geert Stappers
-- 
Leven en laten leven

___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss

[Dnsmasq-discuss] dnsmasq 2.81 release plans?

2019-10-13 Thread Matthias Andree 
Simon,

dnsmasq 2.80 is apparently nearing its anniversary - what are the plans
WRT stabilizing, touching up and releasing 2.81? It seems the Git has a
nice collection of regression and other bug fixes.

FreeBSD releases quarterly according to calendar, and something around
second half of November would fit our bill perfectly.

Regards,
Matthias


___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss

Re: [Dnsmasq-discuss] FreeBSD TFTP issue (failure to serve via IPv4) reported against 11.3/12.x

2019-10-13 Thread Matthias Andree 
Am 13.10.19 um 00:34 schrieb Simon Kelley:
> The patch in the buzilla entry looks good to me. The dnsmasq code is
> wrong and has been for some time. The version-dependent failure is
> probably down to the order in which the data is added to the control
> array, or the relative size of the two datastructures changing.
>
> http://thekelleys.org.uk/gitweb/?p=dnsmasq.git;a=commit;h=936bd82755e8f75fc09c1e9a67fb390175b157d4
>
> should fix things. PLs test.

Simon,

I played around with a FreeBSD 11.3-RELEASE jail for amd64.

I could reproduce the original bug as reported,
and can confirm that the patch fixes TFTP service for me.

The FreeBSD port has been updated in the head and quarterly branches,
and package 2.80_4 should fix things.
Thank you.

Regards,
Matthias


___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss