Bug #43479 for Config-IniHash: 3.00.05 fails to build from the source

Fri Feb 20 14:14:34 2009 antonio [...] dyne.org - Ticket created

Subject:

3.00.05 fails to build from the source

Hi, I'm working on upgrading the version of your package into the debian archive. What I see is that while running test.pl it fails 2 tests of 43 run, it dies just after the 44: ok 42 - heredocs are not allowed by default ok 43 - short value is read fine Heredoc value for [Six]long not closed at end of file! at test.pl line 159 # Looks like you planned 208 tests but only ran 43. # Looks like you failed 2 tests of 43 run. # Looks like your test died just after 43. If I remove test 44, it still fails some other tests, is there anything I should to to prevent these tests from failing? Unless test.pl returns 0 (i.e.: no test fails), I cannot build the package. Cheers Antonio

Sat May 23 09:58:19 2009 RyanRyan52 [...] gmail.com - Correspondence added

From:

RyanRyan52 [...] gmail.com

Hi Jenda, attached are the two patches needed to fix the tests on debian gnu/linux. the first one, fix-tests, is needed because the TEMP variable needs to be capitalized, like it is in test.ini. the second one, remove-carriage-returns, is needed because non windows platforms don't use \r\n for line breaks...they just use \n, and your test.ini file has the "\r"s. Thanks, Ryan On Fri Feb 20 14:14:34 2009, antonio wrote: Show quoted text

> Hi, > I'm working on upgrading the version of your package into the debian > archive. > > What I see is that while running test.pl it fails 2 tests of 43 run, it > dies just after the 44: > > ok 42 - heredocs are not allowed by default > ok 43 - short value is read fine > Heredoc value for [Six]long not closed at end of file! at test.pl line 159 > # Looks like you planned 208 tests but only ran 43. > # Looks like you failed 2 tests of 43 run. > # Looks like your test died just after 43. > > If I remove test 44, it still fails some other tests, is there anything > I should to to prevent these tests from failing? Unless test.pl returns > 0 (i.e.: no test fails), I cannot build the package. > > Cheers > Antonio

Download fix-tests
application/octet-stream 775b

Message body not shown because it is not plain text.

Download remove-carriage-returns
application/octet-stream 1k

Message body not shown because it is not plain text.

Sat May 23 09:58:20 2009 The RT System itself - Status changed from 'new' to 'open'

Sat May 30 17:17:05 2009 JENDA [...] cpan.org - Correspondence added

Thanks. I fixed some of the issues in a little different way, but thanks for your patches anyway :-) Jenda

Sat May 30 17:17:07 2009 JENDA [...] cpan.org - Status changed from 'open' to 'resolved'