Skip to content

Commit d05037f

Browse files
committed
Add tests and smoketest
1 parent 0f87187 commit d05037f

2 files changed

Lines changed: 6 additions & 0 deletions

File tree

.github/workflows/macvim-buildtest.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -284,6 +284,11 @@ jobs:
284284
echo 'Found external dynamic linkage!'; false
285285
fi
286286
287+
# Make sure we are not using system iconv, which has been buggy since macOS 14.
288+
if otool -L ${VIM_BIN} | grep '^\s*/usr/lib/libiconv'; then
289+
echo 'Using system iconv! We should be linking against GNU iconv instead.'; false
290+
fi
291+
287292
# Make sure that --disable-sparkle flag will properly exclude all references to Sparkle symbols. This is
288293
# necessary because we still use weak linking to Sparkle when that flag is set and so references to Sparkle
289294
# wouldn't fail the build (we just remove Sparkle.framework from the built app after the fact).

src/testdir/test_macvim.vim

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ func Test_macvim_options_commands_exist()
3535
call assert_true(has('clientserver'), 'Missing feature "clientserver"')
3636
call assert_true(has('clipboard'), 'Missing feature "clipboard"')
3737
call assert_true(has('clipboard_working'), 'Missing feature "clipboard_working"')
38+
call assert_true(has('iconv'), 'Missing feature "iconv"')
3839
call assert_true(has('sound'), 'Missing feature "sound"')
3940
call assert_true(has('terminal'), 'Missing feature "terminal"')
4041
call assert_true(has('xim'), 'Missing feature "xim"')

0 commit comments

Comments
 (0)