Jiang Xin <worldhello.net@xxxxxxxxx> writes: > But it is rare to maintain po/git.pot file for 'maint' branch. And if > I need, I will switch to a different version of gettext. Makefile > will throw a error message, if I use a wrong version of gettext. Is that the "v1" in the check in your Makefile patch is about? That is, when we need to change the underlying type, your updated gettext would say something different from "v1" and you will have Makefile update to expect that new version? Then it would be workable, I'd guess. >> Compared to that, Dscho's "hack" at least ties what PRItime is >> replaced with and what the source code says by being in the >> Makefile, which is tracked alongside the rest of the source. So I >> somehow feel that the approach has smaller chance of going wrong. > > Benefit of using the tweak version of gettext: > > 1. `make pot` can be run in a tar extract directory (without git controlled). > 2. do not need to run `git reset --hard`. > 3. it's quick (nobody cares). This is about a tool and workflow only you (and your successor, when the time comes) need to use, so ultimately I'd prefer to leave it up to you, but I'd want to make sure you are making your decision with sound rationale. I personally do not think 1. and 2. above are real issues in practice, because (1) you'd be committing the result of "make pot" so that translators can work off of it---which at least to me means that being able to run on a tarball extract is not a useful feature, and (2) when you are about to run xgettext to extract strings from the message, you do not want to have local modifications, extracting potentially modified strings with potentially offset line numbers in the resulting pot file, so having to run "git reset --hard" at the end is not a problem. I do not know if 3. is a practical issue or not (I did try "make pot" with the tip of 'master' tonight myself, and I didn't find it much slower than the unmodified one). Having said all that, again, I'd prefer to leave this up to you, so unless I hear that you changed your mind, the tip of 'master' I'll tag, probably sometime tomorrow my time, will have your patch to the Makefile to rely on your private edition of xgettext. Thanks.
