NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit23be520

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 parentcc90d78 commit23be520Copy full SHA for 23be520

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
`@@ -805,7 +805,10 @@ if add_languages('cpp', required: llvmopt, native: false)`
`805`	`805`	`llvm_binpath= llvm.get_variable(configtool:'bindir')`
`806`	`806`
`807`	`807`	`ccache=find_program('ccache',native:true,required:false)`
`808`		`- clang=find_program(llvm_binpath/'clang',required:true)`
	`808`	`+`
	`809`	`+# Some distros put LLVM and clang in different paths, so fallback to`
	`810`	`+# find via PATH, too.`
	`811`	`+ clang=find_program(llvm_binpath/'clang','clang',required:true)`
`809`	`812`	`endif`
`810`	`813`	`elif llvmopt.auto()`
`811`	`814`	`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!

Commit23be520

File tree

1 file changed

1 file changed

`‎meson.build`

0 commit comments