Bug #72293 for XML-Parser: Devel::CheckLib bug causes XML::Parser test failures on NFS

Tue Nov 08 19:57:20 2011 martin [...] arp242.net - Ticket created

Subject:

[PATCH] Build fails on NFS

CheckLib (included in XML::Parser) fails when building on NFS on my FreeBSD system (See: http://www.freebsd.org/cgi/query-pr.cgi? pr=ports/155991) The problem is that it tries to use File::Temp::tempfile() to create a file within the source directory, which may cause problems. A patch which solves the issue is attached ... I'm not familiar with Perl, and couldn't find a "get_tmp_dir()"-like function with a quick search, but you should get the idea ;) I also reported this at Devel::CheckLib: https://rt.cpan.org/Public/Bug/Display.html?id=72291

Tue Nov 08 19:57:57 2011 martin [...] arp242.net - Correspondence added

From:

martin [...] arp242.net

Oops, I forgot the patch

Subject:

patch-inc-Devel-CheckLib.pm

--- inc/Devel/CheckLib.pm.orig 2011-05-25 05:39:14.000000000 +0200 +++ inc/Devel/CheckLib.pm 2011-11-09 01:10:42.000000000 +0100 @@ -222,7 +222,7 @@ for my $header (@headers) { push @use_headers, $header; my($ch, $cfile) = File::Temp::tempfile( - 'assertlibXXXXXXXX', SUFFIX => '.c' + 'assertlibXXXXXXXX', SUFFIX => '.c', DIR => '/tmp/' ); print $ch qq{#include <$_>\n} for @use_headers; print $ch qq{int main(void) { return 0; }\n}; @@ -262,7 +262,7 @@ # now do each library in turn with headers my($ch, $cfile) = File::Temp::tempfile( - 'assertlibXXXXXXXX', SUFFIX => '.c' + 'assertlibXXXXXXXX', SUFFIX => '.c', DIR => '/tmp/' ); print $ch qq{#include <$_>\n} foreach (@headers); print $ch "int main(void) { ".($args{function} || 'return 0;')." }\n";

Wed Nov 09 03:46:43 2011 TODDR [...] cpan.org - Correspondence added

I am happy to update XML::Parser's inc paths when upstream (Devel::CheckLib) fixes their problem. I'm very disinclined to mess with anything in inc by patching it directly, since the way that tree is generated is mostly automated. If you will ping this ticket when they update, I will be happy to do the legwork on the update and release. Thank you for reporting it.

Wed Nov 09 03:46:45 2011 The RT System itself - Status changed from 'new' to 'open'

Wed Nov 09 03:48:03 2011 TODDR [...] cpan.org - Subject changed from '[PATCH] Build fails on NFS' to 'Devel::CheckLib bug causes XML::Parser test failures on NFS'

Wed Nov 09 03:48:03 2011 TODDR [...] cpan.org - Status changed from 'open' to 'stalled'

Wed Nov 09 03:48:03 2011 TODDR [...] cpan.org - Broken in 2.41 added

Wed Mar 14 05:02:31 2012 CHORNY [...] cpan.org - Dependency on ticket #72291 added

Mon Sep 23 23:45:27 2019 TODDR [...] cpan.org - Correspondence added

Ticket migrated to github as https://github.com/toddr/XML-Parser/issues/76

Mon Sep 23 23:45:27 2019 The RT System itself - Status changed from 'stalled' to 'open'

Mon Sep 23 23:50:29 2019 TODDR [...] cpan.org - Dependency on ticket #72291 deleted

Mon Sep 23 23:50:38 2019 TODDR [...] cpan.org - Status changed from 'open' to 'resolved'