]> granicus.if.org Git - vim/commitdiff
patch 8.0.0680: plugins in start packages are sourced twice v8.0.0680
authorBram Moolenaar <Bram@vim.org>
Tue, 27 Jun 2017 12:43:55 +0000 (14:43 +0200)
committerBram Moolenaar <Bram@vim.org>
Tue, 27 Jun 2017 12:43:55 +0000 (14:43 +0200)
Problem:    Plugins in start packages are sourced twice. (mseplowitz)
Solution:   Use the unmodified runtime path when loading plugins (test by Ingo
            Karkat, closes #1801)

src/ex_cmds2.c
src/main.c
src/proto/ex_cmds2.pro
src/testdir/test_startup.vim
src/version.c

index 33ad924d222348e5dab90770e1245e6581effeaa..74dae92b5b0746be5f8e272f53967a4520821b94 100644 (file)
@@ -3285,19 +3285,6 @@ source_callback(char_u *fname, void *cookie UNUSED)
     (void)do_source(fname, FALSE, DOSO_NONE);
 }
 
-/*
- * Source the file "name" from all directories in 'runtimepath'.
- * "name" can contain wildcards.
- * When "flags" has DIP_ALL: source all files, otherwise only the first one.
- *
- * return FAIL when no file could be sourced, OK otherwise.
- */
-    int
-source_runtime(char_u *name, int flags)
-{
-    return do_in_runtimepath(name, flags, source_callback, NULL);
-}
-
 /*
  * Find the file "name" in all directories in "path" and invoke
  * "callback(fname, cookie)".
@@ -3435,18 +3422,19 @@ do_in_path(
 }
 
 /*
- * Find "name" in 'runtimepath'.  When found, invoke the callback function for
+ * Find "name" in "path".  When found, invoke the callback function for
  * it: callback(fname, "cookie")
  * When "flags" has DIP_ALL repeat for all matches, otherwise only the first
  * one is used.
  * Returns OK when at least one match found, FAIL otherwise.
  *
- * If "name" is NULL calls callback for each entry in runtimepath. Cookie is
+ * If "name" is NULL calls callback for each entry in "path". Cookie is
  * passed by reference in this case, setting it to NULL indicates that callback
  * has done its job.
  */
