Skip to content

Conversation

@nashif
Copy link
Member

@nashif nashif commented Jan 11, 2023

We want to be able to have platform or architecture extra configs
without having to duplicate a whole section of the test specification.

This adds support for namespacing of extra configs, for example:

arch:nios2:CONFIG_SYS_CLOCK_TICKS_PER_SEC=1000

or

platform:qemu_x86:CONFIG_FOO=y

Signed-off-by: Anas Nashif anas.nashif@intel.com

We want to be able to have platform or architecture extra configs
without having to duplicate a whole section of the test specification.

This adds support for namespacing of extra configs, for example:

arch:nios2:CONFIG_SYS_CLOCK_TICKS_PER_SEC=1000

or

platform:qemu_x86:CONFIG_FOO=y

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
@keith-packard
Copy link
Contributor

Works great! Could this be done for 'west build' too?

@carlescufi
Copy link
Member

Works great! Could this be done for 'west build' too?

But west build only ever builds a single arch (for now) @keith-packard, unlike twister.

@carlescufi carlescufi merged commit 73584df into zephyrproject-rtos:main Jan 12, 2023
@PerMac
Copy link
Member

PerMac commented Jan 12, 2023

west has option --test-item where one can point to a particular scenario from yestcase/sample.yaml and west would build with using extra_args from yaml. I guess it can (or in fact should, for complementary?) updated as well.

@keith-packard
Copy link
Contributor

west has option --test-item where one can point to a particular scenario from yestcase/sample.yaml and west would build with using extra_args from yaml. I guess it can (or in fact should, for complementary?) updated as well.

yeah, I actually tried using west like this before I realized the patch was only to twister...

@savvn001
Copy link

savvn001 commented Oct 4, 2023

Could this namespacing feature be added to extra_args too?
Would be good to have something like this, rather than listing all KConfig options in the testcase.yaml file.

tests:
  drivers.flash.flash1234:
    platform_allow: native_posix_64 my_custom_board
    tags: drivers

    extra_args:       
      - platform:my_custom_board: "CONF_FILE=prj.conf;myboardoptions.conf"
      - platform:native_posix_64: "CONF_FILE=prj.conf"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants