You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
package require wibble
package require de1_machine
set plugin_name "web_api"
Causes failures on load in environments without wibble installed. If one is not going to use the plugin, there should be no reason to have to install wibble. Though preloaded in AndroWish, it is not present in, for example, macOS wish, including those installed with brew.
Suggest moving the offending dependency into a later phase of plugin initialization, where it is only evaluated if the plugin is active.
The loading problem seems to have been resolved by 495b831
The questions of where to declare proc ::wibble::check_auth are primarily stylistic in this specific instance. They still are done outside of proc main and register both a namespace and procs.
Causes failures on load in environments without
wibble
installed. If one is not going to use the plugin, there should be no reason to have to installwibble
. Though preloaded in AndroWish, it is not present in, for example, macOSwish
, including those installed withbrew
.Suggest moving the offending dependency into a later phase of plugin initialization, where it is only evaluated if the plugin is active.
git merge-base main HEAD
55c65f1
The text was updated successfully, but these errors were encountered: