Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

crash.log: loaded plugin #8228

Closed
hius07 opened this issue Sep 18, 2021 · 6 comments · Fixed by #8230
Closed

crash.log: loaded plugin #8228

hius07 opened this issue Sep 18, 2021 · 6 comments · Fixed by #8230
Milestone

Comments

@hius07
Copy link
Member

hius07 commented Sep 18, 2021

Every start of the program, then every opening a book, then every switching to file browser and back we have a record in the crash.log

logger.info("FM loaded plugin", name,

and the same from reader.
Is it necessary? Can it be changed to the debug info to lighten regular crash.log?

@poire-z
Copy link
Contributor

poire-z commented Sep 18, 2021

I exactly do this in my own patch.

-            logger.info("RD loaded plugin", plugin_module.name,
+            logger.dbg("RD loaded plugin", plugin_module.name,

No real idea if it can be useful for some users, to know one plugin has loaded correctly - and/or if we need to do something to log them only once for FM and only once for RD.
Pinging @zwim @yparitcher : has this ever been useful to you when developping your own plugins ?

@zwim
Copy link
Contributor

zwim commented Sep 18, 2021

The message is/was usefull. But it would be no problem if it is changed from logger.info to logger.dbg

@NiLuJe
Copy link
Member

NiLuJe commented Sep 18, 2021

Keep in mind that there's one in filemanager and one in readerui ;).

@hius07
Copy link
Member Author

hius07 commented Sep 18, 2021

@NiLuJe, I have permanently errors in crash.log about these kindle fonts
cannot register font </usr/java/lib/fonts/Caecilia_LT_66_Medium_Italic.ttf>
cannot register font </usr/java/lib/fonts/Caecilia_LT_76_Bold_Italic.ttf>

May I change to true? (I've tried and the errors have gone)

["Caecilia_LT_66_Medium_Italic.ttf"] = false,

["Caecilia_LT_76_Bold_Italic.ttf"] = false,

@NiLuJe
Copy link
Member

NiLuJe commented Sep 18, 2021

It possibly depends on FW version, but, sure, go ahead ;).

@yparitcher
Copy link
Member

The message is/was usefull. But it would be no problem if it is changed from logger.info to logger.dbg

Same here

Frenzie pushed a commit that referenced this issue Sep 20, 2021
Some Kindle system fonts cannot be registered and cause errors in crash.log.
Disabling them, and enabling one good font.
Tested on PW4, Voyage, Oasis.

Also #8228 (comment).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants