search

beehive-lab / TornadoVM

TornadoVM: A practical and efficient heterogeneous programming framework for managed languages
https://www.tornadovm.org
Apache License 2.0
1.18k stars 113 forks source link

Fix logger for all backends #426

Closed jjfumero closed 4 months ago

jjfumero commented 4 months ago

Description

Fix logger initialization after the Tornado class refactoring.

Problem description

The issue was a missing initialisation for the factory logging methods.

Backend/s tested

Mark the backends affected by this PR.

OS tested

Mark the OS where this PR is tested.

Did you check on FPGAs?

If it is applicable, check your changes on FPGAs.

How to test the new patch?

tornado --debug -ea  --jvm "-Xmx6g -Dtornado.recover.bailout=False -Dtornado.unittests.verbose=True "  -m  tornado.unittests/uk.ac.manchester.tornado.unittests.tools.TornadoTestRunner  --params "uk.ac.manchester.tornado.unittests.foundation.TestIntegers"
jjfumero commented 4 months ago

I have a better solution for the Logging. I will provide a new patch in a bit

jjfumero commented 4 months ago

Let's have a new look. The reason behind this new design is that we are forced to declare the class and fine tune from which class we are enabling the logger instead of having the "Anonymous" option on all the time.