Remove property definition, and rename to PICOTOOL_PROCESSING_CONFIGURED and picotool_check_configurable

Author: will-v-pi

tools/CMakeLists.txt

@@ -71,12 +71,6 @@ define_property(TARGET
     BRIEF_DOCS "Extra arguments to pass to uf2 conversion"
     FULL_DOCS "Extra arguments to pass to uf2 conversion"
 )
-define_property(TARGET
-    PROPERTY PICOTOOL_PROCESSING_DONE
-    INHERITED
-    BRIEF_DOCS "Set once post-processing has been completed in CMake"
-    FULL_DOCS "Set once post-processing has been completed in CMake"
-)
 
 # Check pioasm is installed, or build it if not installed
 function(pico_init_pioasm)
@@ -152,8 +146,8 @@ function(pico_init_picotool)
     endif()
 endfunction()
 
-function(picotool_check_not_done TARGET)
-    get_target_property(done ${TARGET} PICOTOOL_PROCESSING_DONE)
+function(picotool_check_configurable TARGET)
+    get_target_property(done ${TARGET} PICOTOOL_PROCESSING_CONFIGURED)
     if (done)
         message(FATAL_ERROR "All picotool post-processing functions for \"${TARGET}\" must come before pico_add_extra_outputs(${TARGET})")
     endif()
@@ -211,7 +205,7 @@ endfunction()
 # dropping, and it will be copied to SRAM by the bootrom before execution.
 # This sets PICOTOOL_UF2_PACKAGE_ADDR to PACKADDR.
 function(pico_package_uf2_output TARGET PACKADDR)
-    picotool_check_not_done(${TARGET})
+    picotool_check_configurable(${TARGET})
     set_target_properties(${TARGET} PROPERTIES
         PICOTOOL_UF2_PACKAGE_ADDR ${PACKADDR}
     )
@@ -221,7 +215,7 @@ endfunction()
 # Output the public key hash and other necessary rows to an otp JSON file.
 # This sets PICOTOOL_OTP_FILE to OTPFILE.
 function(pico_set_otp_key_output_file TARGET OTPFILE)
-    picotool_check_not_done(${TARGET})
+    picotool_check_configurable(${TARGET})
     set_target_properties(${TARGET} PROPERTIES
         PICOTOOL_OTP_FILE ${OTPFILE}
     )
@@ -232,7 +226,7 @@ endfunction()
 # before loading the binary. This appends the `--clear` argument
 # to PICOTOOL_EXTRA_PROCESS_ARGS.
 function(pico_load_map_clear_sram TARGET)
-    picotool_check_not_done(${TARGET})
+    picotool_check_configurable(${TARGET})
     # get and set, to inherit list
     get_target_property(extra_args ${TARGET} PICOTOOL_EXTRA_PROCESS_ARGS)
     if (extra_args)
@@ -250,7 +244,7 @@ endfunction()
 # to PICOTOOL_EXTRA_PROCESS_ARGS if setting the rollback version, or set as compile
 # definitions if only setting the major/minor versions.
 function(pico_set_binary_version TARGET)
-    picotool_check_not_done(${TARGET})
+    picotool_check_configurable(${TARGET})
     set(oneValueArgs MAJOR MINOR ROLLBACK)
     set(multiValueArgs ROWS)
     cmake_parse_arguments(PARSE_ARGV 1 SV "" "${oneValueArgs}" "${multiValueArgs}")
@@ -300,7 +294,7 @@ endfunction()
 # Set the UF2 family to use when creating the UF2.
 # This sets PICOTOOL_UF2_FAMILY to FAMILY.
 function(pico_set_uf2_family TARGET FAMILY)
-    picotool_check_not_done(${TARGET})
+    picotool_check_configurable(${TARGET})
     set_target_properties(${TARGET} PROPERTIES
         PICOTOOL_UF2_FAMILY ${FAMILY}
     )
@@ -313,7 +307,7 @@ endfunction()
 # specify a common SIGFILE for multiple targets, the SIGFILE property can be
 # set for a given scope, and then the SIGFILE argument is optional.
 function(pico_sign_binary TARGET)
-    picotool_check_not_done(${TARGET})
+    picotool_check_configurable(${TARGET})
     # Enforce signing through target properties
     set_target_properties(${TARGET} PROPERTIES
         PICOTOOL_SIGN_OUTPUT true
@@ -339,7 +333,7 @@ endfunction()
 # pico_hash_binary(TARGET)
 # Hash the target binary. This sets PICOTOOL_HASH_OUTPUT to true.
 function(pico_hash_binary TARGET)
-    picotool_check_not_done(${TARGET})
+    picotool_check_configurable(${TARGET})
     # Enforce hashing through target properties
     set_target_properties(${TARGET} PROPERTIES
         PICOTOOL_HASH_OUTPUT true
@@ -350,7 +344,7 @@ endfunction()
 # Create the specified partition table from JSON, and embed it in the
 # block loop. This sets PICOTOOL_EMBED_PT to PTFILE.
 function(pico_embed_pt_in_binary TARGET PTFILE)
-    picotool_check_not_done(${TARGET})
+    picotool_check_configurable(${TARGET})
     set_target_properties(${TARGET} PROPERTIES
         PICOTOOL_EMBED_PT ${PTFILE}
     )
@@ -362,7 +356,7 @@ endfunction()
 # This sets PICOTOOL_AESFILE to AESFILE, and PICOTOOL_ENC_SIGFILE to SIGFILE
 # if present, else PICOTOOL_SIGFILE.
 function(pico_encrypt_binary TARGET AESFILE)
-    picotool_check_not_done(${TARGET})
+    picotool_check_configurable(${TARGET})
     set_target_properties(${TARGET} PROPERTIES
         PICOTOOL_AESFILE ${AESFILE}
     )
@@ -445,7 +439,7 @@ endfunction()
 # all required properties have been set
 function(picotool_postprocess_binary TARGET)
     set_target_properties(${TARGET} PROPERTIES
-        PICOTOOL_PROCESSING_DONE true
+        PICOTOOL_PROCESSING_CONFIGURED true
     )
     # Read target properties
     get_target_property(picotool_sign_output ${TARGET} PICOTOOL_SIGN_OUTPUT)