Skip to content

build: Fix --no-sync with no arguments

Ryan Roberts requested to merge ryarob01/fix-no-sync into main

Previously --no-sync would allow 0 or 1 arguments. For the 0 args case, that meant "do not sync any of the components' repos". However, this usage causes incorrect parsing if the config is immediately following --no-sync on the command line; in this case, the config would be interpretted as the argument.

Let's fix this by separating the 0 arg use into a separate option, --no-sync-all. Now --no-sync always requires an argument.

Signed-off-by: Ryan Roberts ryan.roberts@arm.com

Merge request reports