[Commotion-admin] [luci-i18n-commotion] Clean up PO File Generator (#6)

areynold notifications at github.com
Tue Jan 7 19:40:16 UTC 2014


The original PO file generator was pretty messy. This is a first attempt to improve readability and efficiency.
There is still some duplicate code, most notably in the Git functions, and the string parsing isn't perfect.

To test, you will need to install all of the prerequisite perl modules, then run parse.pl and look the new PO files over for flaws.
You can merge this Pull Request by running:

  git pull https://github.com/opentechinstitute/luci-i18n-commotion PO-script-rewrite

Or you can view, comment on it, or merge it online at:

  https://github.com/opentechinstitute/luci-i18n-commotion/pull/6

-- Commit Summary --

  * rewrote repos, branches, sources as hash for more modular handling
  * Keep only unique strings from each file.
  * Removed duplicate date var from _Generate_PO_Header
  * Added better function separation
  * Deleted unused git upload code
  * Added copyright statement
  * Replaced uniq function with code from List::MoreUtils

-- File Changes --

    M scripts/parse.pl (650)

-- Patch Links --

https://github.com/opentechinstitute/luci-i18n-commotion/pull/6.patch
https://github.com/opentechinstitute/luci-i18n-commotion/pull/6.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/opentechinstitute/luci-i18n-commotion/pull/6
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.chambana.net/pipermail/commotion-admin/attachments/20140107/fed0b9dc/attachment.html>


More information about the Commotion-admin mailing list