Re: Test failure for popt 1.16
Actually the previous patch is bogus. Right idea, just
the wrong way to fix.
Try this patch instead:
Index: popthelp.c
===
RCS file: /v/rpm/cvs/popt/popthelp.c,v
retrieving revision 1.94
diff -p -u -w -r1.94 popthelp.c
--- popthelp.c 28 Aug 2009 00:06:33 - 1.94
+++ popthelp.c 11 May 2010 19:12:28 -
@@ -652,6 +652,8 @@ static size_t showHelpIntro(poptContext
const char * fn = (os->argv ? os->argv[0] : NULL);
if (fn == NULL) return len;
if (strchr(fn, '/')) fn = strrchr(fn, '/') + 1;
+ if (fn[0] == 'l' && fn[1] == 't' && fn[2] == '-')
+ fn += sizeof("lt-") - 1;
/* XXX POPT_fprintf not needed for argv[0] display. */
fprintf(fp, " %s", fn);
len += strlen(fn) + 1;
Index: testit.sh
===
RCS file: /v/rpm/cvs/popt/testit.sh,v
retrieving revision 1.31
diff -p -u -w -r1.31 testit.sh
--- testit.sh 26 Jul 2009 16:25:23 - 1.31
+++ testit.sh 11 May 2010 19:12:29 -
@@ -115,7 +115,7 @@ run test1 "test1 - 56" "arg1: 0 arg2: (n
run test1 "test1 - 57" "arg1: 0 arg2: (none) aBits: foo,baz" --bits
foo,bar,baz,!bar
run test1 "test1 - 58" "\
-Usage: lt-test1 [-I?] [-c|--cb2=STRING] [--arg1] [-2|--arg2=ARG]
+Usage: test1 [-I?] [-c|--cb2=STRING] [--arg1] [-2|--arg2=ARG]
[-3|--arg3=ANARG] [-onedash] [--optional=STRING] [--val]
[-i|--int=INT] [-s|--short=SHORT] [-l|--long=LONG]
[-L|--longlong=LONGLONG] [-f|--float=FLOAT] [-d|--double=DOUBLE]
@@ -124,7 +124,7 @@ Usage: lt-test1 [-I?] [-c|--cb2=STRING]
[--bitxor] [--nstr=STRING] [--lstr=STRING] [-I|--inc]
[-c|--cb=STRING] [--longopt] [-?|--help] [--usage] [--simple=ARG]"
--usage
run test1 "test1 - 59" "\
-Usage: lt-test1 [OPTION...]
+Usage: test1 [OPTION...]
--arg1 First argument with a really long
description. After all, we have to test
argument help wrapping somehow, right?
And if you can confirm the patch "works" in the next 24 hours,
I'll re-roll the (unannounced) popt-1.16.tar.gz.
Otherwise, I'll figger better for popt-1.17.
hth
73 de Jeff
On May 11, 2010, at 2:54 PM, Jeff Johnson wrote:
>
> On May 10, 2010, at 7:57 PM, Pieter Bowman wrote:
>
>> I did builds of popt 1.16 on a number of our systems here. Test 59
>> failed on a number of the systems with the following output:
>>
>
> The "fix" for the failure in popt-1.16 "make check" is likely (I have
> easy no easy means of testing across all the platforms that you have)
> as simple as the atached patch.
>
> If you can confirm that the patch "works", I'll re-roll the popt-1.16
> tarballs, and re-release (it hasn't been announced anywhere)
> the popt-1.16 tarball.
>
> hth
>
> 73 de Jeff
> ==
> Index: test1.c
> ===
> RCS file: /v/rpm/cvs/popt/test1.c,v
> retrieving revision 1.44
> diff -p -u -w -r1.44 test1.c
> --- test1.c 26 Jul 2009 16:38:25 - 1.44
> +++ test1.c 11 May 2010 18:50:39 -
> @@ -252,7 +252,7 @@ int main(int argc, const char ** argv)
> resetVars();
> /*...@=modobserver@*/
> /*...@-temptrans@*/
> -optCon = poptGetContext("test1", argc, argv, options, 0);
> +optCon = poptGetContext("test1", argc, argv, options,
> POPT_CONTEXT_KEEP_FIRST);
> /*...@=temptrans@*/
> (void) poptReadConfigFile(optCon, "./test-poptrc");
> (void) poptReadDefaultConfig(optCon, 1);
>
>
> __
> POPT Library http://rpm5.org
> Developer Communication List [email protected]
__
POPT Library http://rpm5.org
Developer Communication List [email protected]
Re: Test failure for popt 1.16
On May 10, 2010, at 7:57 PM, Pieter Bowman wrote:
> I did builds of popt 1.16 on a number of our systems here. Test 59
> failed on a number of the systems with the following output:
>
The "fix" for the failure in popt-1.16 "make check" is likely (I have
easy no easy means of testing across all the platforms that you have)
as simple as the atached patch.
If you can confirm that the patch "works", I'll re-roll the popt-1.16
tarballs, and re-release (it hasn't been announced anywhere)
the popt-1.16 tarball.
hth
73 de Jeff
==
Index: test1.c
===
RCS file: /v/rpm/cvs/popt/test1.c,v
retrieving revision 1.44
diff -p -u -w -r1.44 test1.c
--- test1.c 26 Jul 2009 16:38:25 - 1.44
+++ test1.c 11 May 2010 18:50:39 -
@@ -252,7 +252,7 @@ int main(int argc, const char ** argv)
resetVars();
/*...@=modobserver@*/
/*...@-temptrans@*/
-optCon = poptGetContext("test1", argc, argv, options, 0);
+optCon = poptGetContext("test1", argc, argv, options,
POPT_CONTEXT_KEEP_FIRST);
/*...@=temptrans@*/
(void) poptReadConfigFile(optCon, "./test-poptrc");
(void) poptReadDefaultConfig(optCon, 1);
__
POPT Library http://rpm5.org
Developer Communication List [email protected]
Re: Test failure for popt 1.16
This is largely a cosmetic issue introduced by using libtool (i.e. "test1" != "lt-test1"in argv[0]). But yes, could/should be fixed. 73 de Jeff On May 10, 2010, at 7:57 PM, Pieter Bowman wrote: > I did builds of popt 1.16 on a number of our systems here. Test 59 > failed on a number of the systems with the following output: > > Test "test1 --usage" failed with: "Usage: test1 [-I?] [-c|--cb2=STRING] > [--arg1] [-2|--arg2=ARG] >[-3|--arg3=ANARG] [-onedash] [--optional=STRING] [--val] >[-i|--int=INT] [-s|--short=SHORT] [-l|--long=LONG] >[-L|--longlong=LONGLONG] [-f|--float=FLOAT] [-d|--double=DOUBLE] >[--randint=INT] [--randshort=SHORT] [--randlong=LONG] >[--randlonglong=LONGLONG] [--argv=STRING] [--bitset] [--bitclr] >[--bitxor] [--nstr=STRING] [--lstr=STRING] [-I|--inc] >[-c|--cb=STRING] [--longopt] [-?|--help] [--usage] [--simple=ARG]" != > "Usage: lt-test1 [-I?] [-c|--cb2=STRING] [--arg1] [-2|--arg2=ARG] >[-3|--arg3=ANARG] [-onedash] [--optional=STRING] [--val] >[-i|--int=INT] [-s|--short=SHORT] [-l|--long=LONG] >[-L|--longlong=LONGLONG] [-f|--float=FLOAT] [-d|--double=DOUBLE] >[--randint=INT] [--randshort=SHORT] [--randlong=LONG] >[--randlonglong=LONGLONG] [--argv=STRING] [--bitset] [--bitclr] >[--bitxor] [--nstr=STRING] [--lstr=STRING] [-I|--inc] >[-c|--cb=STRING] [--longopt] [-?|--help] [--usage] [--simple=ARG]" > > The operating systems which failed were: > > gentoo/Linux, Alpha, PowerPC, PowerPC64, SPARC > MacOS X 10.5, PowerPC, Intel > FreeBSD 8, x86 > MirOS (aka. MirBSD), x86 > NetBSD 5.0.2, x86 > OpenBSD 4.7, x86 > OpenSolaris snv_134 X86 > OpenSUSE 11.2, x86_64 > Solaris 10 x86, x86_64, SPARC > > The operating systems which succeeded were: > > RedHat Enterprise Linux 5.5, x86, x86_64, IA64 > Ubuntu 9.10, x86 > Fedora 12, x86 > > Thanks for looking into this. > > Pieter > __ > POPT Library http://rpm5.org > Developer Communication List [email protected] __ POPT Library http://rpm5.org Developer Communication List [email protected]
