Fix alert flags so that Merlin understands them#5245
Conversation
Signed-off-by: Andrey Mokhov <amokhov@janestreet.com>
| (env | ||
| (_ | ||
| (flags :standard \ -alert=-unstable))) | ||
| (flags :standard \ -alert -unstable))) |
There was a problem hiding this comment.
That'll work for this case, but in general it means: exclude "-alert" and exclude "-unstable" which is not the same as exclude "-alert -unstable". It'll work for us as "-unstable" doesn't appear anywhere else.
There was a problem hiding this comment.
Yes, removing flags in this way is unsafe but I guess we don't have a safer mechanism. Hopefully, it'll not bite us in this particular case.
There was a problem hiding this comment.
If the intent is to enable the unstable alert, I think you can use :standard -alert +unstable.
There was a problem hiding this comment.
Oh, good point, let me try that.
There was a problem hiding this comment.
The intent is to remove the flag because configurator needs to build with 4.02 and 4.02 doesn't know the -alert flag.
There was a problem hiding this comment.
Ah right, then it doesn't help, sorry for the noise.
There was a problem hiding this comment.
OK, merging as is then.
As discussed on Slack, it seems that Merlin behaves differently from the compiler when interpreting
alertflags. As suggested by @voodoos, I'm tweaking the flags to help Merlin a bit. This solves the issue for me locally.