Skip to content

Removing score_logging dependency & update dependency versions for 0.7 release#185

Merged
NicolasFussberger merged 10 commits intoeclipse-score:mainfrom
etas-contrib:remote-logging-dep
Apr 28, 2026
Merged

Removing score_logging dependency & update dependency versions for 0.7 release#185
NicolasFussberger merged 10 commits intoeclipse-score:mainfrom
etas-contrib:remote-logging-dep

Conversation

@MaciejKaszynski
Copy link
Copy Markdown
Contributor

@MaciejKaszynski MaciejKaszynski commented Apr 27, 2026

Removing the score_logging dependency as we do not rely on remote logging.
This resolves:
#181

  • This allowed to update other dependency versions

Note there is an issue with flatbuffers, check github.com//issues/186

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 27, 2026

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.4.2) and connecting to it...
INFO: Invocation ID: e54d0b7b-a6a2-4518-ab09-cd9fecb7b86a
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (29 packages loaded, 10 targets configured)

Analyzing: target //:license-check (83 packages loaded, 10 targets configured)

Analyzing: target //:license-check (129 packages loaded, 747 targets configured)

Analyzing: target //:license-check (144 packages loaded, 5360 targets configured)

Analyzing: target //:license-check (154 packages loaded, 5429 targets configured)

Analyzing: target //:license-check (154 packages loaded, 5429 targets configured)

Analyzing: target //:license-check (158 packages loaded, 8212 targets configured)

Analyzing: target //:license-check (161 packages loaded, 10106 targets configured)

INFO: Analyzed target //:license-check (162 packages loaded, 10232 targets configured).
[8 / 15] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox ... ( 1 action running)
[13 / 16] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 1s disk-cache, processwrapper-sandbox
[14 / 16] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 0s disk-cache, processwrapper-sandbox
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 24.412s, Critical Path: 2.92s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

@github-actions
Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

Comment thread MODULE.bazel Outdated
Comment thread MODULE.bazel Outdated
Comment thread MODULE.bazel Outdated
Comment thread MODULE.bazel Outdated
Comment thread MODULE.bazel Outdated
Copy link
Copy Markdown
Contributor

@pawelrutkaq pawelrutkaq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why this would make sense ? All exec shall use logging backend from score logging.

@MaciejKaszynski
Copy link
Copy Markdown
Contributor Author

why this would make sense ? All exec shall use logging backend from score logging.

The logging interface is coming from baselibs, that is what we are already using. I also got the info from Ragha that the score_logging/mw/log target will be soon deprecated and it is expected for users to use score_baselibs.

- This breaks reference integration build
@NicolasFussberger NicolasFussberger changed the title Removing score_logging dependency Removing score_logging dependency & update dependency versions for 0.7 release Apr 27, 2026
@pawelrutkaq
Copy link
Copy Markdown
Contributor

why this would make sense ? All exec shall use logging backend from score logging.

The logging interface is coming from baselibs, that is what we are already using. I also got the info from Ragha that the score_logging/mw/log target will be soon deprecated and it is expected for users to use score_baselibs.

Thats true, bue app needs to link backend, so we wer ejsut maybe linking wrong target from score_logging but we need to link it

@MaciejKaszynski MaciejKaszynski deployed to workflow-approval April 28, 2026 05:05 — with GitHub Actions Active
@NicolasFussberger NicolasFussberger merged commit 2f2e9c4 into eclipse-score:main Apr 28, 2026
19 of 31 checks passed
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 this pull request may close these issues.

3 participants