Skip to content

Commit

Permalink
tests: fix java JNI test assert for jni.h existing
Browse files Browse the repository at this point in the history
compiler.has_header() isn't an assert, even though the comments say it
is. With `required: true` it is an actual assert.
  • Loading branch information
eli-schwartz committed Jan 2, 2024
1 parent 4a96ad0 commit 3156bb5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test cases/java/9 jni/meson.build
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,9 @@ java = find_program('java')
jni_dep = dependency('jni', version : '>=1.8', modules: ['jvm', 'awt'])

# Assert that the header can actually be found with the dependency.
cc.has_header('jni.h', dependencies: [jni_dep])
cc.has_header('jni.h', dependencies: [jni_dep], required: true)
# Assert that the platform-specific include directory is included in the compiler arguments.
cc.has_header('jni_md.h', dependencies: [jni_dep])
cc.has_header('jni_md.h', dependencies: [jni_dep], required: true)

# generate native headers
subdir('src')
Expand Down

0 comments on commit 3156bb5

Please sign in to comment.