[Aptitude-devel] String freeze for aptitude?

[Jens Seidel]

On Sat, Apr 12, 2008 at 06:50:45PM +0200, Christian Perrier wrote:
> Quoting Daniel Burrows (dburrows at debian.org):
> >   I think that I've got all the changes into aptitude that I wanted to
> > get by Lenny.  I know the translators have some catching up to do, so
> > I'll try to restrain myself from fixing any more bugs or adding features

Translators work should never stop you from fixing bugs :-)

> > until that's caught up.  Hopefully we can get a .3 uploaded with
> > complete translations before the base freeze.
> 
> Sounds very fair. Thanks for the proposal (I usually have to propose
> this myself so that's much appreciated).
> 
> I think that Jens can even send a call for translations updates and
> new translation for the very courageous translators (either those who
> didn't follow the development and now have dozens of strings to
> update....or new ones

I still need some translators comments. I marked such strings in the
attached patch (not useful for you as it contains mainly TODO or FIXME
items). An excerpt:

diff -r 8066f9615345 src/cmdline/cmdline_why.cc
--- a/src/cmdline/cmdline_why.cc	Sun Apr 13 18:17:07 2008 +0200
+++ b/src/cmdline/cmdline_why.cc	Sun Apr 13 18:20:08 2008 +0200
@@ -255,7 +255,7 @@ namespace aptitude
       switch(dep_level)
 	{
 	case DependsOnly:
-	  rval += W_("DependsOnly");
+	  rval += W_("DependsOnly"); // Missing translator comment!

You really have to explain this! Are you sure this should be
translateable? "DependsOnly" looks like a special keyword or action
but is no proper English! Is there a limitation for this string such as
"no spaces allowed"?

-      case InstallType:
+      case InstallType: // CHECKME: Is a space needed? (#453776)
 	return cw::fragf(_("Install(%s)"), mpkg.Name());

Clytie asked whether it should/could be "Install (%s)". Hard to say
without a comment.

   cw::menu_info(cw::menu_info::MENU_ITEM, N_("^News"), NULL,
-		N_("View the important changes made in each version of " PACKAGE),
+		N_("View the important changes made in each version of " PACKAGE), // TODO: use %s!

This string is really a problem for me. I honestly do not know how to
translate this sentence such that PACKAGE (== "aptitude") will be the last
word.

Jens