Merge lp:~mfisch/ubuntu/precise/live-manual/live-manual-fix-ftbfs-831392 into lp:ubuntu/precise/live-manual

[Matt Fischer]

I have tracked down the FTBFS issue, it is because that inside the chroot the environment (specifically LC_ALL is set to "C") and so when it tries to process the manuals for the other languages, like de, it bails when it sees "invalid byte sequence in US-ASCII". Then later on it really fails when it can't find the localized manuals to complete the processing.

What the fix is is that you need to install the us locale in your chroot and setting LC_ALL before the build. I think the key here is that the environment is <something>.UTF-8, but there's no reason not to use en_US, the other manuals are not broken by this process from what I see.

This failure also explains to me why it builds fine outside a chroot on my dev box or in a regular VM.

[Stefano Rivera (stefanor)]

This issue isn't specific to Ubuntu, so it would be nice to have a generic solution that would work for Debian too.

Debian doesn't have langpacks. It does have locales-all, but that's huge, and takes forever to install. So, the best solution is usually to generate a locale during install (python-djvulibre is an example of a package doing something like this).

[Matt Fischer (mfisch)]

From what I can tell it builds fine for debian since they have binary packages for it. I assumed that their build systems already had locales installed. Can you confirm that this is also broken for them?

[Matt Fischer (mfisch)]

Okay, so I see what python-djvulibre does (another package on the FTBFS list btw), I can try that method and remove the new dependency. I'd still like to know how this builds for debian and not for us or perhaps I'm mistaken.

[Matt Fischer (mfisch)]

Stefano,

I'm able to get it to build by doing this in debian/rules:

%:
        mkdir -p debian/tmp/locale/
        localedef -f UTF-8 -i en_US ./debian/tmp/locale/en_US.UTF-8/
        export LOCPATH=$(CURDIR)/debian/tmp/locale/ && \
        export LC_ALL=en_US.UTF-8 && \
        dh ${@}

Doing those steps in override_dh_builddeb didn't do it.

I also removed the changes to the control file. If this is what you're looking for, I'll push a new version. Let me know.

[Timo Aaltonen (tjaalton)]

review: Approve

=== modified file 'debian/changelog'
--- debian/changelog	2011-12-04 10:25:10 +0000
+++ debian/changelog	2012-03-04 18:07:20 +0000
@@ -1,3 +1,10 @@
+live-manual (1:3.0~a9-1ubuntu1) precise; urgency=low
+
+  * fix FTBFS by adding a dependency on language-pack-en and setting the 
+    LC_ALL to en_US.UTF-8 (LP: #831392)
+
+ -- Matthew Fischer <[e-mail]>  Sat, 03 Mar 2012 21:36:04 -0700
+
 live-manual (1:3.0~a9-1) unstable; urgency=low
 
   [ chals ]

=== modified file 'debian/control'
--- debian/control	2011-12-04 10:25:10 +0000
+++ debian/control	2012-03-04 18:07:20 +0000
@@ -1,12 +1,13 @@
 Source: live-manual
 Section: doc
 Priority: optional
-Maintainer: Debian Live Project <[e-mail]>
+Maintainer: Ubuntu Developers <[e-mail]>
+XSBC-Original-Maintainer: Debian Live Project <[e-mail]>
 Uploaders:
  Ben Armstrong <[e-mail]>,
  Daniel Baumann <[e-mail]>,
  Chris Lamb <[e-mail]>
-Build-Depends: debhelper (>= 8), ruby, libnokogiri-ruby
+Build-Depends: debhelper (>= 8), ruby, libnokogiri-ruby, language-pack-en
 Build-Depends-Indep: sisu-complete (>= 3), po4a
 Standards-Version: 3.9.2
 Homepage: http://live.debian.net/devel/live-manual/

=== modified file 'debian/rules'
--- debian/rules	2011-12-04 10:25:10 +0000
+++ debian/rules	2012-03-04 18:07:20 +0000
@@ -1,7 +1,7 @@
 #!/usr/bin/make -f
 
 %:
-	dh ${@}
+	LC_ALL=en_US.UTF-8 dh ${@}
 
 override_dh_builddeb:
 	dh_builddeb -- -Zgzip -z9