NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commita32ffee

committed

Fallback to clang in PATH with meson

Some distributions put clang into a different path than the llvmbinary path.For example, this is the case on NixOS / nixpkgs, which failed to findclang with meson before this patch.Author: Wolfgang WaltherReviewed-by: Nazir Bilal Yavuz, Alvaro Herrera, Peter EisentrautReviewed-by: Tristan PartinDiscussion:https://www.postgresql.org/message-id/ca8f37e1-a2c3-40e2-91f6-59c3d3652ad4@technowledgy.deBackpatch: 16-, where meson support was added

1 parent469b97c commita32ffeeCopy full SHA for a32ffee

File tree

1 file changed

-1

lines changed

meson.build

1 file changed

-1

lines changed

`‎meson.build`

Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -799,7 +799,10 @@ if add_languages('cpp', required: llvmopt, native: false)`
`799`	`799`	`llvm_binpath= llvm.get_variable(configtool:'bindir')`
`800`	`800`
`801`	`801`	`ccache=find_program('ccache',native:true,required:false)`
`802`		`- clang=find_program(llvm_binpath/'clang',required:true)`
	`802`	`+`
	`803`	`+# Some distros put LLVM and clang in different paths, so fallback to`
	`804`	`+# find via PATH, too.`
	`805`	`+ clang=find_program(llvm_binpath/'clang','clang',required:true)`
`803`	`806`	`endif`
`804`	`807`	`elif llvmopt.auto()`
`805`	`808`	`message('llvm requires a C++ compiler')`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commita32ffee

File tree

1 file changed

1 file changed

`‎meson.build`

0 commit comments