* [bug#43539] [PATCH] gnu: vim: Automatically find vim plugins. @ 2020-09-20 19:15 Efraim Flashner 2020-09-20 19:29 ` Ricardo Wurmus 0 siblings, 1 reply; 10+ messages in thread From: Efraim Flashner @ 2020-09-20 19:15 UTC (permalink / raw) To: 43539; +Cc: Efraim Flashner * gnu/packages/vim.scm (vim)[arguments]: Add new 'install-guix.vim phase to install vendor specific vimrc. * gnu/packages/aux-files/guix.vim: New file. * Makefile.am (AUX_FILES): Register it. --- Makefile.am | 1 + gnu/packages/aux-files/guix.vim | 7 +++++++ gnu/packages/vim.scm | 10 +++++++++- 3 files changed, 17 insertions(+), 1 deletion(-) create mode 100644 gnu/packages/aux-files/guix.vim diff --git a/Makefile.am b/Makefile.am index 8e91e1e558..d0867ecd22 100644 --- a/Makefile.am +++ b/Makefile.am @@ -332,6 +332,7 @@ dist_noinst_DATA = \ AUX_FILES = \ gnu/packages/aux-files/chromium/master-preferences.json \ gnu/packages/aux-files/emacs/guix-emacs.el \ + gnu/packages/aux-files/guix.vim \ gnu/packages/aux-files/linux-libre/5.8-arm.conf \ gnu/packages/aux-files/linux-libre/5.8-arm64.conf \ gnu/packages/aux-files/linux-libre/5.8-i686.conf \ diff --git a/gnu/packages/aux-files/guix.vim b/gnu/packages/aux-files/guix.vim new file mode 100644 index 0000000000..ca97f451d8 --- /dev/null +++ b/gnu/packages/aux-files/guix.vim @@ -0,0 +1,7 @@ +" This appends all of the vim plugins to the end of Vim's runtimepath. +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_ENVIRONMENT] + let testpath = directory . "/share/vim/vimfiles" + if isdirectory(testpath) + let &rtp = join([&rtp,testpath], ',') + endif +endfor diff --git a/gnu/packages/vim.scm b/gnu/packages/vim.scm index c6b1092e71..dba5b5a38e 100644 --- a/gnu/packages/vim.scm +++ b/gnu/packages/vim.scm @@ -136,7 +136,14 @@ (substitute* "src/testdir/test_popupwin.vim" ((".*Test_popup_drag_termwin.*" line) (string-append line "return\n"))) - #t))))) + #t)) + (add-after 'install 'install-guix.vim + (lambda* (#:key inputs outputs #:allow-other-keys) + (let ((vimdir (string-append (assoc-ref outputs "out") "/share/vim"))) + (mkdir-p vimdir) + (copy-file (assoc-ref inputs "guix.vim") + (string-append vimdir "/vimrc")) + #t)))))) (inputs `(("gawk" ,gawk) ("ncurses" ,ncurses) @@ -144,6 +151,7 @@ ("tcsh" ,tcsh))) ; For runtime/tools/vim32 (native-inputs `(("libtool" ,libtool) + ("guix.vim" ,(search-auxiliary-file "guix.vim")) ;; For tests. ("tzdata" ,tzdata-for-tests))) -- 2.28.0 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [bug#43539] [PATCH] gnu: vim: Automatically find vim plugins. 2020-09-20 19:15 [bug#43539] [PATCH] gnu: vim: Automatically find vim plugins Efraim Flashner @ 2020-09-20 19:29 ` Ricardo Wurmus 2020-09-20 19:32 ` Efraim Flashner 0 siblings, 1 reply; 10+ messages in thread From: Ricardo Wurmus @ 2020-09-20 19:29 UTC (permalink / raw) To: Efraim Flashner; +Cc: 43539 Efraim Flashner <efraim@flashner.co.il> writes: > diff --git a/gnu/packages/aux-files/guix.vim b/gnu/packages/aux-files/guix.vim > new file mode 100644 > index 0000000000..ca97f451d8 > --- /dev/null > +++ b/gnu/packages/aux-files/guix.vim > @@ -0,0 +1,7 @@ > +" This appends all of the vim plugins to the end of Vim's runtimepath. > +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_ENVIRONMENT] How about using $GUIX_PROFILE as well or instead of $HOME/.guix-profile? This would allow users to keep a separate profile for Vim things. -- Ricardo ^ permalink raw reply [flat|nested] 10+ messages in thread
* [bug#43539] [PATCH] gnu: vim: Automatically find vim plugins. 2020-09-20 19:29 ` Ricardo Wurmus @ 2020-09-20 19:32 ` Efraim Flashner 2020-09-20 19:50 ` Ricardo Wurmus 0 siblings, 1 reply; 10+ messages in thread From: Efraim Flashner @ 2020-09-20 19:32 UTC (permalink / raw) To: Ricardo Wurmus; +Cc: 43539 [-- Attachment #1: Type: text/plain, Size: 1088 bytes --] On Sun, Sep 20, 2020 at 09:29:27PM +0200, Ricardo Wurmus wrote: > > Efraim Flashner <efraim@flashner.co.il> writes: > > > diff --git a/gnu/packages/aux-files/guix.vim b/gnu/packages/aux-files/guix.vim > > new file mode 100644 > > index 0000000000..ca97f451d8 > > --- /dev/null > > +++ b/gnu/packages/aux-files/guix.vim > > @@ -0,0 +1,7 @@ > > +" This appends all of the vim plugins to the end of Vim's runtimepath. > > +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_ENVIRONMENT] > > How about using $GUIX_PROFILE as well or instead of $HOME/.guix-profile? > This would allow users to keep a separate profile for Vim things. That was the one part I didn't really like about my patch. Unfortunately I don't seem to have $GUIX_PROFILE set on my machine. (ins)efraim@E5400 ~$ echo $GUIX_PROFILE (ins)efraim@E5400 ~$ -- Efraim Flashner <efraim@flashner.co.il> אפרים פלשנר GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351 Confidentiality cannot be guaranteed on emails sent or received unencrypted [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* [bug#43539] [PATCH] gnu: vim: Automatically find vim plugins. 2020-09-20 19:32 ` Efraim Flashner @ 2020-09-20 19:50 ` Ricardo Wurmus 2020-09-20 20:57 ` Efraim Flashner 0 siblings, 1 reply; 10+ messages in thread From: Ricardo Wurmus @ 2020-09-20 19:50 UTC (permalink / raw) To: Efraim Flashner; +Cc: 43539 Efraim Flashner <efraim@flashner.co.il> writes: > On Sun, Sep 20, 2020 at 09:29:27PM +0200, Ricardo Wurmus wrote: >> >> Efraim Flashner <efraim@flashner.co.il> writes: >> >> > diff --git a/gnu/packages/aux-files/guix.vim b/gnu/packages/aux-files/guix.vim >> > new file mode 100644 >> > index 0000000000..ca97f451d8 >> > --- /dev/null >> > +++ b/gnu/packages/aux-files/guix.vim >> > @@ -0,0 +1,7 @@ >> > +" This appends all of the vim plugins to the end of Vim's runtimepath. >> > +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_ENVIRONMENT] >> >> How about using $GUIX_PROFILE as well or instead of $HOME/.guix-profile? >> This would allow users to keep a separate profile for Vim things. > > That was the one part I didn't really like about my patch. Unfortunately > I don't seem to have $GUIX_PROFILE set on my machine. > > (ins)efraim@E5400 ~$ echo $GUIX_PROFILE > > (ins)efraim@E5400 ~$ Me neither! But when loading profiles we suggest this approach: export GUIX_PROFILE=/the/location/of/the/.guix-profile source $GUIX_PROFILE/etc/profile The variable is set in that case, because that affects whether absolute store file names are used or the symlink “/the/location/of/the/.guix-profile”. -- Ricardo ^ permalink raw reply [flat|nested] 10+ messages in thread
* [bug#43539] [PATCH] gnu: vim: Automatically find vim plugins. 2020-09-20 19:50 ` Ricardo Wurmus @ 2020-09-20 20:57 ` Efraim Flashner 2020-09-22 9:37 ` Efraim Flashner 0 siblings, 1 reply; 10+ messages in thread From: Efraim Flashner @ 2020-09-20 20:57 UTC (permalink / raw) To: Ricardo Wurmus; +Cc: 43539 [-- Attachment #1: Type: text/plain, Size: 1795 bytes --] On Sun, Sep 20, 2020 at 09:50:12PM +0200, Ricardo Wurmus wrote: > > Efraim Flashner <efraim@flashner.co.il> writes: > > > On Sun, Sep 20, 2020 at 09:29:27PM +0200, Ricardo Wurmus wrote: > >> > >> Efraim Flashner <efraim@flashner.co.il> writes: > >> > >> > diff --git a/gnu/packages/aux-files/guix.vim b/gnu/packages/aux-files/guix.vim > >> > new file mode 100644 > >> > index 0000000000..ca97f451d8 > >> > --- /dev/null > >> > +++ b/gnu/packages/aux-files/guix.vim > >> > @@ -0,0 +1,7 @@ > >> > +" This appends all of the vim plugins to the end of Vim's runtimepath. > >> > +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_ENVIRONMENT] > >> > >> How about using $GUIX_PROFILE as well or instead of $HOME/.guix-profile? > >> This would allow users to keep a separate profile for Vim things. > > > > That was the one part I didn't really like about my patch. Unfortunately > > I don't seem to have $GUIX_PROFILE set on my machine. > > > > (ins)efraim@E5400 ~$ echo $GUIX_PROFILE > > > > (ins)efraim@E5400 ~$ > > Me neither! But when loading profiles we suggest this approach: > > export GUIX_PROFILE=/the/location/of/the/.guix-profile > source $GUIX_PROFILE/etc/profile > > The variable is set in that case, because that affects whether absolute > store file names are used or the symlink > “/the/location/of/the/.guix-profile”. It seems to be the case for the default profile. And in Guix System. I could also just add $GUIX_PROFILE as another location in the list of directories to search. -- Efraim Flashner <efraim@flashner.co.il> אפרים פלשנר GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351 Confidentiality cannot be guaranteed on emails sent or received unencrypted [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* [bug#43539] [PATCH] gnu: vim: Automatically find vim plugins. 2020-09-20 20:57 ` Efraim Flashner @ 2020-09-22 9:37 ` Efraim Flashner 2020-09-25 17:48 ` Marius Bakke 0 siblings, 1 reply; 10+ messages in thread From: Efraim Flashner @ 2020-09-22 9:37 UTC (permalink / raw) To: Ricardo Wurmus; +Cc: 43539 [-- Attachment #1.1: Type: text/plain, Size: 2421 bytes --] On Sun, Sep 20, 2020 at 11:57:58PM +0300, Efraim Flashner wrote: > On Sun, Sep 20, 2020 at 09:50:12PM +0200, Ricardo Wurmus wrote: > > > > Efraim Flashner <efraim@flashner.co.il> writes: > > > > > On Sun, Sep 20, 2020 at 09:29:27PM +0200, Ricardo Wurmus wrote: > > >> > > >> Efraim Flashner <efraim@flashner.co.il> writes: > > >> > > >> > diff --git a/gnu/packages/aux-files/guix.vim b/gnu/packages/aux-files/guix.vim > > >> > new file mode 100644 > > >> > index 0000000000..ca97f451d8 > > >> > --- /dev/null > > >> > +++ b/gnu/packages/aux-files/guix.vim > > >> > @@ -0,0 +1,7 @@ > > >> > +" This appends all of the vim plugins to the end of Vim's runtimepath. > > >> > +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_ENVIRONMENT] > > >> > > >> How about using $GUIX_PROFILE as well or instead of $HOME/.guix-profile? > > >> This would allow users to keep a separate profile for Vim things. > > > > > > That was the one part I didn't really like about my patch. Unfortunately > > > I don't seem to have $GUIX_PROFILE set on my machine. > > > > > > (ins)efraim@E5400 ~$ echo $GUIX_PROFILE > > > > > > (ins)efraim@E5400 ~$ > > > > Me neither! But when loading profiles we suggest this approach: > > > > export GUIX_PROFILE=/the/location/of/the/.guix-profile > > source $GUIX_PROFILE/etc/profile > > > > The variable is set in that case, because that affects whether absolute > > store file names are used or the symlink > > “/the/location/of/the/.guix-profile”. > > It seems to be the case for the default profile. And in Guix System. I > could also just add $GUIX_PROFILE as another location in the list of > directories to search. The more I think about it the more I see $HOME/.guix-profile as a "special profile" which is nearly always included in $PATH and other variables, similar to /run/current-system/profile. With that in mind I like the attached patch which keeps that path but adds a search for $GUIX_PROFILE. I could use environment variables to override vim's builtin search for $VIM and $VIMRUNTIME, but then we would have to decide to support vim installed by the user or system-wide. -- Efraim Flashner <efraim@flashner.co.il> אפרים פלשנר GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351 Confidentiality cannot be guaranteed on emails sent or received unencrypted [-- Attachment #1.2: 0001-gnu-vim-Automatically-find-vim-plugins.patch --] [-- Type: text/plain, Size: 2835 bytes --] From 89a465c32a1d75a473cb643c49bbf65c093b1d4e Mon Sep 17 00:00:00 2001 From: Efraim Flashner <efraim@flashner.co.il> Date: Fri, 18 Sep 2020 18:25:06 +0300 Subject: [PATCH] gnu: vim: Automatically find vim plugins. * gnu/packages/vim.scm (vim)[arguments]: Add new 'install-guix.vim phase to install vendor specific vimrc. * gnu/packages/aux-files/guix.vim: New file. * Makefile.am (AUX_FILES): Register it. --- Makefile.am | 1 + gnu/packages/aux-files/guix.vim | 7 +++++++ gnu/packages/vim.scm | 10 +++++++++- 3 files changed, 17 insertions(+), 1 deletion(-) create mode 100644 gnu/packages/aux-files/guix.vim diff --git a/Makefile.am b/Makefile.am index 8e91e1e558..d0867ecd22 100644 --- a/Makefile.am +++ b/Makefile.am @@ -332,6 +332,7 @@ dist_noinst_DATA = \ AUX_FILES = \ gnu/packages/aux-files/chromium/master-preferences.json \ gnu/packages/aux-files/emacs/guix-emacs.el \ + gnu/packages/aux-files/guix.vim \ gnu/packages/aux-files/linux-libre/5.8-arm.conf \ gnu/packages/aux-files/linux-libre/5.8-arm64.conf \ gnu/packages/aux-files/linux-libre/5.8-i686.conf \ diff --git a/gnu/packages/aux-files/guix.vim b/gnu/packages/aux-files/guix.vim new file mode 100644 index 0000000000..7dc359e52b --- /dev/null +++ b/gnu/packages/aux-files/guix.vim @@ -0,0 +1,7 @@ +" This appends all of the vim plugins to the end of Vim's runtimepath. +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_PROFILE, $GUIX_ENVIRONMENT] + let vimplugins = directory . "/share/vim/vimfiles" + if isdirectory(vimplugins) + let &rtp = join([&rtp,vimplugins], ',') + endif +endfor diff --git a/gnu/packages/vim.scm b/gnu/packages/vim.scm index c6b1092e71..dba5b5a38e 100644 --- a/gnu/packages/vim.scm +++ b/gnu/packages/vim.scm @@ -136,7 +136,14 @@ (substitute* "src/testdir/test_popupwin.vim" ((".*Test_popup_drag_termwin.*" line) (string-append line "return\n"))) - #t))))) + #t)) + (add-after 'install 'install-guix.vim + (lambda* (#:key inputs outputs #:allow-other-keys) + (let ((vimdir (string-append (assoc-ref outputs "out") "/share/vim"))) + (mkdir-p vimdir) + (copy-file (assoc-ref inputs "guix.vim") + (string-append vimdir "/vimrc")) + #t)))))) (inputs `(("gawk" ,gawk) ("ncurses" ,ncurses) @@ -144,6 +151,7 @@ ("tcsh" ,tcsh))) ; For runtime/tools/vim32 (native-inputs `(("libtool" ,libtool) + ("guix.vim" ,(search-auxiliary-file "guix.vim")) ;; For tests. ("tzdata" ,tzdata-for-tests))) -- 2.28.0 [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [bug#43539] [PATCH] gnu: vim: Automatically find vim plugins. 2020-09-22 9:37 ` Efraim Flashner @ 2020-09-25 17:48 ` Marius Bakke 2020-09-27 8:49 ` Efraim Flashner 2020-12-20 11:10 ` bug#43539: " Efraim Flashner 0 siblings, 2 replies; 10+ messages in thread From: Marius Bakke @ 2020-09-25 17:48 UTC (permalink / raw) To: Efraim Flashner, Ricardo Wurmus; +Cc: 43539 [-- Attachment #1: Type: text/plain, Size: 3369 bytes --] Efraim Flashner <efraim@flashner.co.il> writes: > On Sun, Sep 20, 2020 at 11:57:58PM +0300, Efraim Flashner wrote: >> On Sun, Sep 20, 2020 at 09:50:12PM +0200, Ricardo Wurmus wrote: >> > >> > Efraim Flashner <efraim@flashner.co.il> writes: >> > >> > > On Sun, Sep 20, 2020 at 09:29:27PM +0200, Ricardo Wurmus wrote: >> > >> >> > >> Efraim Flashner <efraim@flashner.co.il> writes: >> > >> >> > >> > diff --git a/gnu/packages/aux-files/guix.vim b/gnu/packages/aux-files/guix.vim >> > >> > new file mode 100644 >> > >> > index 0000000000..ca97f451d8 >> > >> > --- /dev/null >> > >> > +++ b/gnu/packages/aux-files/guix.vim >> > >> > @@ -0,0 +1,7 @@ >> > >> > +" This appends all of the vim plugins to the end of Vim's runtimepath. >> > >> > +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_ENVIRONMENT] >> > >> >> > >> How about using $GUIX_PROFILE as well or instead of $HOME/.guix-profile? >> > >> This would allow users to keep a separate profile for Vim things. >> > > >> > > That was the one part I didn't really like about my patch. Unfortunately >> > > I don't seem to have $GUIX_PROFILE set on my machine. >> > > >> > > (ins)efraim@E5400 ~$ echo $GUIX_PROFILE >> > > >> > > (ins)efraim@E5400 ~$ >> > >> > Me neither! But when loading profiles we suggest this approach: >> > >> > export GUIX_PROFILE=/the/location/of/the/.guix-profile >> > source $GUIX_PROFILE/etc/profile >> > >> > The variable is set in that case, because that affects whether absolute >> > store file names are used or the symlink >> > “/the/location/of/the/.guix-profile”. >> >> It seems to be the case for the default profile. And in Guix System. I >> could also just add $GUIX_PROFILE as another location in the list of >> directories to search. > > The more I think about it the more I see $HOME/.guix-profile as a > "special profile" which is nearly always included in $PATH and other > variables, similar to /run/current-system/profile. With that in mind I > like the attached patch which keeps that path but adds a search for > $GUIX_PROFILE. > > I could use environment variables to override vim's builtin search for > $VIM and $VIMRUNTIME, but then we would have to decide to support vim > installed by the user or system-wide. So vim only expects a single vimfiles directory in those variables? [...] > +++ b/gnu/packages/aux-files/guix.vim > @@ -0,0 +1,7 @@ > +" This appends all of the vim plugins to the end of Vim's runtimepath. > +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_PROFILE, $GUIX_ENVIRONMENT] > + let vimplugins = directory . "/share/vim/vimfiles" > + if isdirectory(vimplugins) > + let &rtp = join([&rtp,vimplugins], ',') > + endif > +endfor I wonder if we could generalize this to a proper search path with something like (pseudo code): for directory in getenv("VIM_PLUGIN_PATH").split(":") let vimplugins = directory . "share/vim/vimfiles" ... And then in the vim package: (native-search-paths (list (search-path-specification (variable "VIM_PLUGIN_PATH") (files '("share/vim/vimfiles"))))) ...which makes Guix populate VIM_PLUGIN_PATH automatically as plugins are added to a profile/environment (as long as it contains the 'vim' package). WDYT? [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 487 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* [bug#43539] [PATCH] gnu: vim: Automatically find vim plugins. 2020-09-25 17:48 ` Marius Bakke @ 2020-09-27 8:49 ` Efraim Flashner 2020-09-27 9:41 ` Efraim Flashner 2020-12-20 11:10 ` bug#43539: " Efraim Flashner 1 sibling, 1 reply; 10+ messages in thread From: Efraim Flashner @ 2020-09-27 8:49 UTC (permalink / raw) To: Marius Bakke; +Cc: Ricardo Wurmus, 43539 [-- Attachment #1: Type: text/plain, Size: 4685 bytes --] On Fri, Sep 25, 2020 at 07:48:34PM +0200, Marius Bakke wrote: > Efraim Flashner <efraim@flashner.co.il> writes: > > > On Sun, Sep 20, 2020 at 11:57:58PM +0300, Efraim Flashner wrote: > >> On Sun, Sep 20, 2020 at 09:50:12PM +0200, Ricardo Wurmus wrote: > >> > > >> > Efraim Flashner <efraim@flashner.co.il> writes: > >> > > >> > > On Sun, Sep 20, 2020 at 09:29:27PM +0200, Ricardo Wurmus wrote: > >> > >> > >> > >> Efraim Flashner <efraim@flashner.co.il> writes: > >> > >> > >> > >> > diff --git a/gnu/packages/aux-files/guix.vim b/gnu/packages/aux-files/guix.vim > >> > >> > new file mode 100644 > >> > >> > index 0000000000..ca97f451d8 > >> > >> > --- /dev/null > >> > >> > +++ b/gnu/packages/aux-files/guix.vim > >> > >> > @@ -0,0 +1,7 @@ > >> > >> > +" This appends all of the vim plugins to the end of Vim's runtimepath. > >> > >> > +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_ENVIRONMENT] > >> > >> > >> > >> How about using $GUIX_PROFILE as well or instead of $HOME/.guix-profile? > >> > >> This would allow users to keep a separate profile for Vim things. > >> > > > >> > > That was the one part I didn't really like about my patch. Unfortunately > >> > > I don't seem to have $GUIX_PROFILE set on my machine. > >> > > > >> > > (ins)efraim@E5400 ~$ echo $GUIX_PROFILE > >> > > > >> > > (ins)efraim@E5400 ~$ > >> > > >> > Me neither! But when loading profiles we suggest this approach: > >> > > >> > export GUIX_PROFILE=/the/location/of/the/.guix-profile > >> > source $GUIX_PROFILE/etc/profile > >> > > >> > The variable is set in that case, because that affects whether absolute > >> > store file names are used or the symlink > >> > “/the/location/of/the/.guix-profile”. > >> > >> It seems to be the case for the default profile. And in Guix System. I > >> could also just add $GUIX_PROFILE as another location in the list of > >> directories to search. > > > > The more I think about it the more I see $HOME/.guix-profile as a > > "special profile" which is nearly always included in $PATH and other > > variables, similar to /run/current-system/profile. With that in mind I > > like the attached patch which keeps that path but adds a search for > > $GUIX_PROFILE. > > > > I could use environment variables to override vim's builtin search for > > $VIM and $VIMRUNTIME, but then we would have to decide to support vim > > installed by the user or system-wide. > > So vim only expects a single vimfiles directory in those variables? > Not exactly. vim sets both $VIM and $VIMRUNTIME. $VIM points to $out/share/vim and $VIMRUNTIME points to $out/share/vim/vim82. In a typical FHS distro this would be /etc/share/vim and /etc/share/vim/vim82. Then any distro specific vimrc would be installed to /etc/share/vim/vimrc, which would also help load any distro installed plugins in /etc/share/vim/vimfiles. Any user installed plugins would be (typically) in ~/.vim/plugins/ with some extra directories for management or for the 'after' keyword to load last or 'opt' to only load on command. Or in a 'pack' path to be activated with :packadd {plugin}. > [...] > > > +++ b/gnu/packages/aux-files/guix.vim > > @@ -0,0 +1,7 @@ > > +" This appends all of the vim plugins to the end of Vim's runtimepath. > > +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_PROFILE, $GUIX_ENVIRONMENT] > > + let vimplugins = directory . "/share/vim/vimfiles" > > + if isdirectory(vimplugins) > > + let &rtp = join([&rtp,vimplugins], ',') > > + endif > > +endfor > > I wonder if we could generalize this to a proper search path with > something like (pseudo code): > > for directory in getenv("VIM_PLUGIN_PATH").split(":") > let vimplugins = directory . "share/vim/vimfiles" > ... > > And then in the vim package: > > (native-search-paths > (list > (search-path-specification > (variable "VIM_PLUGIN_PATH") > (files '("share/vim/vimfiles"))))) > > ...which makes Guix populate VIM_PLUGIN_PATH automatically as plugins > are added to a profile/environment (as long as it contains the 'vim' > package). > > WDYT? I'll try putting something together like that. Rebuilding vim takes about 3 minutes when I disable tests so it shouldn't be hard. I can also experiment a bit with packpath and/or per-plugin paths in share/vim/vimfiles/<package-name>. -- Efraim Flashner <efraim@flashner.co.il> אפרים פלשנר GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351 Confidentiality cannot be guaranteed on emails sent or received unencrypted [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* [bug#43539] [PATCH] gnu: vim: Automatically find vim plugins. 2020-09-27 8:49 ` Efraim Flashner @ 2020-09-27 9:41 ` Efraim Flashner 0 siblings, 0 replies; 10+ messages in thread From: Efraim Flashner @ 2020-09-27 9:41 UTC (permalink / raw) To: Marius Bakke; +Cc: Ricardo Wurmus, 43539 [-- Attachment #1: Type: text/plain, Size: 5247 bytes --] On Sun, Sep 27, 2020 at 11:49:51AM +0300, Efraim Flashner wrote: > On Fri, Sep 25, 2020 at 07:48:34PM +0200, Marius Bakke wrote: > > Efraim Flashner <efraim@flashner.co.il> writes: > > > > > On Sun, Sep 20, 2020 at 11:57:58PM +0300, Efraim Flashner wrote: > > >> On Sun, Sep 20, 2020 at 09:50:12PM +0200, Ricardo Wurmus wrote: > > >> > > > >> > Efraim Flashner <efraim@flashner.co.il> writes: > > >> > > > >> > > On Sun, Sep 20, 2020 at 09:29:27PM +0200, Ricardo Wurmus wrote: > > >> > >> > > >> > >> Efraim Flashner <efraim@flashner.co.il> writes: > > >> > >> > > >> > >> > diff --git a/gnu/packages/aux-files/guix.vim b/gnu/packages/aux-files/guix.vim > > >> > >> > new file mode 100644 > > >> > >> > index 0000000000..ca97f451d8 > > >> > >> > --- /dev/null > > >> > >> > +++ b/gnu/packages/aux-files/guix.vim > > >> > >> > @@ -0,0 +1,7 @@ > > >> > >> > +" This appends all of the vim plugins to the end of Vim's runtimepath. > > >> > >> > +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_ENVIRONMENT] > > >> > >> > > >> > >> How about using $GUIX_PROFILE as well or instead of $HOME/.guix-profile? > > >> > >> This would allow users to keep a separate profile for Vim things. > > >> > > > > >> > > That was the one part I didn't really like about my patch. Unfortunately > > >> > > I don't seem to have $GUIX_PROFILE set on my machine. > > >> > > > > >> > > (ins)efraim@E5400 ~$ echo $GUIX_PROFILE > > >> > > > > >> > > (ins)efraim@E5400 ~$ > > >> > > > >> > Me neither! But when loading profiles we suggest this approach: > > >> > > > >> > export GUIX_PROFILE=/the/location/of/the/.guix-profile > > >> > source $GUIX_PROFILE/etc/profile > > >> > > > >> > The variable is set in that case, because that affects whether absolute > > >> > store file names are used or the symlink > > >> > “/the/location/of/the/.guix-profile”. > > >> > > >> It seems to be the case for the default profile. And in Guix System. I > > >> could also just add $GUIX_PROFILE as another location in the list of > > >> directories to search. > > > > > > The more I think about it the more I see $HOME/.guix-profile as a > > > "special profile" which is nearly always included in $PATH and other > > > variables, similar to /run/current-system/profile. With that in mind I > > > like the attached patch which keeps that path but adds a search for > > > $GUIX_PROFILE. > > > > > > I could use environment variables to override vim's builtin search for > > > $VIM and $VIMRUNTIME, but then we would have to decide to support vim > > > installed by the user or system-wide. > > > > So vim only expects a single vimfiles directory in those variables? > > > > Not exactly. vim sets both $VIM and $VIMRUNTIME. $VIM points to > $out/share/vim and $VIMRUNTIME points to $out/share/vim/vim82. In a > typical FHS distro this would be /etc/share/vim and > /etc/share/vim/vim82. Then any distro specific vimrc would be installed > to /etc/share/vim/vimrc, which would also help load any distro installed > plugins in /etc/share/vim/vimfiles. Any user installed plugins would be > (typically) in ~/.vim/plugins/ with some extra directories for > management or for the 'after' keyword to load last or 'opt' to only load > on command. Or in a 'pack' path to be activated with :packadd {plugin}. > ... Actually replace all of these 'etc' with 'usr'. > > [...] > > > > > +++ b/gnu/packages/aux-files/guix.vim > > > @@ -0,0 +1,7 @@ > > > +" This appends all of the vim plugins to the end of Vim's runtimepath. > > > +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_PROFILE, $GUIX_ENVIRONMENT] > > > + let vimplugins = directory . "/share/vim/vimfiles" > > > + if isdirectory(vimplugins) > > > + let &rtp = join([&rtp,vimplugins], ',') > > > + endif > > > +endfor > > > > I wonder if we could generalize this to a proper search path with > > something like (pseudo code): > > > > for directory in getenv("VIM_PLUGIN_PATH").split(":") > > let vimplugins = directory . "share/vim/vimfiles" > > ... > > > > And then in the vim package: > > > > (native-search-paths > > (list > > (search-path-specification > > (variable "VIM_PLUGIN_PATH") > > (files '("share/vim/vimfiles"))))) > > > > ...which makes Guix populate VIM_PLUGIN_PATH automatically as plugins > > are added to a profile/environment (as long as it contains the 'vim' > > package). > > > > WDYT? > > I'll try putting something together like that. Rebuilding vim takes > about 3 minutes when I disable tests so it shouldn't be hard. I can also > experiment a bit with packpath and/or per-plugin paths in > share/vim/vimfiles/<package-name>. > > > > -- > Efraim Flashner <efraim@flashner.co.il> אפרים פלשנר > GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351 > Confidentiality cannot be guaranteed on emails sent or received unencrypted -- Efraim Flashner <efraim@flashner.co.il> אפרים פלשנר GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351 Confidentiality cannot be guaranteed on emails sent or received unencrypted [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#43539: [PATCH] gnu: vim: Automatically find vim plugins. 2020-09-25 17:48 ` Marius Bakke 2020-09-27 8:49 ` Efraim Flashner @ 2020-12-20 11:10 ` Efraim Flashner 1 sibling, 0 replies; 10+ messages in thread From: Efraim Flashner @ 2020-12-20 11:10 UTC (permalink / raw) To: Marius Bakke; +Cc: 43539-done [-- Attachment #1: Type: text/plain, Size: 4188 bytes --] On Fri, Sep 25, 2020 at 07:48:34PM +0200, Marius Bakke wrote: > Efraim Flashner <efraim@flashner.co.il> writes: > > > On Sun, Sep 20, 2020 at 11:57:58PM +0300, Efraim Flashner wrote: > >> On Sun, Sep 20, 2020 at 09:50:12PM +0200, Ricardo Wurmus wrote: > >> > > >> > Efraim Flashner <efraim@flashner.co.il> writes: > >> > > >> > > On Sun, Sep 20, 2020 at 09:29:27PM +0200, Ricardo Wurmus wrote: > >> > >> > >> > >> Efraim Flashner <efraim@flashner.co.il> writes: > >> > >> > >> > >> > diff --git a/gnu/packages/aux-files/guix.vim b/gnu/packages/aux-files/guix.vim > >> > >> > new file mode 100644 > >> > >> > index 0000000000..ca97f451d8 > >> > >> > --- /dev/null > >> > >> > +++ b/gnu/packages/aux-files/guix.vim > >> > >> > @@ -0,0 +1,7 @@ > >> > >> > +" This appends all of the vim plugins to the end of Vim's runtimepath. > >> > >> > +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_ENVIRONMENT] > >> > >> > >> > >> How about using $GUIX_PROFILE as well or instead of $HOME/.guix-profile? > >> > >> This would allow users to keep a separate profile for Vim things. > >> > > > >> > > That was the one part I didn't really like about my patch. Unfortunately > >> > > I don't seem to have $GUIX_PROFILE set on my machine. > >> > > > >> > > (ins)efraim@E5400 ~$ echo $GUIX_PROFILE > >> > > > >> > > (ins)efraim@E5400 ~$ > >> > > >> > Me neither! But when loading profiles we suggest this approach: > >> > > >> > export GUIX_PROFILE=/the/location/of/the/.guix-profile > >> > source $GUIX_PROFILE/etc/profile > >> > > >> > The variable is set in that case, because that affects whether absolute > >> > store file names are used or the symlink > >> > “/the/location/of/the/.guix-profile”. > >> > >> It seems to be the case for the default profile. And in Guix System. I > >> could also just add $GUIX_PROFILE as another location in the list of > >> directories to search. > > > > The more I think about it the more I see $HOME/.guix-profile as a > > "special profile" which is nearly always included in $PATH and other > > variables, similar to /run/current-system/profile. With that in mind I > > like the attached patch which keeps that path but adds a search for > > $GUIX_PROFILE. > > > > I could use environment variables to override vim's builtin search for > > $VIM and $VIMRUNTIME, but then we would have to decide to support vim > > installed by the user or system-wide. > > So vim only expects a single vimfiles directory in those variables? > > [...] > > > +++ b/gnu/packages/aux-files/guix.vim > > @@ -0,0 +1,7 @@ > > +" This appends all of the vim plugins to the end of Vim's runtimepath. > > +for directory in ["/run/current-system/profile", $HOME . "/.guix-profile", $GUIX_PROFILE, $GUIX_ENVIRONMENT] > > + let vimplugins = directory . "/share/vim/vimfiles" > > + if isdirectory(vimplugins) > > + let &rtp = join([&rtp,vimplugins], ',') > > + endif > > +endfor > > I wonder if we could generalize this to a proper search path with > something like (pseudo code): > > for directory in getenv("VIM_PLUGIN_PATH").split(":") > let vimplugins = directory . "share/vim/vimfiles" > ... Note for the future, 'let vimplugins = directory' is what works here. > > And then in the vim package: > > (native-search-paths > (list > (search-path-specification > (variable "VIM_PLUGIN_PATH") > (files '("share/vim/vimfiles"))))) > > ...which makes Guix populate VIM_PLUGIN_PATH automatically as plugins > are added to a profile/environment (as long as it contains the 'vim' > package). > > WDYT? I wasn't able to get anything like that to work. For some reason the only combination I got using VIM_PLUGIN_PATH to work was a guix-environment with vim and a plugin in it. I pushed the version with the hardcoded list of directories since I was able to actually get that one to work. -- Efraim Flashner <efraim@flashner.co.il> אפרים פלשנר GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351 Confidentiality cannot be guaranteed on emails sent or received unencrypted [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2020-12-20 11:11 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-09-20 19:15 [bug#43539] [PATCH] gnu: vim: Automatically find vim plugins Efraim Flashner 2020-09-20 19:29 ` Ricardo Wurmus 2020-09-20 19:32 ` Efraim Flashner 2020-09-20 19:50 ` Ricardo Wurmus 2020-09-20 20:57 ` Efraim Flashner 2020-09-22 9:37 ` Efraim Flashner 2020-09-25 17:48 ` Marius Bakke 2020-09-27 8:49 ` Efraim Flashner 2020-09-27 9:41 ` Efraim Flashner 2020-12-20 11:10 ` bug#43539: " Efraim Flashner
Code repositories for project(s) associated with this public inbox https://git.savannah.gnu.org/cgit/guix.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).