Re: [LEDE-DEV] [PATCH] jshn: add functionality to read big JSON

2018-01-17 Thread John Crispin
On 17/01/18 10:44, Arjen de Korte wrote: Citeren John Crispin : On 07/01/18 18:08, Christian Beier wrote: The existing read functionality feeds the complete JSON to jshn as a cmdline argument, leading to `-ash: jshn: Argument list too long` errors for JSONs bigger than ca.

Re: [LEDE-DEV] [PATCH] jshn: add functionality to read big JSON

2018-01-17 Thread Arjen de Korte
Citeren John Crispin : On 07/01/18 18:08, Christian Beier wrote: The existing read functionality feeds the complete JSON to jshn as a cmdline argument, leading to `-ash: jshn: Argument list too long` errors for JSONs bigger than ca. 100KB. This commit adds the ability to

Re: [LEDE-DEV] [PATCH] jshn: add functionality to read big JSON

2018-01-17 Thread John Crispin
On 07/01/18 18:08, Christian Beier wrote: The existing read functionality feeds the complete JSON to jshn as a cmdline argument, leading to `-ash: jshn: Argument list too long` errors for JSONs bigger than ca. 100KB. This commit adds the ability to read the JSON directly from a file if

Re: [LEDE-DEV] [PATCH] jshn: add functionality to read big JSON

2017-12-22 Thread Christian Beier
Am Tue, 12 Dec 2017 19:39:48 +0100 schrieb Christian Beier : > Am Tue, 12 Dec 2017 12:03:50 +0100 > schrieb John Crispin : > > > Hi Christian, > > > > small formatting nitpick inline ... > > Hi John, > > Re-sending with tab adjustments and a

Re: [LEDE-DEV] [PATCH] jshn: add functionality to read big JSON

2017-12-12 Thread Christian Beier via Lede-dev
The sender domain has a DMARC Reject/Quarantine policy which disallows sending mailing list messages using the original "From" header. To mitigate this problem, the original message has been wrapped automatically by the mailing list software.--- Begin Message --- Am Tue, 12 Dec 2017 12:03:50

Re: [LEDE-DEV] [PATCH] jshn: add functionality to read big JSON

2017-12-12 Thread John Crispin
Hi Christian, small formatting nitpick inline ... On 16/11/17 19:52, Christian Beier wrote: The existing read functionality feeds the complete JSON to jshn as a cmdline argument, leading to `-ash: jshn: Argument list too long` errors for JSONs bigger than ca. 100KB. This commit adds the

Re: [LEDE-DEV] [PATCH] jshn: add functionality to read big JSON

2017-11-19 Thread Alexandru Ardelean
On Sat, Nov 18, 2017 at 3:55 PM, Christian Beier wrote: > Am Fri, 17 Nov 2017 09:17:39 +0200 > schrieb Alexandru Ardelean : > >> On Thu, Nov 16, 2017 at 8:52 PM, Christian Beier >> wrote: >> > The existing read functionality feeds

Re: [LEDE-DEV] [PATCH] jshn: add functionality to read big JSON

2017-11-16 Thread Alexandru Ardelean
On Thu, Nov 16, 2017 at 8:52 PM, Christian Beier wrote: > The existing read functionality feeds the complete JSON to jshn as a > cmdline argument, leading to `-ash: jshn: Argument list too long` > errors for JSONs bigger than ca. 100KB. > > This commit adds the ability to

[LEDE-DEV] [PATCH] jshn: add functionality to read big JSON

2017-11-16 Thread Christian Beier
The existing read functionality feeds the complete JSON to jshn as a cmdline argument, leading to `-ash: jshn: Argument list too long` errors for JSONs bigger than ca. 100KB. This commit adds the ability to read the JSON directly from a file if wanted, removing this shell-imposed size limit.