[ 
https://issues.apache.org/jira/browse/TS-2036?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13709194#comment-13709194
 ] 

Nick Kew edited comment on TS-2036 at 7/15/13 11:49 PM:
--------------------------------------------------------

Aaargh!  "Provide patch" option just marks it as PatchAvailable, without 
offering the option to provide a patch.

git push is also eluding me with "error: no DAV locking support on 
https://git-wip-us.apache.org/repos/asf/trafficserver.git/";, so here's the 
patch:
{code}
-#ifndef TS_NO_API
-    plugin_init(system_config_directory);        // plugin.config
-#else
+#ifdef TS_NO_API
     api_init();                 // we still need to initialize some of the data
     // i.e. http_global_hooks
+#else
+    plugin_init(system_config_directory);        // plugin.config
     pmgmt->registerPluginCallbacks(global_config_cbs);
 #endif
{code}

Summary: plugin callbacks need to be registered when there is an API, not when 
there isn't!  The patch switches #ifndef to #ifdef, because confusion over the 
double-negative is probably what caused the bug in the first place.
                
      was (Author: [email protected]):
    Aaargh!  "Provide patch" option just marks it as PatchAvailable, without 
offering the option to provide a patch.

git push is also eluding me with "error: no DAV locking support on 
https://git-wip-us.apache.org/repos/asf/trafficserver.git/";, so here's the 
patch:

-#ifndef TS_NO_API
-    plugin_init(system_config_directory);        // plugin.config
-#else
+#ifdef TS_NO_API
     api_init();                 // we still need to initialize some of the data
     // i.e. http_global_hooks
+#else
+    plugin_init(system_config_directory);        // plugin.config
     pmgmt->registerPluginCallbacks(global_config_cbs);
 #endif

Summary: plugin callbacks need to be registered when there is an API, not when 
there isn't!  The patch switches #ifndef to #ifdef, because confusion over the 
double-negative is probably what caused the bug in the first place.
                  
> Management update API for plugins doesn't work
> ----------------------------------------------
>
>                 Key: TS-2036
>                 URL: https://issues.apache.org/jira/browse/TS-2036
>             Project: Traffic Server
>          Issue Type: Bug
>          Components: Management API
>            Reporter: Nick Kew
>            Assignee: Nick Kew
>
> Functions registered with TSMgmtUpdateRegister are never called.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to