tree 699aff887b81917140ebfead9e35ea9c861a347e
parent 77014c1590164715fff8f3f3d3c77e9ff01b4d70
author David 'Digit' Turner <digit@google.com> 1679056084 +0100
committer GN LUCI <gn-scoped@luci-project-accounts.iam.gserviceaccount.com> 1679163613 +0000

Make `gn args --list` deterministic

This fixes Args::GetAllArguments() to return deterministic results,
even when declare_args() are used in args.gn or some of its
imported files.

While this is ugly, this is used by the Fuchsia build in order to
override variables in third-party .gni files. This also creates
two Settings instance that return `is_default() == true`:

- The first is just the regular default toolchain.
- The second one has an empty label, and is used by the scope
  created when parsing the args.gn file.

The std::sort() comparator assumed there was only one default
toolchain, this resulted in practice in non-deterministic
outputs for the list of arguments.

This CL fixes the issue by ensuring the comparator places
the empty toolchain first, then the default one second,
followed by all others sorted by their label.

Bug: 327

Change-Id: I342d285a15339572adf72c2ddb29bbd81b814d10
Reviewed-on: https://gn-review.googlesource.com/c/gn/+/15160
Commit-Queue: David Turner <digit@google.com>
Reviewed-by: Brett Wilson <brettw@chromium.org>