Re: [LEDE-DEV] [PATCH] odhcpd: Update to 2016-12-01 snapshot

2016-12-04 Thread John Crispin
Hi Karl, this comment is completely devoid any usefulness. let me help by rephrasing it to make it useful. "Hi community, i have noticed that it is common practice to not annotate the upstream changes properly in commit bumping the git hash of packages. i feel that it would be much better if we

Re: [LEDE-DEV] [PATCH v5 2/3] ipq806x: Add support for new device: tew827dru

2016-12-04 Thread J Mo
On 12/01/2016 11:18 PM, Mathias Kresin wrote: 02.12.2016 06:28, J Mo: On 11/27/2016 02:29 AM, Mathias Kresin wrote: I asked you three (!) times to _explain_ what this code should do [0][1][2]. Now I see the very same code again without having ever seen the requested explanation. This stil

[LEDE-DEV] [PATCH v2] odhcpd: Update to 2016-12-01 snapshot

2016-12-04 Thread Florian Fainelli
Brings in the following changes: 41b526884030 dhcpv6-ia : Fix static DHCPv6 assignments becoming non static be6c5159a46b dhcpv6-ia: Fix assignment of static DHCPv6 leases 374dc3f13623 cmake: Find libubox/uloop.h 01c919c49ccb odhcpd: Display infinite valid lifetime as -1 Signed-off-by: Florian Fai

Re: [LEDE-DEV] [PATCH] odhcpd: Update to 2016-12-01 snapshot

2016-12-04 Thread Florian Fainelli
On 12/04/2016 03:17 PM, Karl Palsson wrote: > > Florian Fainelli wrote: >> On 12/04/2016 02:53 PM, Karl Palsson wrote: >>> Care to explain what you're updating? This patch is completely >>> devoid of useful information without resorting to external >>> sources. Surely, you have motivation for upd

Re: [LEDE-DEV] [PATCH] odhcpd: Update to 2016-12-01 snapshot

2016-12-04 Thread Karl Palsson
Florian Fainelli wrote: > On 12/04/2016 02:53 PM, Karl Palsson wrote: > > Care to explain what you're updating? This patch is completely > > devoid of useful information without resorting to external > > sources. Surely, you have motivation for updating? > > Having this commit below, and the oth

[LEDE-DEV] [PATCH uclient v2] Fix unused results warnings

2016-12-04 Thread Florian Fainelli
Fixes: uclient-http.c:385:8: error: ignoring return value of 'fread', declared with attribute warn_unused_result [-Werror=unused-result] fread(&val, sizeof(val), 1, f); ^ uclient-fetch.c: In function 'main': uclient-fetch.c:664:12: error: ignoring return value of 'asprintf', declared

[LEDE-DEV] [PATCH fstools v2] libfstools: Check return values for fread and system

2016-12-04 Thread Florian Fainelli
libfstools/rootdisk.c: In function 'rootdisk_volume_identify': libfstools/rootdisk.c:172:7: error: ignoring return value of 'fread', declared with attribute warn_unused_result [-Werror=unused-result] fread(&magic, sizeof(magic), 1, f); ^ libfstools/rootdisk.c:179:7: error: ignoring return

Re: [LEDE-DEV] [PATCH] odhcpd: Update to 2016-12-01 snapshot

2016-12-04 Thread Florian Fainelli
On 12/04/2016 02:53 PM, Karl Palsson wrote: > Care to explain what you're updating? This patch is completely > devoid of useful information without resorting to external > sources. Surely, you have motivation for updating? Having this commit below, and the other two make on top which are bugfixes

Re: [LEDE-DEV] [PATCH] odhcpd: Update to 2016-12-01 snapshot

2016-12-04 Thread Karl Palsson
Care to explain what you're updating? This patch is completely devoid of useful information without resorting to external sources. Surely, you have motivation for updating? Sincerely, Karl Palsson Florian Fainelli wrote: > Signed-off-by: Florian Fainelli > --- > package/network/services/odhcp

Re: [LEDE-DEV] [PATCH uclient] Fix unused results warnings

2016-12-04 Thread Florian Fainelli
Le 12/04/16 à 07:14, John Crispin a écrit : > > > On 03/12/2016 18:58, Florian Fainelli wrote: >> Fixes: >> >> uclient-http.c:385:8: error: ignoring return value of 'fread', declared with >> attribute warn_unused_result [-Werror=unused-result] >>fread(&val, sizeof(val), 1, f); >> ^ >

Re: [LEDE-DEV] [PATCH uclient] Fix unused results warnings

2016-12-04 Thread John Crispin
On 03/12/2016 18:58, Florian Fainelli wrote: > Fixes: > > uclient-http.c:385:8: error: ignoring return value of 'fread', declared with > attribute warn_unused_result [-Werror=unused-result] >fread(&val, sizeof(val), 1, f); > ^ > > uclient-fetch.c: In function 'main': > uclient-fetc

Re: [LEDE-DEV] [PATCH fstools] libfstools: Check return values for fread and system

2016-12-04 Thread John Crispin
On 03/12/2016 18:32, Florian Fainelli wrote: > libfstools/rootdisk.c: In function 'rootdisk_volume_identify': > libfstools/rootdisk.c:172:7: error: ignoring return value of 'fread', > declared with attribute warn_unused_result [-Werror=unused-result] > fread(&magic, sizeof(magic), 1, f); >