-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Travis CI testing #26
Conversation
This fixes things for old versions of msgmerge such as the one used on Travis CI (Ubuntu 14.04) Signed-off-by: Daniel Axtens <dja@axtens.net>
Signed-off-by: Daniel Axtens <dja@axtens.net>
@@ -57,7 +57,7 @@ USE_MSGCTXT = no | |||
# Useful options are in particular: | |||
# --previous to keep previous msgids of translated messages, | |||
# --quiet to reduce the verbosity. | |||
MSGMERGE_OPTIONS = --add-location=never | |||
MSGMERGE_OPTIONS = --no-location |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commit 55ffff9 did the inverse of this, IIRC it was because of a difference between older and newer versions of msgmerge. It could be that adding both may work, I'll have to check.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like the ability to take a parameter for add-location was only added in mid-2014, which would explain why it isn't in 14.04: https://lists.gnu.org/archive/html/bug-gettext/2014-03/msg00021.html
What is the difference between --add-location=never and --no-location? My reading of the man page was that they were identical...
Ignore the new commits, I was ... testing something ... |
Merged as 865097f manually, thanks! What's that about the date? 20 something? What? I can't hear you? |
Hi Sam,
This allows testing with Travis CI.
If you merge it, you will also want to turn on testing for this repository in Travis CI.
Regards,
Daniel
This change is![Reviewable](https://camo.githubusercontent.com/23b05f5fb48215c989e92cc44cf6512512d083132bd3daf689867c8d9d386888/68747470733a2f2f72657669657761626c652e696f2f7265766965775f627574746f6e2e737667)