[Kimchi-devel] [PATCH V2] add a make check-local command to verify the i18n string format

shaohef at linux.vnet.ibm.com shaohef at linux.vnet.ibm.com
Wed Apr 2 05:51:42 UTC 2014


From: ShaoHe Feng <shaohef at linux.vnet.ibm.com>

When I do i18n translation, I find some string format are wrong.
So I add a check-local command to help the developer to check their
string format.

Every developers please run this command before submit your patch.
Thanks.

After you run this command, it may report some invalid string formats.
$ make check-local
Checking for invalid string formatting...
    "KCHREPOS0018E": _("Could not write repository configuration file
%(repo_file)"),

You should check %(repo_file) is what you want.

Ref:
http://docs.python.org/2/library/string.html

Signed-off-by: ShaoHe Feng <shaohef at linux.vnet.ibm.com>
Signed-off-by: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
Signed-off-by: Christy Perez <christy at linux.vnet.ibm.com>
---
 Makefile.am | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/Makefile.am b/Makefile.am
index c68f050..75d457e 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -73,7 +73,15 @@ PEP8_WHITELIST = \
 
 SKIP_PYFLAKES_ERR = "\./src/kimchi/websocket\.py"
 
+I18N_FILES = plugins/*/i18n.py \
+	src/kimchi/i18n.py \
+	$(NULL)
+
 check-local:
+	@echo "Checking for invalid string formatting..."
+	@grep -P "%\([^\)]*?\)[^0-9\.bcdeEfgGnosxX%]" $(I18N_FILES) | \
+		while read LINE; do echo "$$LINE"; false; done
+	@echo "Checking for invalid string formatting successfully"
 	find . -path './.git' -prune -type f -o \
 		-name '*.py' -o -name '*.py.in'  | xargs $(PYFLAKES) | \
 		grep -w -v $(SKIP_PYFLAKES_ERR) | \
-- 
1.8.5.3




More information about the Kimchi-devel mailing list