Clojurians Log v2

does anyone know what could be happening here, in a directory with nothing but this deps.edn pointing at clojurescript at latest master

{:deps {org.clojure/clojurescript {:local/root "/path/to/clojurescript"}}}

➜ clj -m cljs.main --repl-env node
WARNING: When invoking clojure.main, use -M
ClojureScript 0.0.2008668038
cljs.user=> 

ah seems to be better after a script/build

I see this as well in the REPL, but in the actual dependencies it's the correct version.

you're probably just picking up a resource file that stores the ClojureScript version that is old would be my guess

Looking at the goog.log changes for Glögi (https://github.com/lambdaisland/glogi/), seems several instance methods have become static methods/functions

(.setLevel logger level)  ->  (glog/setLevel logger level)
(.addHandler logger handler) -> (glog/addHandler logger level)
(.logRecord logger record)  ->  (glog/publishLogRecord logger record)

(require '[goog.log :as glog])

now the name of your library makes even more sense

If I want to support both I guess I can do a (exists? glog/publishLogRecord) etc. Unless someone has a better idea 🙂

dynamic check seems best

Yes, and also calling the log functions themselves has changed

I'm seeing this when using master as a git dep as well. Where would that resource file be?

local/git deps are basically used the same way. don't know, just guessing

just in general, projects with build steps do not play well (yet!) as local/git deps, and cljs is certainly such a project

while I'm here let me also warmly encourage people to use Glögi (or goog.log) in their projects/libraries. It is really nice to get consistent logging and be able to control log levels all in one place

(log/set-levels
 '{:glogi/root :debug
   lambdaisland :all
   co.gaiwan.some.ns :warn
   some.library :info})

@kommen didn’t you open a PR?

@plexus I guess not yet but here’s the commit https://github.com/nextjournal/glogi/commit/d17e48da78f2ae24938104b7ee2567023b9bc007

opened a draft https://github.com/lambdaisland/glogi/pull/8

thanks, but I was nearly there. I'd like to retain backwards compatibility at least for a while.

So if you do a log/info in lambdaisland it will override to :all?

It's not clear in the README how that behavior is supposed to work. The example does a log in the root namespace. It would be helpful to have an example in one of the defined namespaces.

it follows the rules of goog.log or pretty much any logging library out there, if there's a log level defined for a given logger it uses that, otherwise it goes up the hierarchy.

{foo :info
 foo.bar :warn
 foo.bar.baz :debug
 :glogi/root :error}

So it would be good for someone that hasn't used that to be in the README. 🙂

doc PRs are always welcome!

Since the documentation for goog.log isn't very clear as you mentioned

Documentation provided by someone whom has never used the library doesn't make any sense to me.

maybe just linking to something like http://tutorials.jenkov.com/java-logging/logger-hierarchy.html

So then according to that article the answer to my original question is no. It just sets the filter level. You still have to call the appropriate goog.log function. So I'm failing to see why this is any better than just calling the log function you want for the given thing you are logging.

I don't follow. You put logging in your code at the appropriate level for what you're logging, log/trace for highly detailed debug stuff, log/error for exceptional situations. Then you can use the hierarchies for fine grained control of how much detail you want to see for a given tree of namespaces/loggers at a given time.

see Layer your Logging https://lambdaisland.com/blog/2020-09-28-logging-in-practice-glogi-pedestal

Is there any coordination with the JVM team when adding things to cljs.core, e.g. about the function iter, etc? For context: https://ask.clojure.org/index.php/10303/interop-clojure-pattern-clojure-consider-adding-iter-clojure