search

att / rcloud

Collaborative data analysis and visualization
http://rcloud.social
MIT License
430 stars 142 forks source link

use/allow `ulog::ulog` instead of `Rserve::ulog` #2666

Open s-u opened 5 years ago

s-u commented 5 years ago

RCloud uses Rserve's configuration to setup ulog which means the ulog package has no access to that information. Ideally, packages should use ulog::ulog so they don't need to depend on Rserve and can use ulog for general logging. Most compatible way forward would be for RCloud to retrieve the ulog target setting from Rserve and use it to initialize ulog::ulog.init.

s-u commented 4 years ago

Won't do for 2.2 because too complex - requires modifications of both Rserve and ulog