blead h2xs.t failure on HP-UX 11.23 IA-64 #11747
Comments
From @icerider70Created by philip.monsen@pobox.comh2xs.t failure observed during "make test". Details follow. $ ./perl harness -v ../lib/h2xs.t # Failed test 'running ./perl "-I../lib" "-I../../lib" ../utils/h2xs -f Test Summary Report Perl Info |
From @icerider70The attached patch to utils/h2xs.PL moves the failing h2xs.t test to --Phil |
From @icerider700001-Fixed-uninit-val-warning-to-cure-h2xs.t-test-failure.patchFrom 193d369f46c0649d62f4fb88de854959d336815d Mon Sep 17 00:00:00 2001
From: Philip Monsen <philip.monsen@pobox.com>
Date: Mon, 14 Nov 2011 09:14:46 -0600
Subject: [PATCH] Fixed uninit val warning to cure h2xs.t test failure
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="------------1.7.7.3"
This is a multi-part message in MIME format.
--------------1.7.7.3
Content-Type: text/plain; charset=UTF-8; format=fixed
Content-Transfer-Encoding: 8bit
---
utils/h2xs.PL | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
--------------1.7.7.3
Content-Type: text/x-patch; name="0001-Fixed-uninit-val-warning-to-cure-h2xs.t-test-failure.patch"
Content-Transfer-Encoding: 8bit
Content-Disposition: attachment; filename="0001-Fixed-uninit-val-warning-to-cure-h2xs.t-test-failure.patch"
diff --git a/utils/h2xs.PL b/utils/h2xs.PL
index 634e891..cb38a4b 100644
--- a/utils/h2xs.PL
+++ b/utils/h2xs.PL
@@ -789,7 +789,8 @@ if( @path_h ){
}
else {
@paths = (File::Spec->curdir(), $Config{usrinc},
- (split ' ', $Config{locincpth}), '/usr/include');
+ (split ' ', defined($Config{locincpth}) ? $Config{locincpth} : ''),
+ '/usr/include');
}
foreach my $path_h (@path_h) {
$name ||= $path_h;
--------------1.7.7.3--
|
From @TuxOn Mon, 14 Nov 2011 09:22:04 -0600, Philip Monsen
Why does it not fail on my HP-UX 11.23 IA-64? We're rather modern perl by now ... And I'm somewhat icky about split ' '. Shouldn't that be split / / ? --- a/utils/h2xs.PL -- |
|
The RT System itself - Status changed from 'new' to 'open' |
From @icerider70On Mon, Nov 14, 2011 at 10:32 AM, H.Merijn Brand <h.m.brand@xs4all.nl>wrote:
Probably because your $Config{locincpth} is set to something and mine
I don't have strong feelings about the exact syntax used for the patch. --Phil |
From @icerider70On Mon, Nov 14, 2011 at 11:09 AM, Philip Monsen <philip.monsen@gmail.com>wrote:
Thusly, I think the name of the bug should be changed more generally to --Phil |
From @icerider70Looks like this was resolved with commit This ticket can be closed. |
|
@dcollinsn - Status changed from 'open' to 'resolved' |
From @dcollinsnOn Thu Nov 17 10:16:11 2011, icerider wrote:
Resolving per Phil. -- |
From [Unknown Contact. See original ticket]On Thu Nov 17 10:16:11 2011, icerider wrote:
Resolving per Phil. -- |
Migrated from rt.perl.org#103566 (status was 'resolved')
Searchable as RT103566$
The text was updated successfully, but these errors were encountered: