You are viewing a plain text version of this content. The canonical link for it is here.
Posted to test-dev@httpd.apache.org by Stas Bekman <st...@stason.org> on 2003/07/10 06:14:48 UTC

Re: Makefile.PL Questions

David Wheeler wrote:
> On Monday, June 30, 2003, at 12:11  PM, David Wheeler wrote:
> 
>> BEGIN{
>>     if (eval {require Apache::Test}) {
>>         Apache::Test->import(qw(have_lwp plan));
>>         require Apache::TestRequest;
>>         Apache::TestRequest->import(qw(GET POST));
>>         plan tests => 43, have_lwp;
>>     } else {
>>         plan skip_all => 'Apache::Test required to run tests.';
>>     }
>> }
> 
> 
> Actually, to make it cooperate with Test::More, I had to do this:
> 
> BEGIN {
>     if (eval {require Apache::Test}) {
>         if (Apache::Test::have_lwp()) {
>             require Apache::TestRequest;
>             Apache::TestRequest->import(qw(GET POST));
>             plan tests => 43;
>         } else {
>             plan skip_all => 'libwww-perl is not installed';
>         }
>     } else {
>         plan skip_all => 'Apache::Test required to run tests';
>     }
> }
>

David, also please check the latest Apache::Peek on CPAN, I have started 
abstracting this functionality especially for complex builds which has to 
support both mod_perl versions. I haven't finished yet, but since pretty much 
all Apache:: module authors will need this code, rather than duplicating it we 
better have a special module (which can be at least bundled to start with).

Another approach is to use Module::Install, which I haven't tried yet, but 
which is supposed to resolve this kind of a problem generically.

__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com


Re: Makefile.PL Questions

Posted by David Wheeler <da...@kineticode.com>.
On Saturday, July 19, 2003, at 01:33  PM, Stas Bekman wrote:

> Yes, please. I doubt anybody is going to need it anyways.

mercury% make test
/usr/local/bin/perl -Iblib/arch -Iblib/lib \
t/TEST -clean
*** setting ulimit to allow core files
ulimit -c unlimited; t/TEST -clean
APACHE_USER= APACHE_GROUP= APACHE_PORT= APACHE=  
APXS=/usr/local/apache/bin/apxs \
/usr/local/bin/perl -Iblib/arch -Iblib/lib \
t/TEST -verbose=0
*** setting ulimit to allow core files
ulimit -c unlimited; t/TEST -verbose=0
/usr/local/apache/bin/httpd -X -d  
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t -f  
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/conf/httpd.conf  
-DAPACHE1
using Apache/1.3.27

waiting for server to start: .
waiting for server to start: ok (waited 0 secs)
server localhost:8529 started
apachepeek/basic....# Failed test 1 in apachepeek/basic.t at line 13
# Failed test 2 in apachepeek/basic.t at line 15
apachepeek/basic....FAILED tests 1-2
         Failed 2/2 tests, 0.00% okay
Failed Test        Stat Wstat Total Fail  Failed  List of Failed
------------------------------------------------------------------------ 
-------
apachepeek/basic.t                2    2 100.00%  1-2
!!! error running tests (please examine t/logs/error_log)
make: *** [run_tests] Error 1
mercury% cat t/logs/error_log
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/lib
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/arch
/usr/local/bricolage/lib
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/response
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t
/usr/local/lib/perl5/5.8.0/darwin
/usr/local/lib/perl5/5.8.0
/usr/local/lib/perl5/site_perl/5.8.0/darwin
/usr/local/lib/perl5/site_perl/5.8.0
/usr/local/lib/perl5/site_perl
.
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/lib/perl at  
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/response/ 
TestApachePeek/basic.pm line 23.
dyld: /usr/local/apache/bin/httpd multiple definitions of symbol  
_DeadCode
/usr/local/lib/perl5/5.8.0/darwin/auto/Devel/Peek/Peek.bundle  
definition of _DeadCode
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/arch/auto/ 
Apache/Peek/Peek.bundle definition of _DeadCode
mercury%


-- 
David Wheeler                                     AIM: dwTheory
david@kineticode.com                              ICQ: 15726394
http://kineticode.com/                         Yahoo!: dew7e
                                                Jabber: Theory@jabber.org
