Re: [pacman-dev] [PATCH v3 6/8] bacman: add otpion to alter the output directory

2016-08-31 Thread Allan McRae
On 01/09/16 07:05, Gordian Edenhofer wrote: > Signed-off-by: Gordian Edenhofer > --- > contrib/bacman.sh.in | 14 ++ > 1 file changed, 10 insertions(+), 4 deletions(-) > > diff --git a/contrib/bacman.sh.in b/contrib/bacman.sh.in > index 322bb66..706c6fb 100644 > --- a/contrib/bacman.

Re: [pacman-dev] [PATCH v3 4/8] bacman: rewrite usage page and version information

2016-08-31 Thread Allan McRae
On 01/09/16 13:59, Allan McRae wrote: > On 01/09/16 07:05, Gordian Edenhofer wrote: >> Signed-off-by: Gordian Edenhofer >> --- >> contrib/bacman.sh.in | 33 + >> 1 file changed, 21 insertions(+), 12 deletions(-) >> > > The format for the header is the same for all

Re: [pacman-dev] [PATCH v3 5/8] bacman: add option to print fewer status updates

2016-08-31 Thread Allan McRae
On 01/09/16 07:05, Gordian Edenhofer wrote: > Signed-off-by: Gordian Edenhofer > --- > contrib/bacman.sh.in | 21 - > 1 file changed, 16 insertions(+), 5 deletions(-) > Look at the definition of msg2() etc: scripts/library/output_format.sh

Re: [pacman-dev] [PATCH v3 4/8] bacman: rewrite usage page and version information

2016-08-31 Thread Allan McRae
On 01/09/16 07:05, Gordian Edenhofer wrote: > Signed-off-by: Gordian Edenhofer > --- > contrib/bacman.sh.in | 33 + > 1 file changed, 21 insertions(+), 12 deletions(-) > The format for the header is the same for all scripts in the pacman codebase. Do not change

Re: [pacman-dev] [PATCH v3 3/8] bacman: proper option handling

2016-08-31 Thread Allan McRae
On 01/09/16 07:05, Gordian Edenhofer wrote: > Switch to parseopts instead of merely checking the first argument. > > Signed-off-by: Gordian Edenhofer > --- > contrib/bacman.sh.in | 63 > ++-- > 1 file changed, 41 insertions(+), 22 deletions(-) >

[pacman-dev] [PATCH] Fix typo in pacman-db-upgrade usage message

2016-08-31 Thread Allan McRae
Signed-off-by: Allan McRae --- scripts/pacman-db-upgrade.sh.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/pacman-db-upgrade.sh.in b/scripts/pacman-db-upgrade.sh.in index 94085ba..79e1c59 100644 --- a/scripts/pacman-db-upgrade.sh.in +++ b/scripts/pacman-db-upgrade.

Re: [pacman-dev] [PATCH v3 1/8] bacman: allow for multiple packages as arguments

2016-08-31 Thread Allan McRae
On 01/09/16 07:05, Gordian Edenhofer wrote: > To enable the creation of multiple packages with one command move the > assembly process into its own function. > Handle SIGHUP, SIGINT, SIGTERM and remove working directories > accordingly. > Add some comments. > > Signed-off-by: Gordian Edenhofer

Re: [pacman-dev] [PATCH v2 1/3] bacman: allow for parallel packaging

2016-08-31 Thread Allan McRae
On 01/09/16 09:44, Gordian Edenhofer wrote: > On Thu, 2016-09-01 at 08:28 +1000, Allan McRae wrote: >> On 01/09/16 08:08, Dave Reisner wrote: >>> >>> On Wed, Aug 31, 2016 at 11:18:32PM +0200, Gordian Edenhofer wrote: > >> >>> >>> The second probably would not be accepted... >>>

Re: [pacman-dev] [PATCH v2 1/3] bacman: allow for parallel packaging

2016-08-31 Thread Gordian Edenhofer
On Thu, 2016-09-01 at 08:28 +1000, Allan McRae wrote: > On 01/09/16 08:08, Dave Reisner wrote: > > > > On Wed, Aug 31, 2016 at 11:18:32PM +0200, Gordian Edenhofer wrote: > > > > > > > > > > > > > > > > > > > > > > > > The second probably would not be accepted... > > > > > > > > > > I urge you

Re: [pacman-dev] [PATCH v2 1/3] bacman: allow for parallel packaging

2016-08-31 Thread Gordian Edenhofer
On Wed, 2016-08-31 at 18:08 -0400, Dave Reisner wrote: > On Wed, Aug 31, 2016 at 11:18:32PM +0200, Gordian Edenhofer wrote: > > > > > > > > > > > > > > > > > > > The second probably would not be accepted... > > > > > > > > I urge you to reconsider. Parallelization increases the speed > > > > o

Re: [pacman-dev] [PATCH v2 1/3] bacman: allow for parallel packaging

2016-08-31 Thread Allan McRae
On 01/09/16 08:08, Dave Reisner wrote: > On Wed, Aug 31, 2016 at 11:18:32PM +0200, Gordian Edenhofer wrote: > The second probably would not be accepted... I urge you to reconsider. Parallelization increases the speed of this >>> >>> I don't think anyone is suggesting that packagi

Re: [pacman-dev] [PATCH v2 1/3] bacman: allow for parallel packaging

2016-08-31 Thread Dave Reisner
On Wed, Aug 31, 2016 at 11:18:32PM +0200, Gordian Edenhofer wrote: > > > > The second probably would not be accepted... > > > > > > I urge you to reconsider. Parallelization increases the speed of > > > this > > > > I don't think anyone is suggesting that packaging multiple things in > > parallel

Re: [pacman-dev] [PATCH v2 1/3] bacman: allow for parallel packaging

