-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Read base path for config files from TMPDIR env var #33846
base: master
Are you sure you want to change the base?
Read base path for config files from TMPDIR env var #33846
Conversation
PR #33846: Size comparison from bd0422b to 76750ee Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I leave an optional suggestion for you!
76750ee
to
1e73d53
Compare
PR #33846: Size comparison from dc8187b to 1e73d53 Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
1e73d53
to
b460245
Compare
PR #33846: Size comparison from e1d746a to b460245 Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
b460245
to
76e4435
Compare
PR #33846: Size comparison from f6ac926 to 76e4435 Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #33846: Size comparison from 38f664f to c81b16e Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #33846: Size comparison from ee49ebd to 0bd8b13 Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
Fixes #30790
Change hard coded paths for
chip_factory.ini
,chip_config.ini
andchip_counters.ini
to rather use existingTMPDIR
environment variable.Before this change these files are stored under
/tmp/*
. After the change they are stored under$TMPDIR/*
, with a fallback to/tmp/*
if the variable is not set.Rationale
Chip Tool might be used by different users on a system. If the paths are set at build time, multiple users will use the same storage file. This causes a permission error. The first user that runs Chip Tool will be the owner of the files, and the only user that can read and write the files. Subsequent users will see Chip Tool failing with a permission error on these files.
By allowing to override the path for storage file with an environment variable, one can set different paths for different users. This allows isolating different users' storage files, and prevents permission errors on these files.