🔧 Update deprecated auto_build.py (#23427)
This commit is contained in:
parent
24f9c3a777
commit
1170ed995e
@ -72,7 +72,7 @@
|
|||||||
from __future__ import print_function
|
from __future__ import print_function
|
||||||
from __future__ import division
|
from __future__ import division
|
||||||
|
|
||||||
import sys,os
|
import sys,os,re
|
||||||
|
|
||||||
pwd = os.getcwd() # make sure we're executing from the correct directory level
|
pwd = os.getcwd() # make sure we're executing from the correct directory level
|
||||||
pwd = pwd.replace('\\', '/')
|
pwd = pwd.replace('\\', '/')
|
||||||
@ -123,7 +123,7 @@ from datetime import datetime, date, time
|
|||||||
#
|
#
|
||||||
##########################################################################################
|
##########################################################################################
|
||||||
|
|
||||||
def get_answer(board_name, cpu_label_txt, cpu_a_txt, cpu_b_txt):
|
def get_answer(board_name, question_txt, options, default_value=1):
|
||||||
|
|
||||||
if python_ver == 2:
|
if python_ver == 2:
|
||||||
import Tkinter as tk
|
import Tkinter as tk
|
||||||
@ -151,10 +151,10 @@ def get_answer(board_name, cpu_label_txt, cpu_a_txt, cpu_b_txt):
|
|||||||
root_get_answer.protocol("WM_DELETE_WINDOW", disable_event)
|
root_get_answer.protocol("WM_DELETE_WINDOW", disable_event)
|
||||||
root_get_answer.resizable(False, False)
|
root_get_answer.resizable(False, False)
|
||||||
|
|
||||||
root_get_answer.radio_state = 1 # declare variables used by TK and enable
|
root_get_answer.radio_state = default_value # declare variables used by TK and enable
|
||||||
|
|
||||||
global get_answer_val
|
global get_answer_val
|
||||||
get_answer_val = 2 # return get_answer_val, set default to match radio_state default
|
get_answer_val = default_value # return get_answer_val, set default to match radio_state default
|
||||||
|
|
||||||
radio_state = tk.IntVar()
|
radio_state = tk.IntVar()
|
||||||
radio_state.set(get_answer_val)
|
radio_state.set(get_answer_val)
|
||||||
@ -162,35 +162,27 @@ def get_answer(board_name, cpu_label_txt, cpu_a_txt, cpu_b_txt):
|
|||||||
l1 = tk.Label(text=board_name, fg="light green", bg="dark green",
|
l1 = tk.Label(text=board_name, fg="light green", bg="dark green",
|
||||||
font="default 14 bold").grid(row=0, columnspan=2, sticky='EW', ipadx=2, ipady=2)
|
font="default 14 bold").grid(row=0, columnspan=2, sticky='EW', ipadx=2, ipady=2)
|
||||||
|
|
||||||
l2 = tk.Label(text=cpu_label_txt).grid(row=1, pady=4, columnspan=2, sticky='EW')
|
l2 = tk.Label(text=question_txt).grid(row=1, pady=4, columnspan=2, sticky='EW')
|
||||||
|
|
||||||
b4 = tk.Radiobutton(
|
buttons = []
|
||||||
text=cpu_a_txt,
|
|
||||||
|
for index, val in enumerate(options):
|
||||||
|
buttons.append(
|
||||||
|
tk.Radiobutton(
|
||||||
|
text=val,
|
||||||
fg="black",
|
fg="black",
|
||||||
bg="lightgray",
|
bg="lightgray",
|
||||||
relief=tk.SUNKEN,
|
relief=tk.SUNKEN,
|
||||||
selectcolor="green",
|
selectcolor="green",
|
||||||
variable=radio_state,
|
variable=radio_state,
|
||||||
value=1,
|
value=index + 1,
|
||||||
indicatoron=0,
|
indicatoron=0,
|
||||||
command=CPU_exit_3
|
command=CPU_exit_3
|
||||||
).grid(row=2, pady=1, ipady=2, ipadx=10, columnspan=2)
|
).grid(row=index + 2, pady=1, ipady=2, ipadx=10, columnspan=2))
|
||||||
|
|
||||||
b5 = tk.Radiobutton(
|
b6 = tk.Button(text="Cancel", fg="red", command=kill_session).grid(row=(2 + len(options)), column=0, padx=4, pady=4, ipadx=2, ipady=2)
|
||||||
text=cpu_b_txt,
|
|
||||||
fg="black",
|
|
||||||
bg="lightgray",
|
|
||||||
relief=tk.SUNKEN,
|
|
||||||
selectcolor="green",
|
|
||||||
variable=radio_state,
|
|
||||||
value=2,
|
|
||||||
indicatoron=0,
|
|
||||||
command=CPU_exit_3
|
|
||||||
).grid(row=3, pady=1, ipady=2, ipadx=10, columnspan=2) # use same variable but inverted so they will track
|
|
||||||
|
|
||||||
b6 = tk.Button(text="Cancel", fg="red", command=kill_session).grid(row=4, column=0, padx=4, pady=4, ipadx=2, ipady=2)
|
b7 = tk.Button(text="Continue", fg="green", command=got_answer).grid(row=(2 + len(options)), column=1, padx=4, pady=4, ipadx=2, ipady=2)
|
||||||
|
|
||||||
b7 = tk.Button(text="Continue", fg="green", command=got_answer).grid(row=4, column=1, padx=4, pady=4, ipadx=2, ipady=2)
|
|
||||||
|
|
||||||
def got_answer_():
|
def got_answer_():
|
||||||
root_get_answer.destroy()
|
root_get_answer.destroy()
|
||||||
@ -485,6 +477,11 @@ def get_env_from_line(line, start_position):
|
|||||||
return env, next_position
|
return env, next_position
|
||||||
|
|
||||||
|
|
||||||
|
def invalid_board():
|
||||||
|
print('ERROR - invalid board')
|
||||||
|
print(board_name)
|
||||||
|
raise SystemExit(0) # quit if unable to find board
|
||||||
|
|
||||||
# scan pins.h for board name and return the environment(s) found
|
# scan pins.h for board name and return the environment(s) found
|
||||||
def get_starting_env(board_name_full, version):
|
def get_starting_env(board_name_full, version):
|
||||||
# get environment starting point
|
# get environment starting point
|
||||||
@ -496,48 +493,26 @@ def get_starting_env(board_name_full, version):
|
|||||||
with open(path, 'r') as myfile:
|
with open(path, 'r') as myfile:
|
||||||
pins_h = myfile.read()
|
pins_h = myfile.read()
|
||||||
|
|
||||||
env_A = ''
|
|
||||||
env_B = ''
|
|
||||||
env_C = ''
|
|
||||||
|
|
||||||
board_name = board_name_full[6:] # only use the part after "BOARD_" since we're searching the pins.h file
|
board_name = board_name_full[6:] # only use the part after "BOARD_" since we're searching the pins.h file
|
||||||
pins_h = pins_h.split('\n')
|
pins_h = pins_h.split('\n')
|
||||||
environment = ''
|
|
||||||
board_line = ''
|
|
||||||
cpu_A = ''
|
|
||||||
cpu_B = ''
|
|
||||||
i = 0
|
|
||||||
list_start_found = False
|
list_start_found = False
|
||||||
for lines in pins_h:
|
possible_envs = None
|
||||||
i = i + 1 # i is always one ahead of the index into pins_h
|
for i, line in enumerate(pins_h):
|
||||||
if 0 < lines.find("Unknown MOTHERBOARD value set in Configuration.h"):
|
if 0 < line.find("Unknown MOTHERBOARD value set in Configuration.h"):
|
||||||
break # no more
|
invalid_board();
|
||||||
if 0 < lines.find('1280'):
|
if list_start_found == False and 0 < line.find('1280'):
|
||||||
list_start_found = True
|
list_start_found = True
|
||||||
if list_start_found == False: # skip lines until find start of CPU list
|
elif list_start_found == False: # skip lines until find start of CPU list
|
||||||
continue
|
continue
|
||||||
board = lines.find(board_name)
|
|
||||||
comment_start = lines.find('// ')
|
# Use a regex to find the board. Make sure it is surrounded by separators so the full boardname
|
||||||
cpu_A_loc = comment_start
|
# will be matched, even if multiple exist in a single MB macro. This avoids problems with boards
|
||||||
cpu_B_loc = 0
|
# such as MALYAN_M200 and MALYAN_M200_V2 where one board is a substring of the other.
|
||||||
if board > 0: # need to look at the next line for environment info
|
if re.search(r'MB.*[\(, ]' + board_name + r'[, \)]', line):
|
||||||
cpu_line = pins_h[i]
|
# need to look at the next line for environment info
|
||||||
comment_start = cpu_line.find('// ')
|
possible_envs = re.findall(r'env:([^ ]+)', pins_h[i + 1])
|
||||||
env_A, next_position = get_env_from_line(cpu_line, comment_start) # get name of environment & start of search for next
|
|
||||||
env_B, next_position = get_env_from_line(cpu_line, next_position) # get next environment, if it exists
|
|
||||||
env_C, next_position = get_env_from_line(cpu_line, next_position) # get next environment, if it exists
|
|
||||||
break
|
break
|
||||||
return env_A, env_B, env_C
|
return possible_envs
|
||||||
|
|
||||||
|
|
||||||
# Scan input string for CPUs that users may need to select from
|
|
||||||
# return: CPU name
|
|
||||||
def get_CPU_name(environment):
|
|
||||||
CPU_list = ('1280', '2560', '644', '1284', 'LPC1768', 'DUE')
|
|
||||||
CPU_name = ''
|
|
||||||
for CPU in CPU_list:
|
|
||||||
if 0 < environment.find(CPU):
|
|
||||||
return CPU
|
|
||||||
|
|
||||||
|
|
||||||
# get environment to be used for the build
|
# get environment to be used for the build
|
||||||
@ -549,71 +524,81 @@ def get_env(board_name, ver_Marlin):
|
|||||||
print(board_name)
|
print(board_name)
|
||||||
raise SystemExit(0) # no environment so quit
|
raise SystemExit(0) # no environment so quit
|
||||||
|
|
||||||
def invalid_board():
|
possible_envs = get_starting_env(board_name, ver_Marlin)
|
||||||
print('ERROR - invalid board')
|
|
||||||
print(board_name)
|
|
||||||
raise SystemExit(0) # quit if unable to find board
|
|
||||||
|
|
||||||
CPU_question = (('1280', '2560', '1280 or 2560 CPU?'), ('644', '1284', '644 or 1284 CPU?'))
|
if not possible_envs:
|
||||||
|
|
||||||
if 0 < board_name.find('MELZI'):
|
|
||||||
get_answer(
|
|
||||||
board_name, " Which flavor of Melzi? ", "Melzi (Optiboot bootloader)", "Melzi "
|
|
||||||
)
|
|
||||||
if 1 == get_answer_val:
|
|
||||||
target_env = 'melzi_optiboot'
|
|
||||||
else:
|
|
||||||
target_env = 'melzi'
|
|
||||||
else:
|
|
||||||
env_A, env_B, env_C = get_starting_env(board_name, ver_Marlin)
|
|
||||||
|
|
||||||
if env_A == '':
|
|
||||||
no_environment()
|
no_environment()
|
||||||
if env_B == '':
|
|
||||||
return env_A # only one environment so finished
|
|
||||||
|
|
||||||
CPU_A = get_CPU_name(env_A)
|
# Proceed to ask questions based on the available environments to filter down to a smaller list.
|
||||||
CPU_B = get_CPU_name(env_B)
|
# If more then one remains after this filtering the user will be prompted to choose between
|
||||||
|
# all remaining options.
|
||||||
|
|
||||||
for item in CPU_question:
|
# Filter selection based on CPU choice
|
||||||
if CPU_A == item[0]:
|
CPU_questions = [
|
||||||
get_answer(board_name, item[2], item[0], item[1])
|
{'options':['1280', '2560'], 'text':'1280 or 2560 CPU?', 'default':2},
|
||||||
if 2 == get_answer_val:
|
{'options':['644', '1284'], 'text':'644 or 1284 CPU?', 'default':2},
|
||||||
target_env = env_B
|
{'options':['STM32F103RC', 'STM32F103RE'], 'text':'MCU Type?', 'default':1}]
|
||||||
|
|
||||||
|
for question in CPU_questions:
|
||||||
|
if any(question['options'][0] in env for env in possible_envs) and any(question['options'][1] in env for env in possible_envs):
|
||||||
|
get_answer(board_name, question['text'], [question['options'][0], question['options'][1]], question['default'])
|
||||||
|
possible_envs = [env for env in possible_envs if question['options'][get_answer_val - 1] in env]
|
||||||
|
|
||||||
|
# Choose which STM32 framework to use, if both are available
|
||||||
|
if [env for env in possible_envs if '_maple' in env] and [env for env in possible_envs if '_maple' not in env]:
|
||||||
|
get_answer(board_name, 'Which STM32 Framework should be used?', ['ST STM32 (Preferred)', 'Maple (Deprecated)'])
|
||||||
|
if 1 == get_answer_val:
|
||||||
|
possible_envs = [env for env in possible_envs if '_maple' not in env]
|
||||||
else:
|
else:
|
||||||
target_env = env_A
|
possible_envs = [env for env in possible_envs if '_maple' in env]
|
||||||
return target_env
|
|
||||||
|
|
||||||
if env_A == 'LPC1768':
|
# Both USB and non-USB STM32 options exist, filter based on these
|
||||||
|
if any('STM32F103R' in env for env in possible_envs) and any('_USB' in env for env in possible_envs) and any('_USB' not in env for env in possible_envs):
|
||||||
|
get_answer(board_name, 'USB Support?', ['USB', 'No USB'])
|
||||||
|
if 1 == get_answer_val:
|
||||||
|
possible_envs = [env for env in possible_envs if '_USB' in env]
|
||||||
|
else:
|
||||||
|
possible_envs = [env for env in possible_envs if '_USB' not in env]
|
||||||
|
|
||||||
|
if not possible_envs:
|
||||||
|
no_environment()
|
||||||
|
if len(possible_envs) == 1:
|
||||||
|
return possible_envs[0] # only one environment so finished
|
||||||
|
|
||||||
|
target_env = None
|
||||||
|
|
||||||
|
# A few environments require special behavior
|
||||||
|
if 'LPC1768' in possible_envs:
|
||||||
if build_type == 'traceback' or (build_type == 'clean' and get_build_last() == 'LPC1768_debug_and_upload'):
|
if build_type == 'traceback' or (build_type == 'clean' and get_build_last() == 'LPC1768_debug_and_upload'):
|
||||||
target_env = 'LPC1768_debug_and_upload'
|
target_env = 'LPC1768_debug_and_upload'
|
||||||
else:
|
else:
|
||||||
target_env = 'LPC1768'
|
target_env = 'LPC1768'
|
||||||
elif env_A == 'DUE':
|
elif 'DUE' in possible_envs:
|
||||||
target_env = 'DUE'
|
target_env = 'DUE'
|
||||||
if build_type == 'traceback' or (build_type == 'clean' and get_build_last() == 'DUE_debug'):
|
if build_type == 'traceback' or (build_type == 'clean' and get_build_last() == 'DUE_debug'):
|
||||||
target_env = 'DUE_debug'
|
target_env = 'DUE_debug'
|
||||||
elif env_B == 'DUE_USB':
|
elif 'DUE_USB' in possible_envs:
|
||||||
get_answer(board_name, 'DUE Download Port?', '(Native) USB port', 'Programming port')
|
get_answer(board_name, 'DUE Download Port?', ['(Native) USB port', 'Programming port'])
|
||||||
if 1 == get_answer_val:
|
if 1 == get_answer_val:
|
||||||
target_env = 'DUE_USB'
|
target_env = 'DUE_USB'
|
||||||
else:
|
else:
|
||||||
target_env = 'DUE'
|
target_env = 'DUE'
|
||||||
elif env_A == 'STM32F103RC_btt' or env_A == 'STM32F103RE_btt':
|
|
||||||
if env_A == 'STM32F103RE_btt':
|
|
||||||
get_answer(board_name, 'MCU Type?', 'STM32F103RC', 'STM32F103RE')
|
|
||||||
if 1 == get_answer_val:
|
|
||||||
env_A = 'STM32F103RC_btt'
|
|
||||||
target_env = env_A
|
|
||||||
if env_A == 'STM32F103RC_btt':
|
|
||||||
get_answer(board_name, 'RCT6 Flash Size?', '512K', '256K')
|
|
||||||
if 1 == get_answer_val:
|
|
||||||
target_env += '_512K'
|
|
||||||
get_answer(board_name, 'USB Support?', 'USB', 'No USB')
|
|
||||||
if 1 == get_answer_val:
|
|
||||||
target_env += '_USB'
|
|
||||||
else:
|
else:
|
||||||
invalid_board()
|
options = possible_envs
|
||||||
|
# Perform some substitutions for environment names which follow a consistent
|
||||||
|
# naming pattern and are very commonly used. This is fragile code, and replacements
|
||||||
|
# should only be made here for stable environments unlikely to change often.
|
||||||
|
for i, option in enumerate(options):
|
||||||
|
if 'melzi' in option:
|
||||||
|
options[i] = 'Melzi'
|
||||||
|
elif 'sanguino1284p' in option:
|
||||||
|
options[i] = 'sanguino1284p'
|
||||||
|
if 'optiboot' in option:
|
||||||
|
options[i] = options[i] + ' (Optiboot Bootloader)'
|
||||||
|
if 'optimized' in option:
|
||||||
|
options[i] = options[i] + ' (Optimized for Size)'
|
||||||
|
get_answer(board_name, 'Which environment?', options)
|
||||||
|
target_env = possible_envs[get_answer_val - 1]
|
||||||
|
|
||||||
if build_type == 'traceback' and target_env != 'LPC1768_debug_and_upload' and target_env != 'DUE_debug' and Marlin_ver == 2:
|
if build_type == 'traceback' and target_env != 'LPC1768_debug_and_upload' and target_env != 'DUE_debug' and Marlin_ver == 2:
|
||||||
print("ERROR - this board isn't setup for traceback")
|
print("ERROR - this board isn't setup for traceback")
|
||||||
|
Loading…
Reference in New Issue
Block a user