Feb. 7, 2025
7:27 a.m.
Besides, since you mentioned uncertainty around apisets,
I mentioned it because it required a closer look, and I ran out of time that day. That said, I think the change is fine. It needed a closer look to review for this MR anyway.
I wonder if such refactoring would merit a separate MR.
Sure, I created !7287. I think it would be good to settle this first. The whole push/pop approach adds extra effort that wouldn’t be necessary if we decide to go with it.
There are more:
In most cases, I’d say the exclusion shouldn’t apply. A few cases might be better handled as regular imports. Either way, I don’t think changing relay exclude is a big deal. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7#note_94027