Skip to content

Commit 9061a4a

Browse files
fix: rescript-legacy broken for ppx's
This resolves PPX errors with "Ill-formed list of warnings" after removal of -bs-v flag. The mtime from being appended to the -bs-v flag, after the change in #7627 it was being appended to the warnings flag instead, causing the compiler error Fixes #8090 Signed-off-by: Rob <illusionalsagacity@users.noreply.github.com>
1 parent 0010ac4 commit 9061a4a

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
- Fix `@val` shadowing (rewrite using `globalThis`). https://github.com/rescript-lang/rescript/pull/8098
2828
- Fix `@scope` shadowing (rewrite using `globalThis`). https://github.com/rescript-lang/rescript/pull/8100
2929
- Fix rewatch panic on duplicate module name. https://github.com/rescript-lang/rescript/pull/8102
30+
- Fix 'Ill-formed list of warnings' errors for ppx's with rescript-legacy. https://github.com/rescript-lang/rescript/pull/8103
3031

3132
#### :memo: Documentation
3233

compiler/bsb/bsb_ninja_rule.ml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -145,10 +145,6 @@ let make_custom_rules ~(gentype_config : Bsb_config_types.gentype_config)
145145
(match ppx_files with
146146
| [] -> ()
147147
| _ ->
148-
Ext_list.iter ppx_files (fun x ->
149-
match string_of_float (Unix.stat x.name).st_mtime with
150-
| exception _ -> ()
151-
| st -> Ext_buffer.add_char_string buf ',' st);
152148
Ext_buffer.add_char_string buf ' ' (Bsb_build_util.ppx_flags ppx_files));
153149
(match pp_file with
154150
| None -> ()

0 commit comments

Comments
 (0)