Kineticode. Setting knowledge in motion.[sm]


Re: Makefile.PL Questions

Posted by Stas Bekman <st...@stason.org>.
David Wheeler wrote:
> On Saturday, July 19, 2003, at 01:09  AM, Stas Bekman wrote:
> 
>> ok, it's not about XS, but the C function, can you comment it 
>> out/remove as well from .xs file?
> 
> 
> Sure, please tell me what, exactly, to remove from Peek.xs. Would it be 
> the rather huge DeadCode() function (lines 127-237)?

Yes, please. I doubt anybody is going to need it anyways.



__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com


Re: Makefile.PL Questions

Posted by David Wheeler <da...@kineticode.com>.
On Saturday, July 19, 2003, at 01:09  AM, Stas Bekman wrote:

> ok, it's not about XS, but the C function, can you comment it 
> out/remove as well from .xs file?

Sure, please tell me what, exactly, to remove from Peek.xs. Would it be 
the rather huge DeadCode() function (lines 127-237)?

Thanks,

David

-- 
David Wheeler                                     AIM: dwTheory
david@kineticode.com                              ICQ: 15726394
http://kineticode.com/                         Yahoo!: dew7e
                                                Jabber: Theory@jabber.org
Kineticode. Setting knowledge in motion.[sm]


Re: Makefile.PL Questions

Posted by Stas Bekman <st...@stason.org>.
David Wheeler wrote:
> On Friday, July 18, 2003, at 12:00  PM, Stas Bekman wrote:
> 
>> So what t/logs/error_log has to say about it?
> 
> 
> mercury% cat t/logs/error_log
> [Wed Jul 16 09:00:56 2003] [debug] mod_rendezvous_apple.c(1012):  
> mod_rendezvous_apple: Module init count=1 pid=1746.
> [Wed Jul 16 09:00:57 2003] [debug] mod_rendezvous_apple.c(1012):  
> mod_rendezvous_apple: Module init count=2 pid=1746.
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/lib
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/arch
> /usr/local/bricolage/lib
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/response
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t
> /System/Library/Perl/darwin
> /System/Library/Perl
> /Library/Perl/darwin
> /Library/Perl
> /Network/Library/Perl/darwin
> /Network/Library/Perl
> .
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/lib/perl at  
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/response/ 
> TestApachePeek/basic.pm line 23.
> dyld: /usr/sbin/httpd multiple definitions of symbol _DeadCode
> /System/Library/Perl/darwin/auto/Devel/Peek/Peek.bundle definition of  
> _DeadCode
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/arch/auto/ 
> Apache/Peek/Peek.bundle definition of _DeadCode

ok, it's not about XS, but the C function, can you comment it out/remove as 
well from .xs file?



__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com


Re: Makefile.PL Questions

Posted by David Wheeler <da...@kineticode.com>.
On Friday, July 18, 2003, at 12:00  PM, Stas Bekman wrote:

> So what t/logs/error_log has to say about it?

mercury% cat t/logs/error_log
[Wed Jul 16 09:00:56 2003] [debug] mod_rendezvous_apple.c(1012):  
mod_rendezvous_apple: Module init count=1 pid=1746.
[Wed Jul 16 09:00:57 2003] [debug] mod_rendezvous_apple.c(1012):  
mod_rendezvous_apple: Module init count=2 pid=1746.
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/lib
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/arch
/usr/local/bricolage/lib
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/response
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t
/System/Library/Perl/darwin
/System/Library/Perl
/Library/Perl/darwin
/Library/Perl
/Network/Library/Perl/darwin
/Network/Library/Perl
.
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/lib/perl at  
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/response/ 
TestApachePeek/basic.pm line 23.
dyld: /usr/sbin/httpd multiple definitions of symbol _DeadCode
/System/Library/Perl/darwin/auto/Devel/Peek/Peek.bundle definition of  
_DeadCode
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/arch/auto/ 
Apache/Peek/Peek.bundle definition of _DeadCode

-- 
David Wheeler                                     AIM: dwTheory
david@kineticode.com                              ICQ: 15726394
http://kineticode.com/                         Yahoo!: dew7e
                                                Jabber: Theory@jabber.org
