Commit b15a424
committed
CI: the all-rules denominator counts members, not internal features
这一步比较包里每一个 `[features.<x>]` 与夹具激活的那一列,好让第七个成员不能被悄悄漏出
「every rule module compiles for this host」。`[features.surface]` 让它红了,而且红得对:
那是一个夹具没点名的新 feature。
但它不是成员。它承载 surface 的构建程序那一半,每个成员都 implies 它,消费者永远不写它。
把它加进夹具是错的修法 —— 夹具那一列的含义是「消费者能激活的成员」,为了让检查变绿而往里
填东西,只会让检查的含义变少。
所以分母减去「被别的 feature implies 的」。这条规则从 manifest 推出来而不是列在这里,正是
这个检查原本就有的性质:第七个**成员**仍会被抓到,因为没有东西 implies 它。
两条腿都验过:修正后两列一致;临时加一个新成员,它出现在分母里并会让这一步失败。1 parent 7301bce commit b15a424
1 file changed
Lines changed: 13 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
752 | 752 | | |
753 | 753 | | |
754 | 754 | | |
| 755 | + | |
| 756 | + | |
| 757 | + | |
| 758 | + | |
| 759 | + | |
| 760 | + | |
| 761 | + | |
| 762 | + | |
| 763 | + | |
| 764 | + | |
| 765 | + | |
755 | 766 | | |
756 | | - | |
| 767 | + | |
| 768 | + | |
757 | 769 | | |
758 | 770 | | |
759 | 771 | | |
| |||
0 commit comments