Unity Control Center

Merge lp://staging/~attente/unity-control-center/fix-ftbfs-inline-zoom-options into lp://staging/unity-control-center

  1. fix-ftbfs-inline-zoom-options
  2. Merge into trunk
Proposed by William Hua
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 12840
Merged at revision: 12842
Proposed branch: lp://staging/~attente/unity-control-center/fix-ftbfs-inline-zoom-options
Merge into: lp://staging/unity-control-center
Diff against target: 39 lines (+6/-6)
1 file modified
panels/universal-access/zoom-options.c (+6/-6)
To merge this branch: bzr merge lp://staging/~attente/unity-control-center/fix-ftbfs-inline-zoom-options
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
Review via email: mp+278655@code.staging.launchpad.net

Commit message

Fix FTBFS due to non-deterministic compilation of non-static inline functions.

Description of the change

Fix FTBFS due to non-deterministic compilation of non-static inline functions.

To post a comment you must log in.
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

I can't reproduce the FTBFS with the latest version from the archive on my xenial sbuild. How do you see it? I wonder if we shouldn't rather bundle that with another landing (I didn't see any silo shipping those, and so, no FTBFS there either).

review: Needs Information
Revision history for this message
William Hua (attente) wrote :

Just for reference, this is the comment from upstream that explains its irreproducibility: https://bugzilla.gnome.org/show_bug.cgi?id=723994#c0

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

As discussed, we will bundle this with next u-c-c upload

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