Mir

/tmp/buildd/mir-0.1.1/tests/unit-tests/android_input/input_reader.cpp:758:36: note: 'args.android::NotifyConfigurationChangedArgs::eventTime' was declared here NotifyConfigurationChangedArgs args;

Bug #1248014 reported by Chris Gagnon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mir
Fix Released
Critical
Alan Griffiths
mir (Ubuntu)
Fix Released
Critical
Unassigned

Bug Description

Build fails due to warning on armhf

In file included from /tmp/buildd/mir-0.1.1/tests/unit-tests/android_input/input_reader.cpp:23:0:
/usr/src/gmock/gtest/include/gtest/gtest.h: In member function 'virtual void android::InputReaderTest_LoopOnce_WhenDeviceScanFinished_SendsConfigurationChanged_Test::TestBody()':
/usr/src/gmock/gtest/include/gtest/gtest.h:1499:30: error: 'args.android::NotifyConfigurationChangedArgs::eventTime' may be used uninitialized in this function [-Werror=maybe-uninitialized]
                        actual);
                              ^
/tmp/buildd/mir-0.1.1/tests/unit-tests/android_input/input_reader.cpp:758:36: note: 'args.android::NotifyConfigurationChangedArgs::eventTime' was declared here
     NotifyConfigurationChangedArgs args;
                                    ^
cc1plus: all warnings being treated as errors
make[3]: *** [tests/unit-tests/CMakeFiles/unit-tests.dir/android_input/input_reader.cpp.o] Error 1

https://jenkins.qa.ubuntu.com/job/mir-team-mir-development-branch-trusty-armhf-ci/62/console

Related branches

Changed in mir:
importance: Undecided → Critical
status: New → Triaged
milestone: none → 0.1.2
description: updated
Changed in mir:
assignee: nobody → Daniel van Vugt (vanvugt)
status: Triaged → In Progress
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

That's weird. I have no problem when cross-compiling armhf from trusty desktop. Will have to try native builds...

Revision history for this message
Daniel van Vugt (vanvugt) wrote :
Changed in mir:
status: In Progress → Incomplete
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

And to be sure, I've now built Mir on armhf natively. Still no bug.

Changed in mir:
status: Incomplete → Invalid
milestone: 0.1.2 → none
Revision history for this message
Daniel van Vugt (vanvugt) wrote :
Changed in mir:
assignee: Daniel van Vugt (vanvugt) → Alan Griffiths (alan-griffiths)
status: Invalid → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:~mir-team/mir/development-branch at revision None, scheduled for release in mir, milestone Unknown

Changed in mir:
status: In Progress → Fix Committed
Changed in mir:
milestone: none → 0.1.2
Changed in mir:
status: Fix Committed → Fix Released
Changed in mir (Ubuntu):
importance: Undecided → Critical
status: New → Triaged
Changed in mir (Ubuntu):
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (5.3 KiB)

This bug was fixed in the package mir - 0.1.2+14.04.20131128.1-0ubuntu1

---------------
mir (0.1.2+14.04.20131128.1-0ubuntu1) trusty; urgency=low

  [ Kevin Gunn ]
  * New upstream release 0.1.2
    - graphics: android: improve interface for mga::DisplayDevice so its
      just concerned with rendering and posting.
    - surfaces: rename "surfaces" component to "scene".
    - surfaces, shell: Migrate Session data model from shell to surfaces.
    - graphics: change fill_ipc_package() to use real pointers.
    - mir_client_library.h: Fix typo "do and locking" should be "do any
      locking".
    - API enumerations cleanup: Remove slightly misleading *_enum_max_
      values, and replace them with more accurate plural forms.
    - test_android_communication_package: Do not expect opened fd to be >0,
      we may have closed stdin making this a valid value (LP: #1247718).
    - Update docs about running Mir on the desktop to mention new package
      ubuntu-desktop-mir.
    - offscreen: Add a display that renders its output to offscreen buffers
    - graphics: android: fix regression for hwc1.0 devices introduced in r1228
      (LP: #1252433).
    - OffscreenPlatform provides the services that the offscreen display
      needs from the Platform.
    - graphics: android: consolidate the GLContexts classes in use.
    - Fix uninitialized variable causing random drm_auth_magic test
      failures. (LP: #1252144).
    - Add a fullyish functional Udev wrapper. This currently sits in
      graphics/gbm, but will be moved to the top-level when input device
      detection migrates.
    - Add resizing support to example code; demo-shell and clients.
    - eglapp: Clarify messages about pixel formats (LP: #1168304).
    - Adds support to the MirMotionEvent under pointer_coordinates called
      tool_type. This will allow clients to tell what type of tool is
      being used, from mouse/finger/etc. (LP: #1252498)
    - client,frontend: Report the real available surface pixel formats to
      clients. (LP: #1240833)
    - graphics: android: 1) change hwc1.1 to make use of sync fences during
      the compositor's gl renderloop. Note that we no longer wait for the
      render to complete, we pass this responsibility to the driver and the
      kernel. 2) support nexus 10. (LP: #1252173) (LP: #1203268)
    - shell: don't publish SurfacesContainer - it can be private to shell.
    - gbm: Don't mess up the VT mode on setup failure Only restore the
      previous VT mode during shutdown if it was VT_AUTO.
    - Fix a crash due to a failed eglMakeCurrent() call when in nested mode.
    - shell: unity-mir uses shell::FocusSetter - make the header public again
    - Add resize support to client surfaces (mir::client::MirSurface).
    - graphics: android: support 'old aka 2012' nexus 7 hwc (nvidia tegra3
      SoC) better. (LP: #1231917)
    - Add resize support to *ClientBuffer classes. Now always get dimensions
      from the latest buffer package.
    - android: support driver hooks for the Mali T604 (present in nexus 10)
    - Add width and height to the protocol Buffer messages, in preparation
      for resizable surfaces.
    - surfaces, shell, logging, te...

Read more...

Changed in mir (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.