[vim/vim] compiler(rime): add support (PR #14460)

19 views
Skip to first unread message

wzy

unread,
Apr 9, 2024, 9:06:07 AM4/9/24
to vim/vim, Subscribed

You can view, comment on, or merge this pull request online at:

  https://github.com/vim/vim/pull/14460

Commit Summary

  • c25220a compiler(rime): add support

File Changes

(2 files)

Patch Links:


Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/14460@github.com>

Christian Brabandt

unread,
Apr 9, 2024, 3:57:09 PM4/9/24
to vim/vim, Subscribed

@chrisbra commented on this pull request.


In runtime/compiler/rime_deployer.vim:

> +
+" Android Termux
+let s:prefix = getenv('PREFIX')
+if s:prefix == v:null
+  let s:prefix = '/usr'
+endif
+" Android, NixOS, GNU/Linux, BSD
+for s:shared_data_dir in ['/sdcard/rime-data', '/run/current-system/sw/share/rime-data', '/usr/local/share/rime-data', s:prefix . '/share/rime-data']
+  if isdirectory(s:shared_data_dir)
+    break
+  endif
+endfor
+execute 'CompilerSet makeprg=rime_deployer\ --build\ %:p:h\' s:shared_data_dir
+
+let &cpoptions = s:save_cpoptions
+unlet s:save_cpoptions

unlet s:prefix s:shared_data_dir


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/14460/review/1990041897@github.com>

wzy

unread,
Apr 10, 2024, 12:57:10 AM4/10/24
to vim/vim, Subscribed

@Freed-Wu commented on this pull request.


In runtime/compiler/rime_deployer.vim:

> +
+" Android Termux
+let s:prefix = getenv('PREFIX')
+if s:prefix == v:null
+  let s:prefix = '/usr'
+endif
+" Android, NixOS, GNU/Linux, BSD
+for s:shared_data_dir in ['/sdcard/rime-data', '/run/current-system/sw/share/rime-data', '/usr/local/share/rime-data', s:prefix . '/share/rime-data']
+  if isdirectory(s:shared_data_dir)
+    break
+  endif
+endfor
+execute 'CompilerSet makeprg=rime_deployer\ --build\ %:p:h\' s:shared_data_dir
+
+let &cpoptions = s:save_cpoptions
+unlet s:save_cpoptions

Fixed


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/14460/review/1990793805@github.com>

wzy

unread,
Apr 10, 2024, 12:57:34 AM4/10/24
to vim/vim, Push

@Freed-Wu pushed 1 commit.

  • 89eacdb compiler(rime): add support


View it on GitHub.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/14460/push/17929196058@github.com>

Enno

unread,
Apr 11, 2024, 5:45:27 PM4/11/24
to vim/vim, Subscribed

@Konfekt commented on this pull request.


In runtime/compiler/rime_deployer.vim:

> +
+let s:save_cpoptions = &cpoptions
+set cpoptions&vim
+
+" Android Termux
+let s:prefix = getenv('PREFIX')
+if s:prefix == v:null
+  let s:prefix = '/usr'
+endif
+" Android, NixOS, GNU/Linux, BSD
+for s:shared_data_dir in ['/sdcard/rime-data', '/run/current-system/sw/share/rime-data', '/usr/local/share/rime-data', s:prefix . '/share/rime-data']
+  if isdirectory(s:shared_data_dir)
+    break
+  endif
+endfor
+execute 'CompilerSet makeprg=rime_deployer\ --build\ %:p:h\' s:shared_data_dir

Likely %:p:h:S is more robust


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/14460/review/1995536950@github.com>

wzy

unread,
Apr 11, 2024, 11:37:51 PM4/11/24
to vim/vim, Subscribed

@Freed-Wu commented on this pull request.


In runtime/compiler/rime_deployer.vim:

> +
+let s:save_cpoptions = &cpoptions
+set cpoptions&vim
+
+" Android Termux
+let s:prefix = getenv('PREFIX')
+if s:prefix == v:null
+  let s:prefix = '/usr'
+endif
+" Android, NixOS, GNU/Linux, BSD
+for s:shared_data_dir in ['/sdcard/rime-data', '/run/current-system/sw/share/rime-data', '/usr/local/share/rime-data', s:prefix . '/share/rime-data']
+  if isdirectory(s:shared_data_dir)
+    break
+  endif
+endfor
+execute 'CompilerSet makeprg=rime_deployer\ --build\ %:p:h\' s:shared_data_dir

Done.


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/14460/review/1995810300@github.com>

wzy

unread,
Apr 11, 2024, 11:38:04 PM4/11/24
to vim/vim, Push

@Freed-Wu pushed 1 commit.

  • 8652f7e compiler(rime): add support


View it on GitHub.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/14460/push/17963406960@github.com>

wzy

unread,
Apr 12, 2024, 3:51:47 AM4/12/24
to vim/vim, Push

@Freed-Wu pushed 1 commit.

  • b3899f6 compiler(rime): add support


View it on GitHub.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/14460/push/17965845456@github.com>

wzy

unread,
Apr 12, 2024, 2:30:39 PM4/12/24
to vim/vim, Subscribed

Rebased.


Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/14460/c2052263100@github.com>

Christian Brabandt

unread,
Apr 14, 2024, 2:29:20 PM4/14/24
to vim/vim, Subscribed

Merged #14460 into master.


Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/14460/issue_event/12460998523@github.com>

Reply all
Reply to author
Forward
0 new messages