Bug 57642

Summary: Incorrect format in the file header
Product: kbabel Reporter: Antoni Bella <vasten>
Component: generalAssignee: Stanislav Visnovsky <visnovsky>
Severity: wishlist    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:

Description Antoni Bella 2003-04-24 13:04:27 UTC
Version:           1.0.1 (using KDE 3.1.1)
Installed from:     (testing/unstable)
Compiler:          gcc version 3.2.3 20030309 (Debian prerelease)
OS:          Linux (i686) release 2.4.20

Following detected problems are based in my experience on the Translation Project and you ROBOT, which is used to verify the PO files.

  * The action to add the years in the lines of the Copyright and translator:

	- (**) When another year is added this debit to be separated by a
	comma and a space - in this case lacks the space.

	- In the line of the translator a point would have appear at the end.

# Translator name <email_adress>, 2002,2003
# Translator name <email_adress>, 2002, 2003.

  * The Project-Id-Version field:

	- (*) If the name of the file is util-linux-2.11z.ca.po the program 
	should eliminate the code of the language and to leave a space 
	between the name of the program and its version. 

"Project-Id-Version: util-linux-2.11z.ca\n"
"Project-Id-Version: util-linux 2.11z\n"

(*) Mr ROBOT does not like.
(**) Mr ROBOT does not like and reject. 

  Well, all that is what I have observed up to the moment and that can improve kbabel.

Comment 1 Stanislav Visnovsky 2003-04-24 14:33:54 UTC
OK, will try to improve the macros there.
Comment 2 Stanislav Visnovsky 2003-05-26 18:11:38 UTC
Subject: kdesdk/kbabel/common

CVS commit by nanulo: 

Strip language code from PO filename to setup project/package name.

CCMAIL: 57642-done@bugs.kde.org

  M +5 -4      catalog.cpp   1.117

--- kdesdk/kbabel/common/catalog.cpp  #1.116:1.117
@@ -658,5 +658,5 @@ QString Catalog::packageName() const
     QString package=d->_url.fileName();
-    int index=package.findRev(QRegExp("\\.pot?"));
+    int index=package.find(QRegExp("(\\."+d->_identitySettings.languageCode+")?\\.pot?$"));
@@ -681,5 +681,4 @@ void Catalog::setPackage(const QString& 
         d->_packageName = package.right(package.length()-pos);
         d->_packageName.replace( QRegExp("^/+"),"");
-        kdDebug(KBABEL) << "package dir " << d->_packageDir << " and package name " << d->_packageName << endl; 
@@ -2997,5 +2996,7 @@ Catalog::DiffResult Catalog::diff(uint e
                 disconnect( this, SIGNAL( signalStopActivity() ), this, SLOT( stopInternal() ));
-                kdDebug(KBABEL) << "diffv not active" << endl;
+                kdDebug
+                (KBABEL) << "diffv not active" << endl;