Recursive group filter imports are not working properly about west HOT 1 CLOSED

Well, oops. This was actually a documentation bug, not a west bug, but I'm going to keep this open so that we can close it automatically when we turn off the xfail in the tests introduced by commit d0e6b9a.

Earlier in the same section, we see:

The resolved manifest has a group-filter value which is the result of concatenating the group-filter values in the top-level manifest and any imported manifests.

Manifest files which appear earlier in the import order have higher precedence and are therefore concatenated later into the final group-filter.

Where the "import order" is defined as:

Importing is done in this order:

1. Manifests from self-import are imported first.
2. The top-level manifest file’s definitions are handled next.
3. Manifests from import-1, …, import-N, are imported in that order.

This means that import-1 happens earlier than import-N, and should therefore have higher precedence when it comes to computing the final group filter.

That means that, instead of saying this:

The final resolved group-filter value is then filter1 + filter-2 + ... + filter-N + top-filter + self-filter, where + here refers to list concatenation.

The documentation should say:

The final resolved group-filter value is then filter-N + ... + filter-1 + top-filter + self-filter, where + here refers to list concatenation.

Notice it currently goes from filter-1 to filter-N. It should go from filter-N to filter-1.

from west.