-    int
-do_in_runtimepath(
+    static int
+do_in_path_and_pp(
+    char_u     *path,
     char_u     *name,
     int                flags,
     void       (*callback)(char_u *fname, void *ck),
@@ -3459,7 +3447,7 @@ do_in_runtimepath(
     char       *opt_dir = "pack/*/opt/*/%s";
 
     if ((flags & DIP_NORTP) == 0)
-       done = do_in_path(p_rtp, name, flags, callback, cookie);
+       done = do_in_path(path, name, flags, callback, cookie);
 
     if ((done == FAIL || (flags & DIP_ALL)) && (flags & DIP_START))
     {
@@ -3486,6 +3474,42 @@ do_in_runtimepath(
     return done;
 }
 
+/*
+ * Just like do_in_path_and_pp(), using 'runtimepath' for "path".
+ */
+    int
+do_in_runtimepath(
+    char_u     *name,
+    int                flags,
+    void       (*callback)(char_u *fname, void *ck),
+    void       *cookie)
+{
+    return do_in_path_and_pp(p_rtp, name, flags, callback, cookie);
+}
+
+/*
+ * Source the file "name" from all directories in 'runtimepath'.
+ * "name" can contain wildcards.
+ * When "flags" has DIP_ALL: source all files, otherwise only the first one.
+ *
+ * return FAIL when no file could be sourced, OK otherwise.
+ */
+    int
+source_runtime(char_u *name, int flags)
+{
+    return source_in_path(p_rtp, name, flags);
+}
+
+/*
+ * Just like source_runtime(), but use "path" instead of 'runtimepath'.
+ */
+    int
+source_in_path(char_u *path, char_u *name, int flags)
+{
+    return do_in_path_and_pp(path, name, flags, source_callback, NULL);
+}
+
+
 /*
  * Expand wildcards in "pat" and invoke do_source() for each match.
  */
index e30701c6729367578e4c0aea783454e553d712a8..67c0a30d420ed1f04e5af1f3918cc1b8f8e5bf62 100644 (file)
@@ -449,18 +449,28 @@ vim_main2(void)
      */
     if (p_lpl)
     {
+       char_u *rtp_copy = NULL;
+
        /* First add all package directories to 'runtimepath', so that their
         * autoload directories can be found.  Only if not done already with a
-        * :packloadall command. */
+        * :packloadall command.
+        * Make a copy of 'runtimepath', so that source_runtime does not use
+        * the pack directories. */
        if (!did_source_packages)
+       {
+           rtp_copy = vim_strsave(p_rtp);
            add_pack_start_dirs();
+       }
 
+       source_in_path(rtp_copy == NULL ? p_rtp : rtp_copy,
 # ifdef VMS    /* Somehow VMS doesn't handle the "**". */
-       source_runtime((char_u *)"plugin/*.vim", DIP_ALL | DIP_NOAFTER);
+               (char_u *)"plugin/*.vim",
 # else
-       source_runtime((char_u *)"plugin/**/*.vim", DIP_ALL | DIP_NOAFTER);
+               (char_u *)"plugin/**/*.vim",
 # endif
+               DIP_ALL | DIP_NOAFTER);
        TIME_MSG("loading plugins");
+       vim_free(rtp_copy);
 
        /* Only source "start" packages if not done already with a :packloadall
         * command. */
index e8d2814089a108dd53967b2233431a43bc303085..c8730393d3aed8dc978c9aac1ddc34b5f0ea2889 100644 (file)
@@ -69,9 +69,10 @@ void ex_argdelete(exarg_T *eap);
 void ex_listdo(exarg_T *eap);
 void ex_compiler(exarg_T *eap);
 void ex_runtime(exarg_T *eap);
-int source_runtime(char_u *name, int flags);
 int do_in_path(char_u *path, char_u *name, int flags, void (*callback)(char_u *fname, void *ck), void *cookie);
 int do_in_runtimepath(char_u *name, int flags, void (*callback)(char_u *fname, void *ck), void *cookie);
+int source_runtime(char_u *name, int flags);
+int source_in_path(char_u *path, char_u *name, int flags);
 void add_pack_start_dirs(void);
 void load_start_packages(void);
 void ex_packloadall(exarg_T *eap);
index 08a107e8157fee4f14c7fa5cf63642754393c3ac..af2b3c1d4fa4a3fa58643cbb2133e536321d45b2 100644 (file)
@@ -23,28 +23,34 @@ func Test_after_comes_later()
        \ 'set guioptions+=M',
        \ 'let $HOME = "/does/not/exist"',
        \ 'set loadplugins',
-       \ 'set rtp=Xhere,Xafter',
+       \ 'set rtp=Xhere,Xafter,Xanother',
        \ 'set packpath=Xhere,Xafter',
        \ 'set nomore',
+       \ 'let g:sequence = ""',
        \ ]
   let after = [
        \ 'redir! > Xtestout',
        \ 'scriptnames',
        \ 'redir END',
+       \ 'redir! > Xsequence',
+       \ 'echo g:sequence',
+       \ 'redir END',
        \ 'quit',
        \ ]
   call mkdir('Xhere/plugin', 'p')
-  call writefile(['let done = 1'], 'Xhere/plugin/here.vim')
+  call writefile(['let g:sequence .= "here "'], 'Xhere/plugin/here.vim')
+  call mkdir('Xanother/plugin', 'p')
+  call writefile(['let g:sequence .= "another "'], 'Xanother/plugin/another.vim')
   call mkdir('Xhere/pack/foo/start/foobar/plugin', 'p')
-  call writefile(['let done = 1'], 'Xhere/pack/foo/start/foobar/plugin/foo.vim')
+  call writefile(['let g:sequence .= "pack "'], 'Xhere/pack/foo/start/foobar/plugin/foo.vim')
 
   call mkdir('Xafter/plugin', 'p')
-  call writefile(['let done = 1'], 'Xafter/plugin/later.vim')
+  call writefile(['let g:sequence .= "after "'], 'Xafter/plugin/later.vim')
 
   if RunVim(before, after, '')
 
     let lines = readfile('Xtestout')
-    let expected = ['Xbefore.vim', 'here.vim', 'foo.vim', 'later.vim', 'Xafter.vim']
+    let expected = ['Xbefore.vim', 'here.vim', 'another.vim', 'foo.vim', 'later.vim', 'Xafter.vim']
     let found = []
     for line in lines
       for one in expected
@@ -56,8 +62,12 @@ func Test_after_comes_later()
     call assert_equal(expected, found)
   endif
 
+  call assert_equal('here another pack after', substitute(join(readfile('Xsequence', 1), ''), '\s\+$', '', ''))
+
   call delete('Xtestout')
+  call delete('Xsequence')
   call delete('Xhere', 'rf')
+  call delete('Xanother', 'rf')
   call delete('Xafter', 'rf')
 endfunc
 
index 0deebee23d77911a27dce9a16a59f8f95a2f6210..6c2d63c7d736f5b90adee1e72a4fcf5e3063b511 100644 (file)
@@ -764,6 +764,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    680,
 /**/
     679,
 /**/