tests failed when I built php-gettext in a clean chroot

Bug #552256 reported by Marcelo Jorge Vieira
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
php-gettext
Fix Released
High
Данило Шеган

Bug Description

php-gettext version: 1.0.10

phpunit --verbose tests
PHPUnit 3.4.11 by Sebastian Bergmann.

tests
 LocaleTest
 F.

 ParsingTest
 .

Time: 0 seconds, Memory: 7.00Mb

There was 1 failure:

1) LocaleTest::test_setlocale
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-sr_RS
+C

/root/php-gettext/tests/LocalesTest.php:11

FAILURES!
Tests: 3, Assertions: 13, Failures: 1.
make: *** [check] Error 1

To fix it, I've unset the LC_ALL variable. Attached, you will find a patch,
but I don't know if it is the better way to fix the bug...

Cheers,
Marcelo

Related branches

Revision history for this message
Marcelo Jorge Vieira (metal-alucinados) wrote :
Revision history for this message
Данило Шеган (danilo) wrote :

Thanks for the report: I'll just use LC_ALL directly because it takes precedence over everything else. I.e. LC_ALL=sr_RS.

Changed in php-gettext:
status: New → In Progress
importance: Undecided → High
assignee: nobody → Данило Шеган (danilo)
Revision history for this message
Данило Шеган (danilo) wrote :

Actually, fixed it your way because emulated version interprets only LANG variable. :) Fixed in lp:php-gettext r49.

Thanks again.

Changed in php-gettext:
status: In Progress → Fix Committed
Changed in php-gettext:
milestone: none → 1.0.11
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.