Convert App-templates submodule in a CMake module

Description

App-Templates is now a git submodule to be include in each project that want to take advantages of its mechanisms. It helps to create a widget compatible with the application framework and also can generate a test dedicated widget for your project. It also helps you to build no matter which environment you use, Yocto, SDK, Native in any linux distribution.

 

Handling git submodule became more and more confusing and hard to maintains. So it will be a great add to convert it to a CMake module.

Environment

None

Activity

Show:

Walt Miner 
December 7, 2018 at 11:53 PM

Close for GG RC2

Romain Forlot 
October 19, 2018 at 8:58 AM

  • Does your lava/fuego output is screwed-up ? I think that the ouptut without color is ok by now. I already see incompatible terminal without color only displaying the colorless messages

  • Why ? About now, I'm able to debug any software with those flags, but I'm ok if that is needed.

  • As already saw at AMM, this could be accomplished using CMake variable definitions at configuration time. Might the app-templates has to be modified to making sure that debug symbol and stuff are removed from the final binary when building a release or coverage build.

I think that both cmake module and submodule are quite equal about feature, as far as I see, now with the latest commit .

Jan-Simon Moeller 
October 17, 2018 at 9:14 AM

I reviewed the current cmake macros/variables in the app-templates and have a few comments:
https://gerrit.automotivelinux.org/gerrit/gitweb?p=apps/app-templates.git;a=blob;f=cmake/cmake.d/02-variables.cmake;h=693991a6b82ef404b0d4861fd990e84cc1af11a7;hb=HEAD#l28
  – with my CI hat on: We need also a colorless option so that it does not screw-up the (yes, dumb) serial terminal

https://gerrit.automotivelinux.org/gerrit/gitweb?p=apps/app-templates.git;a=blob;f=cmake/cmake.d/01-build_options.cmake;h=abd673466af952a9d60c4f35315dfbc643262339;hb=HEAD#l92

   – we might have to also define -O0

-  we need to be able to build all 4 wgt types - this might require multiple build passes / subfolders :

   – normal build / X.wgt

   –  debug build  /  X-debug.wgt

   –  coverage build / X-coverage.wgt

   –   test build  /  X-test.wgt

   – At best these all would be additive at install time . e.g. Normal + Debug (as distros do for debug rpm's) . We need to see about coverage (that might become exclusive at execution time). The -test wgt only ships the test part then.

 

Jan-Simon Moeller 
October 17, 2018 at 8:49 AM

Please bring the cmake module to feature parity before we add the warning.

A few notes reviewing the current templates in the next comment ...

 

Romain Forlot 
October 2, 2018 at 4:07 PM

Sure, old repo should remains read-only once the warning message is implemented. New features will only lies in the latest cmake_app_modules repo slightly smiling face

Code is in sandbox and would be soon be released.

Fixed

Details

Assignee

Reporter

Fix versions

Labels

Contract ID

Components

Affects versions

Priority

Created August 23, 2018 at 10:07 AM
Updated December 7, 2018 at 11:53 PM
Resolved November 7, 2018 at 1:54 PM

Flag notifications