Bug #19699 for Params-Validate: does not install on HP-UX 11.00.

Mon Jun 05 05:39:49 2006 Guest - Ticket created

Subject:

does not install on HP-UX 11.00.

Params-Validade-0.84 does not install on HP-UX 11.00. The .c generated seems to assume that HP-UX understands inline...I fixed it undef'ing INLINE #define INLINE HP-UX and _MSC_VER should go and the same footing I am using perl5.8.3. I am sgt on perlmonks if you need some test on HP-UX 11.00 (or 11.23) let me know --steph

Mon Jun 05 10:11:29 2006 autarch [...] urth.org - Correspondence added

Subject:	Re: [rt.cpan.org #19699] does not install on HP-UX 11.00.
Date:	Mon, 5 Jun 2006 09:10:15 -0500 (CDT)
To:	Guest via RT <bug-Params-Validate [...] rt.cpan.org>
From:	Dave Rolsky <autarch [...] urth.org>

On Mon, 5 Jun 2006, Guest via RT wrote: Show quoted text

> Params-Validade-0.84 does not install on HP-UX 11.00. The .c generated > seems to assume that HP-UX understands inline...I fixed it undef'ing INLINE > #define INLINE > HP-UX and _MSC_VER should go and the same footing > I am using perl5.8.3.

Can you send me a patch? I don't know exactly what symbol to check for. -dave /*=================================================== VegGuide.Org www.BookIRead.com Your guide to all that's veg. My book blog ===================================================*/

Mon Jun 05 10:11:30 2006 The RT System itself - Status changed from 'new' to 'open'

Mon Jun 05 14:25:03 2006 stephangt4 [...] hotmail.com - Correspondence added

CC:	sgt19 [...] tid.es, stepahngt4 [...] hotmail.com
Subject:	Re: [rt.cpan.org #19699] does not install on HP-UX 11.00.
Date:	Mon, 05 Jun 2006 18:24:07 +0000
To:	bug-Params-Validate [...] rt.cpan.org
From:	"stephan titard" <stephangt4 [...] hotmail.com>

Show quoted text

>From: "autarch@urth.org via RT" <bug-Params-Validate@rt.cpan.org> >Reply-To: bug-Params-Validate@rt.cpan.org >To: stephangt4@hotmail.com >Subject: Re: [rt.cpan.org #19699] does not install on HP-UX 11.00. Date: >Mon, 5 Jun 2006 10:11:30 -0400 (EDT) > > ><URL: http://rt.cpan.org/Ticket/Display.html?id=19699 > > >On Mon, 5 Jun 2006, Guest via RT wrote: >

> > Params-Validade-0.84 does not install on HP-UX 11.00. The .c generated > > seems to assume that HP-UX understands inline...I fixed it undef'ing

>INLINE

> > #define INLINE > > HP-UX and _MSC_VER should go and the same footing > > I am using perl5.8.3.

> >Can you send me a patch? I don't know exactly what symbol to check for. > >

Hi, well to be honest a patch is not so easy (as I have to understand how the INLINE C Macro is generated by MM) except if you get rid of the INLINE C Macro; I can understand it for a C++ class but in C, what is the use?... it seems that any C compiler should complain now it could be that MakeMaker generates something weird (at least on HP-UX) I'll check that and let you know --steph Show quoted text

>-dave > >/*=================================================== >VegGuide.Org www.BookIRead.com >Your guide to all that's veg. My book blog >===================================================*/ >

Mon Jun 05 14:50:42 2006 autarch [...] urth.org - Correspondence added

Subject:	Re: [rt.cpan.org #19699] does not install on HP-UX 11.00.
Date:	Mon, 5 Jun 2006 13:49:35 -0500 (CDT)
To:	"stephangt4 [...] hotmail.com via RT" <bug-Params-Validate [...] rt.cpan.org>
From:	Dave Rolsky <autarch [...] urth.org>

On Mon, 5 Jun 2006, stephangt4@hotmail.com via RT wrote: Show quoted text

> Hi, well to be honest a patch is not so easy (as I have to understand how > the INLINE C Macro is generated by MM) except if you get rid of the INLINE C > Macro; I can understand it for a C++ class > but in C, what is the use?... it seems that any C compiler should complain > > now it could be that MakeMaker generates something weird (at least on HP-UX) > I'll check that and let you know

I'm not sure what you're looking at. If you look at the Validate.xs file you'll see it defines its own INLINE macro, which is either empty (is _MSC_VER is defined) or the string "inline". I'm just asked you to patch the file so that if the compiler is the HP-UX compiler, that macro is empty. -dave /*=================================================== VegGuide.Org www.BookIRead.com Your guide to all that's veg. My book blog ===================================================*/

Mon Jun 05 15:52:53 2006 stephangt4 [...] hotmail.com - Correspondence added

CC:	sgt19 [...] tid.es, stephangt4 [...] hotmail.com
Subject:	Re: [rt.cpan.org #19699] does not install on HP-UX 11.00.
Date:	Mon, 05 Jun 2006 19:52:21 +0000
To:	bug-Params-Validate [...] rt.cpan.org
From:	"stephan titard" <stephangt4 [...] hotmail.com>

Show quoted text

>From: "autarch@urth.org via RT" <bug-Params-Validate@rt.cpan.org> >Reply-To: bug-Params-Validate@rt.cpan.org >To: stephangt4@hotmail.com >Subject: Re: [rt.cpan.org #19699] does not install on HP-UX 11.00. >Date: Mon, 5 Jun 2006 14:50:43 -0400 (EDT) > > ><URL: http://rt.cpan.org/Ticket/Display.html?id=19699 > > >On Mon, 5 Jun 2006, stephangt4@hotmail.com via RT wrote: >

> > Hi, well to be honest a patch is not so easy (as I have to understand

>how

> > the INLINE C Macro is generated by MM) except if you get rid of the

>INLINE C

> > Macro; I can understand it for a C++ class > > but in C, what is the use?... it seems that any C compiler should

>complain

> > > > now it could be that MakeMaker generates something weird (at least on

>HP-UX)

> > I'll check that and let you know

> >I'm not sure what you're looking at. If you look at the Validate.xs file >you'll see it defines its own INLINE macro, which is either empty (is >_MSC_VER is defined) or the string "inline". I'm just asked you to patch >the file so that if the compiler is the HP-UX compiler, that macro is >empty.

ok. I´ll do that. Show quoted text

> > >-dave > >/*=================================================== >VegGuide.Org www.BookIRead.com >Your guide to all that's veg. My book blog >===================================================*/ >

Wed Jun 07 09:22:10 2006 sgt19 [...] tid.es - Ticket #19763: Ticket created

CC:	stephan titard <stephangt4 [...] hotmail.com>
Subject:	patch for 0.84 on HP-UX 11.00
Date:	Wed, 07 Jun 2006 15:21:22 +0200
To:	bug-Params-Validate [...] rt.cpan.org
From:	Stephan Titard <sgt19 [...] tid.es>

Hi Dave, this patch works fine to build on HP-UX 11.00 hth and thanks for your modules --steph p.s I don't mind beta-testing on HP-UX. I am sgt on perlmonks.

--- Validate.xs 2006-05-29 07:36:59 +0200 +++ Validate.xs.new 2006-06-06 12:55:51 +0200 @@ -12,6 +12,8 @@ #ifdef _MSC_VER #define INLINE +#elif __hpux +#define INLINE #else #define INLINE inline #endif

Tue Jun 13 15:34:17 2006 DROLSKY [...] cpan.org - Ticket #19763: Merged into ticket #19699

Tue Jun 13 15:34:17 2006 DROLSKY [...] cpan.org - Merged into ticket #19699

Tue Jun 13 15:34:37 2006 DROLSKY [...] cpan.org - Status changed from 'open' to 'resolved'

Tue Jun 13 15:34:38 2006 DROLSKY [...] cpan.org - Fixed in 0.85 added