patch 7.4.1712

Problem:    For plugins in packages, plugin authors need to take care of all
            dependencies.
Solution:   When loading "start" packages and for :packloadall, first add all
            directories to 'runtimepath' before sourcing plugins.

diff --git a/src/ex_cmds2.c b/src/ex_cmds2.c
index 9f74376f10cb20f16086bdf259152eaa53c48b25..657c77252d7a6cc4f846e1abaee1f75ddac861bc 100644 (file)
--- a/src/ex_cmds2.c
+++ b/src/ex_cmds2.c
@@ -3313,6 +3313,11 @@ source_all_matches(char_u *pat)
     }
 }
 
+/* used for "cookie" of add_pack_plugin() */
+static int APP_ADD_DIR;
+static int APP_LOAD;
+static int APP_BOTH;
+
     static void
 add_pack_plugin(char_u *fname, void *cookie)
 {
@@ -3324,11 +3329,10 @@ add_pack_plugin(char_u *fname, void *cookie)
     int            oldlen;
     int            addlen;
     char_u  *ffname = fix_fname(fname);
-    int            load_files = cookie != NULL;
 
     if (ffname == NULL)
        return;
-    if (strstr((char *)p_rtp, (char *)ffname) == NULL)
+    if (cookie != &APP_LOAD && strstr((char *)p_rtp, (char *)ffname) == NULL)
     {
        /* directory not in 'runtimepath', add it */
        p4 = p3 = p2 = p1 = get_past_head(ffname);
@@ -3374,7 +3378,7 @@ add_pack_plugin(char_u *fname, void *cookie)
        vim_free(new_rtp);
     }
 
-    if (load_files)
+    if (cookie != &APP_ADD_DIR)
     {
        static char *plugpat = "%s/plugin/**/*.vim";
        static char *ftpat = "%s/ftdetect/*.vim";
@@ -3423,8 +3427,14 @@ ex_packloadall(exarg_T *eap)
     if (!did_source_packages || (eap != NULL && eap->forceit))
     {
        did_source_packages = TRUE;
+
+       /* First do a round to add all directories to 'runtimepath', then load
+        * the plugins. This allows for plugins to use an autoload directory
+        * of another plugin. */
+       do_in_path(p_pp, (char_u *)"pack/*/start/*", DIP_ALL + DIP_DIR,
+                                              add_pack_plugin, &APP_ADD_DIR);
        do_in_path(p_pp, (char_u *)"pack/*/start/*", DIP_ALL + DIP_DIR,
-                                                       add_pack_plugin, p_pp);
+                                                 add_pack_plugin, &APP_LOAD);
     }
 }
 
@@ -3444,7 +3454,7 @@ ex_packadd(exarg_T *eap)
        return;
     vim_snprintf(pat, len, plugpat, eap->arg);
     do_in_path(p_pp, (char_u *)pat, DIP_ALL + DIP_DIR + DIP_ERR,
-                                add_pack_plugin, eap->forceit ? NULL : p_pp);
+                   add_pack_plugin, eap->forceit ? &APP_ADD_DIR : &APP_BOTH);
     vim_free(pat);
 }
 

diff --git a/src/testdir/test_packadd.vim b/src/testdir/test_packadd.vim
index 9e75c3383e24fce2183a5ef249c9dabb0691734f..66a344b0dcc61930eb01142ce66a5c86ebc6f2a0 100644 (file)
--- a/src/testdir/test_packadd.vim
+++ b/src/testdir/test_packadd.vim
@@ -87,14 +87,39 @@ func Test_packadd_completion()
 endfunc
 
 func Test_packloadall()
-  let plugindir = &packpath . '/pack/mine/start/foo/plugin'
-  call mkdir(plugindir, 'p')
-  call writefile(['let g:plugin_foo_number = 1234'], plugindir . '/bar.vim')
+  " plugin foo with an autoload directory
+  let fooplugindir = &packpath . '/pack/mine/start/foo/plugin'
+  call mkdir(fooplugindir, 'p')
+  call writefile(['let g:plugin_foo_number = 1234',
+       \ 'let g:plugin_foo_auto = bbb#value',
+       \ 'let g:plugin_extra_auto = extra#value'], fooplugindir . '/bar.vim')
+  let fooautodir = &packpath . '/pack/mine/start/foo/autoload'
+  call mkdir(fooautodir, 'p')
+  call writefile(['let bar#value = 77'], fooautodir . '/bar.vim')
+
+  " plugin aaa with an autoload directory
+  let aaaplugindir = &packpath . '/pack/mine/start/aaa/plugin'
+  call mkdir(aaaplugindir, 'p')
+  call writefile(['let g:plugin_aaa_number = 333',
+       \ 'let g:plugin_aaa_auto = bar#value'], aaaplugindir . '/bbb.vim')
+  let aaaautodir = &packpath . '/pack/mine/start/aaa/autoload'
+  call mkdir(aaaautodir, 'p')
+  call writefile(['let bbb#value = 55'], aaaautodir . '/bbb.vim')
+
+  " plugin extra with only an autoload directory
+  let extraautodir = &packpath . '/pack/mine/start/extra/autoload'
+  call mkdir(extraautodir, 'p')
+  call writefile(['let extra#value = 99'], extraautodir . '/extra.vim')
+
   packloadall
   call assert_equal(1234, g:plugin_foo_number)
+  call assert_equal(55, g:plugin_foo_auto)
+  call assert_equal(99, g:plugin_extra_auto)
+  call assert_equal(333, g:plugin_aaa_number)
+  call assert_equal(77, g:plugin_aaa_auto)
 
   " only works once
-  call writefile(['let g:plugin_bar_number = 4321'], plugindir . '/bar2.vim')
+  call writefile(['let g:plugin_bar_number = 4321'], fooplugindir . '/bar2.vim')
   packloadall
   call assert_false(exists('g:plugin_bar_number'))
 

diff --git a/src/version.c b/src/version.c
index e25465260f3f527132bd46deae858d8634803fb6..91f02a5b88c31f7eb863f8b2cda5b88355cc3d30 100644 (file)
--- a/src/version.c
+++ b/src/version.c
@@ -748,6 +748,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    1712,
 /**/
     1711,
 /**/