Skip to content

Commit

Permalink
Fix clear-config in build-all-clusters-app.py (#8260)
Browse files Browse the repository at this point in the history
  • Loading branch information
cecille authored and pull[bot] committed Aug 28, 2021
1 parent a5be1e6 commit 1154616
Showing 1 changed file with 20 additions and 8 deletions.
28 changes: 20 additions & 8 deletions scripts/examples/build-all-clusters-app.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

import argparse
import os
import shutil
import subprocess
import logging
import coloredlogs
Expand Down Expand Up @@ -41,7 +42,7 @@ def main():
parser.add_argument(
'--clear-config',
default=None,
choices=['m5stack', 'devkit'],
choices=['m5stack', 'devkit', 'curr', 'default'],
)
args = parser.parse_args()

Expand All @@ -54,17 +55,28 @@ def main():
e = IDFExecutor()

if args.clear_config:
old_default_sdkconfig = None
clear_curr = args.clear_config != 'curr'
logging.info('Building a clear configuration')
sdkconfig = 'sdkconfig_%s.defaults' % args.clear_config

curr_sdkconfig = os.path.join(ROOT, 'sdkconfig')
if os.path.exists(curr_sdkconfig):
os.remove(curr_sdkconfig)
if args.clear_config == "m5stack" or args.clear_config == "devkit":
desired_sdkconfig_name = 'sdkconfig_%s.defaults' % args.clear_config
logging.info('Using default' + desired_sdkconfig_name)
desired_sdkconfig = os.path.join(ROOT, desired_sdkconfig_name)
default_sdkconfig = os.path.join(ROOT, 'sdkconfig.defaults')
old_default_sdkconfig = os.path.join(ROOT, 'sdkconfig.defaults.old')
shutil.copy(default_sdkconfig, old_default_sdkconfig)
shutil.copy(desired_sdkconfig, default_sdkconfig)

e.execute('-D SDKCONFIG_DEFAULTS="{sdkname}" build'.format(
sdkname=sdkconfig
))
if clear_curr:
logging.info('Clearing current config')
sdkconfig = os.path.join(ROOT, 'sdkconfig')
os.remove(sdkconfig)

e.execute('menuconfig')

if old_default_sdkconfig is not None:
shutil.move(old_default_sdkconfig, default_sdkconfig)

logging.info('Compiling')
e.execute('build')
Expand Down

0 comments on commit 1154616

Please sign in to comment.