Mailing-List: contact dev-help@perl.apache.org; run by ezmlm
Precedence: bulk
Received-SPF: pass (nike.apache.org: domain of steve.m.hay@googlemail.com
 designates 209.85.215.177 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAHrynWBv5vJsy8jNNay_2Op9CoG=wxEYf49xwK-9OucGMkTK4Q@mail.gmail.com>
References: 
 <CABiGYoW=chaHi-axbUV0DLGKMRNb1nbzmjjYc+jngc9hptA36w@mail.gmail.com>
	<CAHrynWA=72cbARxY+bdjgPWw39QA=gmi738HoiVVmZbp=VGDpA@mail.gmail.com>
	<CADED=K6mO7cs_cnigrCcsoN47mJ12B0NHmQKt08aWb=tFQQN5g@mail.gmail.com>
	<1B32FF956ABF414C9BCE5E487A1497E70DD73C05@ukmail02.planit.group>
	<CAHrynWAQETeYkoL4LKc4acJ1YcH2zKWUO-ptbVFJvjo15zQ6Kg@mail.gmail.com>
	<1B32FF956ABF414C9BCE5E487A1497E70DD74037@ukmail02.planit.group>
	<CAHrynWBv5vJsy8jNNay_2Op9CoG=wxEYf49xwK-9OucGMkTK4Q@mail.gmail.com>
Date: Thu, 9 Aug 2012 01:31:30 +0100
Message-ID: 
 <CADED=K665hp6KOx15uB3hj1xu=Zdb1qM=eAQSMTBetzh3xkTWQ@mail.gmail.com>
Subject: Re: [MP2] Nullav undeclared make error
From: Steve Hay <steve.m.hay@googlemail.com>
To: Fred Moyer <fred@redhotpenguin.com>
Cc: Steve Hay <Steve.Hay@verosoftware.com>, dxs@rentrak.com,
	mod_perl Dev <dev@perl.apache.org>
Content-Type: text/plain; charset=UTF-8

On 8 August 2012 21:50, Fred Moyer <fred@redhotpenguin.com> wrote:
> On Wed, Jul 18, 2012 at 12:20 AM, Steve Hay <Steve.Hay@verosoftware.com> wrote:
>> Fred Moyer wrote on 2012-07-18:
>>> What update needs to happen to Apache-Test? If you want to make the
>>> needed commit I can start the release process.
>>
>> All the necessary commits are in. Commit 1362414 updated Apache-Test, so
>> I'm guessing we need to roll Apache-Test-1.38, and then update mod_perl
>> to include that to get that commit into a mod_perl release?
>
> A-T 1.38 is out. Do we need any other changes for 2.0.8 for this
> issue? I may review the RT list to see if I can resolve a few more
> issues.

No other changes for this are required that I'm aware of. I think
we're all good to go on this issue.


>
>
>>
>>
>>>
>>>> This gets mod_perl building again with a perl that doesn't have large
>>>> files support.
>>>>
>>>> I still think we should also remove our usage of PERL_CORE, though...
>>>
>>> Does this affect the minimum version of Perl we can support? I
>>> remember from the last mp2 release that there was a file with
>>> httpd/perl dependencies in it that I had to update.
>>
>> It shouldn't affect it. I did briefly break the build for 5.10.1 by
>> having used a new symbol which older perls don't have, but the buildbot
>> told me about that and I fixed it in commit 1362409. I will test with
>> 5.8 to be sure when I get a chance, but generally the changes were to
>> use NULL (rather than Nullsv et al ) a lot, so won't have any issues for
>> older perls.
>>
>> You could start the ball rolling with a new Apache-Test release in the
>> meantime, though: that only contained one Nullsv->NULL change, so
>> definitely isn't a problem.
>>
>> Btw, did we get anywhere with granting PAUSE perms to other committers?
>>
>>
>>>
>>>
>>>>
>>>>
>>>>
>>>> From: Steve Hay [mailto:steve.m.hay@googlemail.com]
>>>> Sent: 13 July 2012 08:23
>>>> To: Fred Moyer
>>>> Cc: dxs@rentrak.com; modperl@perl.apache.org; mod_perl Dev
>>>> Subject: Re: [MP2] Nullav undeclared make error
>>>>
>>>> Various perl changes removed Nullsv, Nullav etc from the core (e.g.
>>> see 24792b8dab and 3ae1b22641), but left definitions of them for when
>>> PERL_CORE is not defined, for backwards compatibility with all those
>>> CPAN modules out there which use them.
>>>>
>>>> The problem here is what Nick hinted at in his comment for
>>>> 24792b8dab,
>>> namely that "obviously" nobody outside of the perl core is defining
>>> PERL_CORE... It's a rather too common and surely always wrong thing to
>>> do that, and we're guilty of it ourselves: modperl_perl_includes.h
>>> defines PERL_CORE as some kind of optimization, but only when
>>> USE_ITHREADS is defined and USE_LARGE_FILES is not. That's not a
>>> common configuration, hence we haven't seen this happen before, but
>>> the last line which I've quoted below does indeed undefine large file
>>> support, hence PERL_CORE gets defined and the definitions of Nullsv,
>>> Nullav etc are not provided.
>>>>
>>>> I replaced all uses of Nullxx with (XX*)NULL in my modules some time
>>> ago in the belief that if it was good for the core then it was good
>>> for me, so I will do likewise for mod_perl unless anyone objects (or
>>> beats me to it).
>>>>
>>
>>
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
> For additional commands, e-mail: dev-help@perl.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org