Kineticode. Setting knowledge in motion.[sm]


Re: Makefile.PL Questions

Posted by Stas Bekman <st...@stason.org>.
David Wheeler wrote:
> On Tuesday, July 15, 2003, at 01:15  PM, Stas Bekman wrote:
> 
>> Wholy OS X, why would it fail if it resides in a diffent namespace :(
>>
>> can you try nuking this code:
>>
>> SV *
>> DeadCode()
>> CODE:
>>     RETVAL = DeadCode(aTHX);
>> OUTPUT:
>>     RETVAL
>>
>> from Peek.xs.mp1.perl5.8 and then rerun 'perl Makefile.PL'.
> 
> 
> Better. Now the test just fails:
> 
> mercury% make test TEST_VERBOSE=1
> /usr/local/bin/perl -Iblib/arch -Iblib/lib \
> t/TEST -clean
> *** setting ulimit to allow core files
> ulimit -c unlimited; t/TEST -clean
> APACHE_USER= APACHE_GROUP= APACHE_PORT= APACHE= APXS= \
> /usr/local/bin/perl -Iblib/arch -Iblib/lib \
> t/TEST -verbose=1
> *** setting ulimit to allow core files
> ulimit -c unlimited; t/TEST -verbose=1
> /usr/sbin/httpd -X -d  
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t -f  
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/conf/httpd.conf  
> -DAPACHE1
> using Apache/1.3.27
> 
> waiting for server to start: ....
> waiting for server to start: ok (waited 3 secs)
> server localhost:8529 started
> apachepeek/basic....1..2
> # Failed test 1 in apachepeek/basic.t at line 13
> # Failed test 2 in apachepeek/basic.t at line 15
> not ok 1
> # expected: (?-xism:Apache::Const.*?OK)
> # received:
> not ok 2
> FAILED tests 1-2
>         Failed 2/2 tests, 0.00% okay
> Failed Test        Stat Wstat Total Fail  Failed  List of Failed
> ------------------------------------------------------------------------ 
> -------
> apachepeek/basic.t                2    2 100.00%  1-2
> !!! error running tests (please examine t/logs/error_log)
> make: *** [run_tests] Error 1

So what t/logs/error_log has to say about it?


__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com


Re: Makefile.PL Questions

Posted by David Wheeler <da...@kineticode.com>.
On Tuesday, July 15, 2003, at 01:15  PM, Stas Bekman wrote:

> Wholy OS X, why would it fail if it resides in a diffent namespace :(
>
> can you try nuking this code:
>
> SV *
> DeadCode()
> CODE:
>     RETVAL = DeadCode(aTHX);
> OUTPUT:
>     RETVAL
>
> from Peek.xs.mp1.perl5.8 and then rerun 'perl Makefile.PL'.

Better. Now the test just fails:

mercury% make test TEST_VERBOSE=1
/usr/local/bin/perl -Iblib/arch -Iblib/lib \
t/TEST -clean
*** setting ulimit to allow core files
ulimit -c unlimited; t/TEST -clean
APACHE_USER= APACHE_GROUP= APACHE_PORT= APACHE= APXS= \
/usr/local/bin/perl -Iblib/arch -Iblib/lib \
t/TEST -verbose=1
*** setting ulimit to allow core files
ulimit -c unlimited; t/TEST -verbose=1
/usr/sbin/httpd -X -d  
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t -f  
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/conf/httpd.conf  
-DAPACHE1
using Apache/1.3.27

waiting for server to start: ....
waiting for server to start: ok (waited 3 secs)
server localhost:8529 started
apachepeek/basic....1..2
# Failed test 1 in apachepeek/basic.t at line 13
# Failed test 2 in apachepeek/basic.t at line 15
not ok 1
# expected: (?-xism:Apache::Const.*?OK)
# received:
not ok 2
FAILED tests 1-2
         Failed 2/2 tests, 0.00% okay
Failed Test        Stat Wstat Total Fail  Failed  List of Failed
------------------------------------------------------------------------ 
-------
apachepeek/basic.t                2    2 100.00%  1-2
!!! error running tests (please examine t/logs/error_log)
make: *** [run_tests] Error 1

David

-- 
David Wheeler                                     AIM: dwTheory
david@kineticode.com                              ICQ: 15726394
http://kineticode.com/                         Yahoo!: dew7e
                                                Jabber: Theory@jabber.org
Kineticode. Setting knowledge in motion.[sm]


Re: Makefile.PL Questions

Posted by Stas Bekman <st...@stason.org>.
David Wheeler wrote:
> On Wednesday, July 9, 2003, at 09:14  PM, Stas Bekman wrote:
> 
>> David, also please check the latest Apache::Peek on CPAN, I have  
>> started abstracting this functionality especially for complex builds  
>> which has to support both mod_perl versions. I haven't finished yet,  
>> but since pretty much all Apache:: module authors will need this 
>> code,  rather than duplicating it we better have a special module 
>> (which can  be at least bundled to start with).
> 
> 
> Just tried to install it, and `make test` failed. Here's the contents  
> of t/logs/error_log:
> 
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/lib
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/arch
> /usr/local/bricolage/lib
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/response
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t
> /usr/local/lib/perl5/5.8.0/darwin
> /usr/local/lib/perl5/5.8.0
> /usr/local/lib/perl5/site_perl/5.8.0/darwin
> /usr/local/lib/perl5/site_perl/5.8.0
> /usr/local/lib/perl5/site_perl
> .
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/lib/perl at  
> /Users/david/.cpanpl
> us/5.8.0/build/Apache-Peek-1.01/t/response/TestApachePeek/basic.pm line  
> 23.
> dyld: /usr/local/apache/bin/httpd multiple definitions of symbol  _DeadCode
> /usr/local/lib/perl5/5.8.0/darwin/auto/Devel/Peek/Peek.bundle  
> definition of _DeadCode
> /Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/arch/auto/ 
> Apache/Peek/Peek.bu
> ndle definition of _DeadCode

Wholy OS X, why would it fail if it resides in a diffent namespace :(

can you try nuking this code:

SV *
DeadCode()
CODE:
     RETVAL = DeadCode(aTHX);
OUTPUT:
     RETVAL

from Peek.xs.mp1.perl5.8 and then rerun 'perl Makefile.PL'.



__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com


Re: Makefile.PL Questions

Posted by David Wheeler <da...@kineticode.com>.
On Wednesday, July 9, 2003, at 09:14  PM, Stas Bekman wrote:

> David, also please check the latest Apache::Peek on CPAN, I have  
> started abstracting this functionality especially for complex builds  
> which has to support both mod_perl versions. I haven't finished yet,  
> but since pretty much all Apache:: module authors will need this code,  
> rather than duplicating it we better have a special module (which can  
> be at least bundled to start with).

Just tried to install it, and `make test` failed. Here's the contents  
of t/logs/error_log:

/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/lib
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/arch
/usr/local/bricolage/lib
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/response
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t
/usr/local/lib/perl5/5.8.0/darwin
/usr/local/lib/perl5/5.8.0
/usr/local/lib/perl5/site_perl/5.8.0/darwin
/usr/local/lib/perl5/site_perl/5.8.0
/usr/local/lib/perl5/site_perl
.
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/t/lib/perl at  
/Users/david/.cpanpl
us/5.8.0/build/Apache-Peek-1.01/t/response/TestApachePeek/basic.pm line  
23.
dyld: /usr/local/apache/bin/httpd multiple definitions of symbol  
_DeadCode
/usr/local/lib/perl5/5.8.0/darwin/auto/Devel/Peek/Peek.bundle  
definition of _DeadCode
/Users/david/.cpanplus/5.8.0/build/Apache-Peek-1.01/blib/arch/auto/ 
Apache/Peek/Peek.bu
ndle definition of _DeadCode

> Another approach is to use Module::Install, which I haven't tried yet,  
> but which is supposed to resolve this kind of a problem generically.

Yeah, it's on my list.

David

-- 
David Wheeler                                     AIM: dwTheory
david@kineticode.com                              ICQ: 15726394
http://kineticode.com/                         Yahoo!: dew7e
                                                Jabber: Theory@jabber.org
Kineticode. Setting knowledge in motion.[sm]