← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~cjwatson/launchpad/multiarch-translations into lp:launchpad/db-devel

 

comments.sql change done.  Per a conversation with William, I renamed
the column to include_long_descriptions (as I'd got the sense of the
underlying apt.conf variable wrong, and wanted to keep the boolean
values the same way round), and adjusted the description accordingly; I
hope that doesn't invalidate the DB review.

For William: I checked with Michael Vogt, and apparently apt will fetch
Translation-en files in the LANG=C case, so I don't think we need to
worry about Translation-C files.

I also noticed that apt-ftparchive won't create the i18n directories
itself, so I added a conditional safe_mkdir call to
generateConfigForPocket.

I've asked Michael to prepare an apt backport with sufficient
LongDescription support, as lucid-updates doesn't have everything we
need.

-- 
https://code.launchpad.net/~cjwatson/launchpad/multiarch-translations/+merge/67640
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~cjwatson/launchpad/multiarch-translations into lp:launchpad/db-devel.


References