Skip to content

Fix GCC 14 Warnings#172

Merged
rmaddikery merged 2 commits into
eclipse-score:mainfrom
odra:autosd-fixes
Jul 7, 2026
Merged

Fix GCC 14 Warnings#172
rmaddikery merged 2 commits into
eclipse-score:mainfrom
odra:autosd-fixes

Conversation

@odra

@odra odra commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Notes for Reviewer

To test this PR using AutoSD's toolchainchain (which relies on gcc14):

.bazelrc

build:_autosd-common --force_pic
build:_autosd-common --@score_baselibs//src/log:safety_level=qm
build:_autosd-common --@score_baselibs//score/json:base_library=nlohmann
build:_autosd-common --@score_baselibs//score/memory/shared/flags:use_typedshmd=False
build:_autosd-common --features=minimal_warnings --features=-strict_warnings --features=warnings_as_errors

build:autosd-x86_64 --config=_autosd-common
build:autosd-x86_64 --platforms=@score_bazel_platforms//:x86_64-linux-autosd10
build:autosd-x86_64 --extra_toolchains=@score_autosd10_x86_64_toolchain//:x86_64-linux-autosd10
build:autosd-x86_64 --extra_toolchains=@score_toolchains_rust//toolchains/ferrocene:ferrocene_x86_64_unknown_linux_gnu

build:autosd-aarch64 --config=_autosd-common
build:autosd-aarch64 --platforms=@score_bazel_platforms//:aarch64-linux-autosd10
build:autosd-aarch64 --extra_toolchains=@score_autosd10_aarch64_toolchain//:aarch-linux-autosd10
build:autosd-aarch64 --extra_toolchains=@score_toolchains_rust//toolchains/ferrocene:ferrocene_aarch64_unknown_linux_gnu

MODULE.bazel

bazel_dep(name = "score_bazel_cpp_toolchains", version = "0.5.4", dev_dependency = True)
...
gcc.toolchain(
    name = "score_autosd10_x86_64_toolchain",
    runtime_ecosystem = "autosd10",
    target_cpu = "x86_64",
    target_os = "linux",
    use_default_package = True,
)
gcc.toolchain(
    name = "score_autosd10_aarch64_toolchain",
    runtime_ecosystem = "autosd10",
    target_cpu = "aarch64",
    target_os = "linux",
    use_default_package = True,
)

use_repo(
    gcc,
    "score_autosd10_aarch64_toolchain",
    "score_autosd10_x86_64_toolchain",
    ...
)

Then run a full build by running: bazel build --config autosd-x86_64 //score/....

Pre-Review Checklist for the PR Author

  • PR title is short, expressive and meaningful
  • Commits are properly organized
  • Relevant issues are linked in the References section
  • Tests are conducted
  • Unit tests are added

Checklist for the PR Reviewer

  • Commits are properly organized and messages are according to the guideline
  • Unit tests have been written for new behavior
  • Public API is documented
  • PR title describes the changes

Post-review Checklist for the PR Author

  • All open points are addressed and tracked via issues

References

Closes #

Signed-off-by: Leonardo Rossetti <lrossett@redhat.com>
@github-actions

github-actions Bot commented Jun 30, 2026

Copy link
Copy Markdown

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.3.0) and connecting to it...
INFO: Invocation ID: aadd8e75-2b47-499f-833a-8bb7b383b41c
Computing main repo mapping: 
WARNING: For repository 'score_crates', the root module requires module version score_crates@0.0.9, but got score_crates@0.0.10 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Computing main repo mapping: 
Loading: 
Loading: 4 packages loaded
Loading: 4 packages loaded
    currently loading: 
WARNING: Target pattern parsing failed.
ERROR: Skipping '//:license-check': no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/logging/logging/BUILD
ERROR: no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/logging/logging/BUILD
INFO: Elapsed time: 5.670s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@odra odra temporarily deployed to workflow-approval July 7, 2026 08:57 — with GitHub Actions Inactive
@odra odra temporarily deployed to workflow-approval July 7, 2026 08:57 — with GitHub Actions Inactive
@odra odra temporarily deployed to workflow-approval July 7, 2026 08:57 — with GitHub Actions Inactive
@odra odra temporarily deployed to workflow-approval July 7, 2026 08:57 — with GitHub Actions Inactive
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

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

@github-project-automation github-project-automation Bot moved this from Backlog to On Hold in LOG - Logging FT Jul 7, 2026
@rmaddikery rmaddikery merged commit c91479b into eclipse-score:main Jul 7, 2026
23 checks passed
@github-project-automation github-project-automation Bot moved this from On Hold to Done in LOG - Logging FT Jul 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants