gerrit.automotivelinux Code Review - AGL/meta-agl-devel.git/blob

   1 From 36c7deb35719d2219dc6e868ab14d9a4ce984d1f Mon Sep 17 00:00:00 2001
   2 From: Marius Vlad <marius.vlad@collabora.com>
   3 Date: Mon, 25 Jul 2022 15:51:06 +0300
   4 Subject: [PATCH 2/3] shell/configuration: Obey json configuration file
   5
   6 Setting the name of the application to kApplicationName, by default,
   7 would cause the check in getCliOverrides to always be true, which in
   8 turn will ignore whatever the user specified in the configuration file.
   9
  10 Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
  11 ---
  12  shell/configuration/configuration.cc | 2 ++
  13  shell/main.cc                        | 2 +-
  14  2 files changed, 3 insertions(+), 1 deletion(-)
  15
  16 diff --git a/shell/configuration/configuration.cc b/shell/configuration/configuration.cc
  17 index f8b1e13..616de48 100644
  18 --- a/shell/configuration/configuration.cc
  19 +++ b/shell/configuration/configuration.cc
  20 @@ -220,6 +220,8 @@ std::vector<struct Configuration::Config> Configuration::ParseConfig(
  21      if (cfg.view.height == 0) {
  22        cfg.view.height = kDefaultViewHeight;
  23      }
  24 +    if (cfg.app_id.empty())
  25 +      cfg.app_id = kApplicationName;
  26
  27      res.emplace_back(cfg);
  28    }
  29 diff --git a/shell/main.cc b/shell/main.cc
  30 index 65c3f37..6fdd907 100644
  31 --- a/shell/main.cc
  32 +++ b/shell/main.cc
  33 @@ -44,7 +44,7 @@ void RemoveArgument(std::vector<std::string>& args, const std::string& arg) {
  34
  35  int main(int argc, char** argv) {
  36    struct Configuration::Config config {
  37 -    .app_id = kApplicationName, .json_configuration_path{}, .cursor_theme{},
  38 +    .app_id = {}, .json_configuration_path{}, .cursor_theme{},
  39      .disable_cursor{}, .debug_backend{}, .view {}
  40    };
  41
  42 --
  43 2.35.3
  44