Skip to content

Commit 2aa6079

Browse files
author
jantje
committed
#1268 cleanup of unnneeded environment vars
Basically the settings were saved twice, once in the sloeber project file sproject and in the environment variables
1 parent 5ebc997 commit 2aa6079

File tree

2 files changed

+12
-16
lines changed

2 files changed

+12
-16
lines changed

io.sloeber.core/src/io/sloeber/core/api/BoardDescription.java

+4-15
Original file line numberDiff line numberDiff line change
@@ -52,14 +52,8 @@ public class BoardDescription {
5252
private static final String KEY_LAST_USED_UPLOAD_PROTOCOL = "last Used upload Protocol"; //$NON-NLS-1$
5353
private static final String KEY_LAST_USED_BOARDS_FILE = "Last used Boards file"; //$NON-NLS-1$
5454
private static final String KEY_LAST_USED_BOARD_MENU_OPTIONS = "last used Board custom option selections"; //$NON-NLS-1$
55-
private static final String ENV_KEY_JANTJE_MENU_SELECTION = ENV_KEY_JANTJE_START + MENU;
56-
private static final String ENV_KEY_JANTJE_UPLOAD_PORT = ENV_KEY_JANTJE_START + COM_PORT;
57-
private static final String ENV_KEY_JANTJE_BOARD_NAME = ENV_KEY_JANTJE_START + "board_name"; //$NON-NLS-1$
58-
private static final String ENV_KEY_JANTJE_BOARDS_FILE = ENV_KEY_JANTJE_START + "boards_file"; //$NON-NLS-1$
59-
private static final String ENV_KEY_JANTJE_PACKAGE_ID = ENV_KEY_JANTJE_START + "package_ID"; //$NON-NLS-1$
60-
private static final String ENV_KEY_JANTJE_ARCITECTURE_ID = ENV_KEY_JANTJE_START + "architecture_ID"; //$NON-NLS-1$
61-
private static final String ENV_KEY_JANTJE_BOARD_ID = ENV_KEY_JANTJE_START + "board_ID"; //$NON-NLS-1$
6255
private static final String ENV_KEY_SERIAL_PORT = ERASE_START + "serial_port"; //$NON-NLS-1$
56+
private static final String ENV_KEY_SERIAL_DOT_PORT = ERASE_START + "serial.port"; //$NON-NLS-1$
6357
private static final String ENV_KEY_SERIAL_PORT_FILE = ERASE_START + "serial.port.file"; //$NON-NLS-1$
6458
private static final String ENV_KEY_BUILD_VARIANT_PATH = ERASE_START + BUILD + DOT + VARIANT + DOT + PATH;
6559
private static final String ENV_KEY_BUILD_ACTUAL_CORE_PATH = ERASE_START + BUILD + DOT + CORE + DOT + PATH;
@@ -862,15 +856,10 @@ public Map<String, String> getEnvVars() {
862856
allVars.put(ENV_KEY_HARDWARE_PATH, getreferencedHardwarePath().toOSString());
863857
allVars.put(ENV_KEY_PLATFORM_PATH, getreferencingPlatformPath().toOSString());
864858

865-
allVars.putAll(getEnvVarsConfig(EMPTY));
866-
allVars.put(ENV_KEY_JANTJE_BOARD_NAME, getBoardName());
867-
allVars.put(ENV_KEY_JANTJE_BOARDS_FILE, getReferencingBoardsFile().toString());
868-
allVars.put(ENV_KEY_JANTJE_ARCITECTURE_ID, getArchitecture());
869-
allVars.put(ENV_KEY_JANTJE_PACKAGE_ID, getPackage());
870-
allVars.put(ENV_KEY_JANTJE_UPLOAD_PORT, this.myUploadPort);
871-
allVars.put(JANTJE_ACTION_UPLOAD, this.myProgrammer);
872859

873860
allVars.put(ENV_KEY_SERIAL_PORT, getActualUploadPort());
861+
allVars.put(ENV_KEY_SERIAL_DOT_PORT, getActualUploadPort());
862+
874863
allVars.put(ENV_KEY_SERIAL_PORT_FILE, getActualUploadPort().replace("/dev/", EMPTY)); //$NON-NLS-1$
875864
// if actual core path is osstring regression test issue555 willl fail teensy
876865
// stuff
@@ -911,7 +900,7 @@ public Map<String, String> getEnvVars() {
911900
}
912901
}
913902

914-
// boards setiings not comming from menu selections
903+
// boards settings not coming from menu selections
915904
allVars.putAll(myTxtFile.getAllBoardEnvironVars(getBoardID()));
916905

917906
// board settings from menu selections

io.sloeber.core/src/io/sloeber/core/api/CompileDescription.java

+8-1
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,14 @@ public void set_All_CompileOptions(String my_All_CompileOptions) {
127127
* must be a valid configuration description
128128
*/
129129
public Map<String, String> getEnvVars() {
130-
Map<String, String> ret = getEnvVarsConfig(EMPTY);
130+
Map<String, String> ret = new HashMap<>();
131+
ret.put(ENV_KEY_JANTJE_ADDITIONAL_COMPILE_OPTIONS, this.my_C_andCPP_CompileOptions);
132+
ret.put(ENV_KEY_JANTJE_ADDITIONAL_CPP_COMPILE_OPTIONS, this.my_CPP_CompileOptions);
133+
ret.put(ENV_KEY_JANTJE_ADDITIONAL_C_COMPILE_OPTIONS, this.my_C_CompileOptions);
134+
ret.put(ENV_KEY_JANTJE_ASSEMBLY_COMPILE_OPTIONS, this.my_Assembly_CompileOptions);
135+
ret.put(ENV_KEY_JANTJE_ARCHIVE_COMPILE_OPTIONS, this.my_Archive_CompileOptions);
136+
ret.put(ENV_KEY_JANTJE_LINK_COMPILE_OPTIONS, this.my_Link_CompileOptions);
137+
ret.put(ENV_KEY_JANTJE_ALL_COMPILE_OPTIONS, this.my_All_CompileOptions);
131138

132139
if (this.isWarningLevel()) {
133140
ret.put(ENV_KEY_WARNING_LEVEL_OFF, ENV_KEY_WARNING_LEVEL_ON);

0 commit comments

Comments
 (0)