[pacman-dev] [PATCH] doc/.gitignore: ignore all manpages

2018-02-10 Thread iff
From: Ivy Foster They're all generated during make. --- I was going to add BUILDINFO.5 to the list, but this way covers future additions. doc/.gitignore | 14 +- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/doc/.gitignore b/doc/.gitignore index

Re: [pacman-dev] [PATCH v3 4/5] pacman-conf.c: Make cleanup run automagically at exit

2018-02-10 Thread Andrew Gregory
On 02/10/18 at 10:15pm, i...@escondida.tk wrote: > From: Ivy Foster > > Since using atexit(3) draws in stdlib anyway and this changes every > exit call at large and return call in main(), go ahead and use > EXIT_SUCCESS and EXIt_FAILURE, too. > > Signed-off-by: Ivy Foster

Re: [pacman-dev] [PATCH v3 2/5] pacman-conf.c: simplify usage()

2018-02-10 Thread Andrew Gregory
On 02/10/18 at 10:15pm, i...@escondida.tk wrote: > From: Ivy Foster > > Signed-off-by: Ivy Foster > --- There are two major changes here: adding short options to the usage and the "simplification" of the usage function. The short options should be added

[pacman-dev] [PATCH v2 5/5] pacman-conf.c: make parse_opts only parse opts

2018-02-10 Thread iff
From: Ivy Foster Previously, it also parsed the config file instead of simply setting the (global) config_file variable Signed-off-by: Ivy Foster --- src/pacman/pacman-conf.c | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git

[pacman-dev] [PATCH v2 2/5] pacman-conf.c: simplify usage()

2018-02-10 Thread iff
From: Ivy Foster Signed-off-by: Ivy Foster --- src/pacman/pacman-conf.c | 40 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/src/pacman/pacman-conf.c b/src/pacman/pacman-conf.c index

[pacman-dev] [PATCH v2 4/5] pacman-conf.c: Make cleanup run automagically at exit

2018-02-10 Thread iff
From: Ivy Foster Since using atexit(3) draws in stdlib anyway and this changes every exit call at large and return call in main(), go ahead and use EXIT_SUCCESS and EXIt_FAILURE, too. Signed-off-by: Ivy Foster --- src/pacman/pacman-conf.c | 29

[pacman-dev] [PATCH v2 3/5] pacman-conf: Make myname and myver into cpp macros MYNAME and MYVER

2018-02-10 Thread iff
From: Ivy Foster This allows MYNAME to be used in place of the literal string "pacman-conf" in usage Signed-off-by: Ivy Foster --- src/pacman/pacman-conf.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git

[pacman-dev] [PATCH v2 1/5] pacman-conf.c: accept short options

2018-02-10 Thread iff
From: Ivy Foster Signed-off-by: Ivy Foster --- This verion of the patch simply drops the check for '?', since it's caught by "default" anyway. The other patches are unchanged, except where rebased against this modified patch. src/pacman/pacman-conf.c | 3

[pacman-dev] [PATCH v3 1/5] pacman-conf.c: accept short options

2018-02-10 Thread iff
From: Ivy Foster Signed-off-by: Ivy Foster --- As per agregory's request in #archlinux-pacman, this version of the patch leaves the check for '?' well enough alone. '?' is getopt_long(3)'s way of indicating user error, and we can think of default: as

[pacman-dev] [PATCH v3 5/5] pacman-conf.c: make parse_opts only parse opts

2018-02-10 Thread iff
From: Ivy Foster Previously, it also parsed the config file instead of simply setting the (global) config_file variable Signed-off-by: Ivy Foster --- src/pacman/pacman-conf.c | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git

[pacman-dev] [PATCH v3 3/5] pacman-conf: Make myname and myver into cpp macros MYNAME and MYVER

2018-02-10 Thread iff
From: Ivy Foster This allows MYNAME to be used in place of the literal string "pacman-conf" in usage Signed-off-by: Ivy Foster --- src/pacman/pacman-conf.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git

[pacman-dev] [PATCH v3 2/5] pacman-conf.c: simplify usage()

2018-02-10 Thread iff
From: Ivy Foster Signed-off-by: Ivy Foster --- src/pacman/pacman-conf.c | 40 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/src/pacman/pacman-conf.c b/src/pacman/pacman-conf.c index

Re: [pacman-dev] [PATCH v3 5/5] pacman-conf.c: make parse_opts only parse opts

2018-02-10 Thread Andrew Gregory
On 02/10/18 at 10:15pm, i...@escondida.tk wrote: > From: Ivy Foster > > Previously, it also parsed the config file instead of simply setting > the (global) config_file variable I think the original purpose of this patch is fundamentally unnecessary, but it does (accidentally,

[pacman-dev] [PATCH] makepkg: use localized date strings

2018-02-10 Thread Chih-Hsuan Yen
From: Yen Chi Hsuan Before this change, LC_TIME=zh_TW.UTF-8 makepkg has the following line: ==> Making package: foobar 1-1 (日 2月 11 01:13:42 CST 2018) With this patch, this line becomes: ==> Making package: foobar 1-1 (西元2018年02月11日 (週日) 01時13分57秒) The latter is more