Skip to content

Fix no comms reporting crash#188

Open
WilliamRoebuck wants to merge 6 commits intoeclipse-score:mainfrom
etas-contrib:feature/no-comms-reporting-crash
Open

Fix no comms reporting crash#188
WilliamRoebuck wants to merge 6 commits intoeclipse-score:mainfrom
etas-contrib:feature/no-comms-reporting-crash

Conversation

@WilliamRoebuck
Copy link
Copy Markdown
Contributor

Add test for and fix a case where opening shared memory before attempting to report running from a non-reporting process would cause a crash.
Some formatting changes have been made but are separated into their own commits

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 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: cc106b9f-0aa9-481e-8960-257f485d6048
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: 
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 (84 packages loaded, 10 targets configured)

Analyzing: target //:license-check (134 packages loaded, 392 targets configured)

Analyzing: target //:license-check (145 packages loaded, 5363 targets configured)

Analyzing: target //:license-check (152 packages loaded, 8045 targets configured)

Analyzing: target //:license-check (157 packages loaded, 8094 targets configured)

Analyzing: target //:license-check (157 packages loaded, 8094 targets configured)

INFO: Analyzed target //:license-check (162 packages loaded, 10232 targets configured).
[1 / 1] no actions running
[14 / 16] [Prepa] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar
[15 / 16] Building license.check.license_check.jar (); 0s disk-cache, multiplex-worker
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: 20.670s, Critical Path: 2.47s
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

github-actions Bot commented May 1, 2026

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

@WilliamRoebuck WilliamRoebuck force-pushed the feature/no-comms-reporting-crash branch from bf86f35 to 3e87606 Compare May 1, 2026 08:06
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.

1 participant