Skip Menu |

This queue is for tickets about the Module-Install-XSUtil CPAN distribution.

Report information
The Basics
Id: 77780
Status: open
Priority: 0/
Queue: Module-Install-XSUtil
People
Owner: Nobody in particular
Requestors: djerius [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Tue Jun 12 11:16:28 2012 djerius [...] cpan.org - Ticket created
Subject: Module::Install 1.06 now exports requires_xs
Module::Install version 1.06 now has a requires_xs command which conflicts with that published by this module. It also causes the 01_example.t test to hang waiting for the user to disambiguate requires_xs.
  Tue Feb 12 10:36:46 2013 ppisar [...] redhat.com - Correspondence added
From: ppisar [...] redhat.com
Dne Út 12.čen.2012 11:16:28, DJERIUS napsal(a): Show quoted text
> Module::Install version 1.06 now has a requires_xs command which > conflicts with that published by this module. It also causes the > 01_example.t test to hang waiting for the user to disambiguate
requires_xs. 0.43 renamed requires_xs() to requires_ex_module(), but forgot to do the rename in example/MyMakefile.PL too. Now xt/04_example.t fails because compiler include path is not set to B::Hooks::OP::Annotation's headers. Also there is small glitch. If I call make test with TEST_FILES listing xt tests, then the variable will get propagated to example's make and test_harnes() dies because it cannot locate xt/04_example.t.
  Tue Feb 12 10:36:48 2013 The RT System itself - Status changed from 'new' to 'open'
  Tue Feb 12 10:49:44 2013 ppisar [...] redhat.com - Correspondence added
From: ppisar [...] redhat.com
Dne Út 12.úno.2013 10:36:46, ppisar napsal(a): Show quoted text
> 0.43 renamed requires_xs() to requires_ex_module(), but forgot to do the > rename in example/MyMakefile.PL too. Now xt/04_example.t fails because > compiler include path is not set to B::Hooks::OP::Annotation's headers. >
Attached patch fixes this issue. -- Petr
Subject: 0001-Fix-test-to-use-renamed-requires_xs.patch
From c11031cb15f1568e91a53b3b57a6752cc1e9f176 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Petr=20P=C3=ADsa=C5=99?= <ppisar@redhat.com> Date: Tue, 12 Feb 2013 16:44:15 +0100 Subject: [PATCH] Fix test to use renamed requires_xs() <https://rt.cpan.org/Public/Bug/Display.html?id=77780> --- example/MyMakefile.PL | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/example/MyMakefile.PL b/example/MyMakefile.PL index f6b1adc..e52644f 100644 --- a/example/MyMakefile.PL +++ b/example/MyMakefile.PL @@ -15,7 +15,7 @@ BEGIN{ all_from 'lib/Foo.pm'; -requires_xs 'B::Hooks::OP::Annotation'; # for testing only +requires_xs_module 'B::Hooks::OP::Annotation'; # for testing only use_ppport 3.19; use_xshelper -clean; -- 1.8.1.2