2016-08-31 Thread Gordian Edenhofer
> > > The second probably would not be accepted... > > > > I urge you to reconsider. Parallelization increases the speed of > > this > > I don't think anyone is suggesting that packaging multiple things in > parallel isn't useful. I already suggested that nothing needs to be > implemented in bacm

[pacman-dev] [PATCH v3 7/8] bacman: add option to specify the number of jobs

2016-08-31 Thread Gordian Edenhofer
Signed-off-by: Gordian Edenhofer --- contrib/bacman.sh.in | 27 +-- 1 file changed, 21 insertions(+), 6 deletions(-) diff --git a/contrib/bacman.sh.in b/contrib/bacman.sh.in index 706c6fb..e8e53ea 100644 --- a/contrib/bacman.sh.in +++ b/contrib/bacman.sh.in @@ -57,11 +57,

[pacman-dev] [PATCH v3 5/8] bacman: add option to print fewer status updates

2016-08-31 Thread Gordian Edenhofer
Signed-off-by: Gordian Edenhofer --- contrib/bacman.sh.in | 21 - 1 file changed, 16 insertions(+), 5 deletions(-) diff --git a/contrib/bacman.sh.in b/contrib/bacman.sh.in index ca2945f..322bb66 100644 --- a/contrib/bacman.sh.in +++ b/contrib/bacman.sh.in @@ -28,6 +28,7 @@ de

[pacman-dev] [PATCH v3 6/8] bacman: add otpion to alter the output directory

2016-08-31 Thread Gordian Edenhofer
Signed-off-by: Gordian Edenhofer --- contrib/bacman.sh.in | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/contrib/bacman.sh.in b/contrib/bacman.sh.in index 322bb66..706c6fb 100644 --- a/contrib/bacman.sh.in +++ b/contrib/bacman.sh.in @@ -57,10 +57,11 @@ usage()

[pacman-dev] [PATCH v3 8/8] bacman: add manual page

2016-08-31 Thread Gordian Edenhofer
Signed-off-by: Gordian Edenhofer --- contrib/doc/.gitignore | 1 + contrib/doc/Makefile.am | 5 +++- contrib/doc/bacman.8.txt | 71 3 files changed, 76 insertions(+), 1 deletion(-) create mode 100644 contrib/doc/bacman.8.txt diff --git a/con

[pacman-dev] [PATCH v3 4/8] bacman: rewrite usage page and version information

2016-08-31 Thread Gordian Edenhofer
Signed-off-by: Gordian Edenhofer --- contrib/bacman.sh.in | 33 + 1 file changed, 21 insertions(+), 12 deletions(-) diff --git a/contrib/bacman.sh.in b/contrib/bacman.sh.in index a0bfcf1..ca2945f 100644 --- a/contrib/bacman.sh.in +++ b/contrib/bacman.sh.in @@ -41,

[pacman-dev] [PATCH v3 3/8] bacman: proper option handling

2016-08-31 Thread Gordian Edenhofer
Switch to parseopts instead of merely checking the first argument. Signed-off-by: Gordian Edenhofer --- contrib/bacman.sh.in | 63 ++-- 1 file changed, 41 insertions(+), 22 deletions(-) diff --git a/contrib/bacman.sh.in b/contrib/bacman.sh.in inde

[pacman-dev] [PATCH v3 2/8] bacman: parallel packaging

2016-08-31 Thread Gordian Edenhofer
Signed-off-by: Gordian Edenhofer --- contrib/bacman.sh.in | 23 ++- 1 file changed, 22 insertions(+), 1 deletion(-) diff --git a/contrib/bacman.sh.in b/contrib/bacman.sh.in index a13f074..1241718 100644 --- a/contrib/bacman.sh.in +++ b/contrib/bacman.sh.in @@ -348,8 +348,29 @

[pacman-dev] [PATCH v3 1/8] bacman: allow for multiple packages as arguments

2016-08-31 Thread Gordian Edenhofer
To enable the creation of multiple packages with one command move the assembly process into its own function. Handle SIGHUP, SIGINT, SIGTERM and remove working directories accordingly. Add some comments. Signed-off-by: Gordian Edenhofer --- contrib/bacman.sh.in | 442 +---

Re: [pacman-dev] [PATCH v2 1/3] bacman: allow for parallel packaging

2016-08-31 Thread Gordian Edenhofer
On Tue, 2016-08-30 at 20:30 +1000, Allan McRae wrote: > On 15/08/16 06:39, Gordian Edenhofer wrote: > > > > * move the actual assembly process into its own function > > * allow for packaging multiple packages with one command > > * handle SIGHUP SIGINT SIGTERM and remove working dirs accordingly >

Re: [pacman-dev] [PATCH v2 1/3] bacman: allow for parallel packaging

2016-08-31 Thread Dave Reisner
On Wed, Aug 31, 2016 at 10:35:37PM +0200, Gordian Edenhofer wrote: > On Tue, 2016-08-30 at 20:30 +1000, Allan McRae wrote: > > On 15/08/16 06:39, Gordian Edenhofer wrote: > > > > > > * move the actual assembly process into its own function > > > * allow for packaging multiple packages with one com

Re: [pacman-dev] [PATCH] Replace CURLOPT_PROGRESSFUNCTION with CURLOPT_XFERINFOFUNCTION

2016-08-31 Thread Will Miles
On 2016-08-30 11:01 PM, Allan McRae wrote: OK. I don't think pacman is used anywhere where curl older than August 2013 is an issue... Allan I guess I'm the exception that proves the rule -- I'm using pacman on QNX 6.6, which ships with libcurl 7.24.0 from Jan 2012.Alas, RTOS folks are