Merge lp://qastaging/~seb128/gnome-control-center-unity/gettext-and-intltool into lp://qastaging/gnome-control-center-unity

Proposed by Sebastien Bacher
Status: Merged
Approved by: Didier Roche-Tolomelli
Approved revision: 33
Merged at revision: 33
Proposed branch: lp://qastaging/~seb128/gnome-control-center-unity/gettext-and-intltool
Merge into: lp://qastaging/gnome-control-center-unity
Diff against target: 14 lines (+1/-3)
1 file modified
configure.ac (+1/-3)
To merge this branch: bzr merge lp://qastaging/~seb128/gnome-control-center-unity/gettext-and-intltool
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
Łukasz Zemczak Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+196545@code.qastaging.launchpad.net

Commit message

don't use gettext and intltool together, they are conflicting

Description of the change

don't use gettext and intltool together, they are conflicting

that's similar to e.g https://bugzilla.gnome.org/show_bug.cgi?id=679607

(without that building from the vcs was failing for me on trusty)

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Looks ok, CI says it's ok as well so +1!

review: Approve
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Sounds good to me as per linked bug :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: