Skip to content

Commit

Permalink
illumos linker doesn't currently have -z relro (helix-editor#2602)
Browse files Browse the repository at this point in the history
  • Loading branch information
papertigers authored and mtoohey31 committed Jun 15, 2022
1 parent b53647e commit 4d2afad
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion helix-loader/src/grammar.rs
Original file line number Diff line number Diff line change
Expand Up @@ -350,7 +350,10 @@ fn build_tree_sitter_library(src_path: &Path, grammar: GrammarConfiguration) ->
}
}
command.arg("-xc").arg(parser_path);
if cfg!(all(unix, not(target_os = "macos"))) {
if cfg!(all(
unix,
not(any(target_os = "macos", target_os = "illumos"))
)) {
command.arg("-Wl,-z,relro,-z,now");
}
}
Expand Down

0 comments on commit 4d2afad

Please sign in to comment.