Merge branch 'master' into make-lxc3-optional

.github/stale.yml

@@ -0,0 +1,34 @@
+# Number of days of inactivity before an issue becomes stale
+daysUntilStale: 120
+# Number of days of inactivity before a stale issue is closed
+daysUntilClose: 7
+# Issues with these labels will never be considered stale
+exemptLabels:
+  - android
+  - app-support
+  - bug
+  - build
+  - container
+  - distro-support
+  - documentation
+  - duplicate
+  - enhancement
+  - help wanted
+  - kernel
+  - not-snap
+  - question
+  - session manager
+  - sigill
+  - startup
+  - tools
+  - undecided
+  - upstream-bug
+# Label to use when marking an issue as stale
+staleLabel: decaying
+# Comment to post when marking an issue as stale. Set to `false` to disable
+markComment: >
+  This issue has been automatically marked as stale because it has not had
+  recent activity. It will be closed if no further activity occurs. Thank you
+  for your contributions.
+# Comment to post when closing a stale issue. Set to `false` to disable
+closeComment: false

external/cpu_features/CMakeLists.txt

@@ -5,6 +5,10 @@ project(CpuFeatures VERSION 0.1.0)
 # ANBOX allow to build in our more strict build environment
 set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-error=switch-default -Wno-error=unused-parameter -Wno-error=overflow")
 
+if("${CMAKE_C_COMPILER_ID}" STREQUAL "Clang")
+  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-error=cast-align")
+endif()
+
 # Default Build Type to be Release
 if(NOT CMAKE_BUILD_TYPE)
   set(CMAKE_BUILD_TYPE "Release" CACHE STRING

src/anbox/cmds/check_features.cpp

@@ -44,7 +44,7 @@ anbox::cmds::CheckFeatures::CheckFeatures()
           cli::Name{"check-features"}, cli::Usage{"check-features"},
           cli::Description{"Check that the host system supports all necessary features"}} {
 
-  action([this](const cli::Command::Context&) {
+  action([](const cli::Command::Context&) {
 #if defined(CPU_FEATURES_ARCH_X86)
     const auto info = cpu_features::GetX86Info();
     std::vector<std::string> missing_features;

src/anbox/dbus/bus.cpp

@@ -20,9 +20,7 @@
 
 namespace anbox {
 namespace dbus {
-Bus::Bus(Type type) :
+Bus::Bus(Type type) {
-  type_{type} {
-
   int ret = 0;
   switch (type) {
   case Type::Session:

src/anbox/dbus/bus.h

@@ -48,7 +48,6 @@ class Bus : public DoNotCopyOrMove {
  private:
   void worker_main();
 
-  Type type_;
   sd_bus *bus_ = nullptr;
   std::thread worker_thread_;
   std::atomic_bool running_{false};

src/anbox/dbus/stub/application_manager.cpp

@@ -114,7 +114,12 @@ void ApplicationManager::launch(const android::Intent &intent,
   if (r < 0)
     throw std::runtime_error("Failed to construct DBus message");
 
+  #pragma GCC diagnostic push
+  #pragma GCC diagnostic warning "-Wpragmas"
+  #pragma GCC diagnostic warning "-Wc99-extensions"
   sd_bus_error error = SD_BUS_ERROR_NULL;
+  #pragma GCC diagnostic pop
+
   r = sd_bus_call(bus_->raw(), m, 0, &error, nullptr);
   if (r < 0) {
     const auto msg = utils::string_format("%s", error.message);

src/anbox/graphics/emugl/RenderApi.cpp

@@ -30,7 +30,6 @@ GLESv1Dispatch s_gles1;
 
 namespace {
 constexpr const char *default_egl_lib{"libEGL.so.1"};
-constexpr const char *default_glesv1_lib{"libGLESv1_CM.so.1"};
 constexpr const char *default_glesv2_lib{"libGLESv2.so.2"};
 }