Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

twister: normalize platform name when storing files/data #69338

Merged

Conversation

nashif
Copy link
Member

@nashif nashif commented Feb 22, 2024

Convert slashes into underscores to allow saving of data related to
platforms on disk.

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

Comment on lines 67 to 68
print(self.build_dir)
print(platform.sane_name)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

debug leftover

@PerMac PerMac added the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label Feb 22, 2024
@PerMac
Copy link
Member

PerMac commented Feb 22, 2024

a nit (?): https://developers.google.com/style/word-list#sane I remember we we had to change sanitycheck before and this seems similar case
maybe "sanitized_name" instead? or "filesystem_name"?

@nashif nashif force-pushed the topic/hwm/sane_platform_names branch from 933c023 to 5fb02a1 Compare February 22, 2024 14:20
@nashif nashif requested a review from PerMac February 22, 2024 14:23
Use mps2/an385 for testing the hwmv2 board names.

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Convert slashes into underscores to allow saving of data related to
platforms on disk.

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
@nashif nashif force-pushed the topic/hwm/sane_platform_names branch from 6a76704 to efda3be Compare February 28, 2024 12:19
@carlescufi carlescufi merged commit 76d050a into zephyrproject-rtos:collab-hwm Feb 28, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Twister Twister Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants