Merge branch 'The-OpenROAD-Project:master' into gpl_remove_filler_removal_cap

Author: LucasYuki

.bazelrc

@@ -78,24 +78,16 @@ build --jobs=200
 build --disk_cache=~/.cache/bazel-disk-cache
 build --repository_cache=~/.cache/bazel-repository-cache
 
-# --- CI Settings (write access) ---
-build:ci --remote_cache=https://storage.googleapis.com/openroad-bazel-cache
-build:ci --remote_cache_compression=true
-build:ci --remote_upload_local_results=true
 # CI only needs to know if the build succeeded, not the artifacts themselves.
 build:ci --remote_download_minimal
+build:ci --remote_upload_local_results=true
 # Disable disk cache for CI builds
 build:ci --disk_cache=
 
-# --- OpenROAD Developer Settings (read-only access) ---
-build:openroad-dev --remote_cache=https://storage.googleapis.com/openroad-bazel-cache
-build:openroad-dev --remote_cache_compression=true
-build:openroad-dev --remote_upload_local_results=false
-# Developers need the actual output files to run and use them.
-build:openroad-dev --remote_download_toplevel
-build:openroad-dev --google_default_credentials=true
-build:openroad-dev --disk_cache=/workspace/.cache/bazel-disk-cache
-build:openroad-dev --repository_cache=/workspace/.cache/bazel-repository-cache
+# Setup remote cache
+build --remote_cache=https://bazel.precisioninno.com
+build --remote_cache_compression=true
+build --remote_upload_local_results=false
 
 # Without this, bazelisk build ... builds the bazel-orfs tests
 build --build_tag_filters=-orfs

.buildifier.json

@@ -0,0 +1,6 @@
+{
+    "type": "auto",
+    "mode": "fix",
+    "lint": "fix",
+    "warnings": "all"
+}

.github/workflows/buildifier.yaml

@@ -0,0 +1,44 @@
+name: Lint Bazel
+
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+    branches:
+      - master
+
+jobs:
+  buildifier:
+    runs-on: ${{ vars.USE_SELF_HOSTED == 'true' && 'self-hosted' || 'ubuntu-latest' }}
+    strategy:
+      fail-fast: false
+      matrix:
+        check:
+          - name: Buildifier format
+            run: ./buildifier -r -mode=check -lint=off .
+          - name: Buildifier lint
+            run: ./buildifier -r -lint=warn .
+    name: ${{ matrix.check.name }}
+    env:
+      BUILDIFIER_VERSION: v8.2.1
+
+    steps:
+      - name: Checkout Code
+        uses: actions/checkout@v5
+
+      - name: Cache buildifier
+        id: cache-buildifier
+        uses: actions/cache@v4
+        with:
+          path: ./buildifier
+          key: ${{ runner.os }}-buildifier-${{ env.BUILDIFIER_VERSION }}
+
+      - name: Download buildifier
+        if: steps.cache-buildifier.outputs.cache-hit != 'true'
+        run: |
+          wget https://github.com/bazelbuild/buildtools/releases/download/${BUILDIFIER_VERSION}/buildifier-linux-amd64 -O buildifier
+          chmod +x buildifier
+
+      - name: ${{ matrix.check.name }}
+        run: ${{ matrix.check.run }}

BUILD.bazel

@@ -2,6 +2,9 @@
 # Copyright (c) 2025-2025, The OpenROAD Authors
 
 load("@bazel_skylib//rules:common_settings.bzl", "string_flag")
+load("@openroad_rules_python//python:defs.bzl", "py_library")
+load("@rules_cc//cc:cc_binary.bzl", "cc_binary")
+load("@rules_cc//cc:cc_library.bzl", "cc_library")
 load("//bazel:python_wrap_cc.bzl", "PYTHON_STABLE_API_DEFINE", "python_wrap_cc")
 load("//bazel:tcl_encode_or.bzl", "tcl_encode")
 load("//bazel:tcl_wrap_cc.bzl", "tcl_wrap_cc")
@@ -104,8 +107,8 @@ OPENROAD_LIBRARY_DEPS = [
     "@edu_berkeley_abc//:abc-lib",
 ] + select(
     {
-        ":platform_gui": ["//src/gui:gui_qt"],
         ":platform_cli": ["//src/gui"],
+        ":platform_gui": ["//src/gui:gui_qt"],
     },
 )
 
@@ -161,8 +164,8 @@ cc_binary(
 
 GUI_BUILD_FLAGS = select(
     {
-        ":platform_gui": ["BUILD_GUI=true"],
         ":platform_cli": ["BUILD_GUI=false"],
+        ":platform_gui": ["BUILD_GUI=true"],
     },
 )
 

Jenkinsfile

@@ -219,13 +219,9 @@ def bazelTest = {
         }
         withDockerContainer(args: '-u root -v /var/run/docker.sock:/var/run/docker.sock', image: 'openroad/bazel-ci:latest') {
             stage('bazelisk test ...') {
-                withCredentials([file(credentialsId: 'bazel-cache-sa', variable: 'GCS_SA_KEY')]) {
+                withCredentials([string(credentialsId: 'bazel-auth-token-b64', variable: 'BAZEL_AUTH_TOKEN_B64')]) {
                     timeout(time: 120, unit: 'MINUTES') {
-                        def cmd = 'bazelisk test --config=ci --show_timestamps --test_output=errors --curses=no --force_pic';
-                        if (env.BRANCH_NAME != 'master') {
-                            cmd += ' --remote_upload_local_results=false';
-                        }
-                        cmd += ' --google_credentials=$GCS_SA_KEY';
+                        def cmd = 'bazelisk test --config=ci --show_timestamps --test_output=errors --curses=no --force_pic --remote_header="Authorization=Basic $BAZEL_AUTH_TOKEN_B64"'
                         try {
                             sh label: 'Bazel Build', script: cmd + ' ...';
                         } catch (e) {

MODULE.bazel

@@ -90,6 +90,7 @@ bazel_dep(name = "or-tools", version = "9.12")
 bazel_dep(name = "spdlog", version = "1.15.1")
 bazel_dep(name = "tcmalloc", version = "0.0.0-20250331-43fcf6e")
 bazel_dep(name = "zlib", version = "1.3.1.bcr.5")
+bazel_dep(name = "yaml-cpp", version = "0.8.0")
 
 # A from source build of QT that allows it to link into OpenROAD.
 # Building like any other bazel project. scripts in the docker folder
@@ -153,7 +154,7 @@ bazel_dep(name = "bazel-orfs")
 # To bump version, run: bazelisk run @bazel-orfs//:bump
 git_override(
     module_name = "bazel-orfs",
-    commit = "2cf0198a49a220cfcabae35bba64b0034bbdbffc",
+    commit = "acd76cb3d123e18ee1b65a632cd05be30b46fbbf",
     remote = "https://github.com/The-OpenROAD-Project/bazel-orfs.git",
 )
 
@@ -162,18 +163,16 @@ orfs = use_extension("@bazel-orfs//:extension.bzl", "orfs_repositories")
 # To bump version, run: bazelisk run @bazel-orfs//:bump
 orfs.default(
     # Official image https://hub.docker.com/r/openroad/orfs/tags
-    image = "docker.io/openroad/orfs:v3.0-3845-gbefd38218",
+    image = "docker.io/openroad/orfs:v3.0-3868-g832999cb0",
     # Use OpenROAD of this repo instead of from the docker image
     openroad = "//:openroad",
-    sha256 = "4982a5b87588e4699f11deaed3286cba8b65d144e439c6f5845656abebfc0971",
+    sha256 = "01c6532234a4708847d1fe98d71dfc586d7566e5428bf7dfa23099bddf195f81",
 )
 use_repo(orfs, "com_github_nixos_patchelf_download")
 use_repo(orfs, "docker_orfs")
 
 SCALA_VERSION = "2.13.16"
 
-SCALA_VERSION_SHORT = SCALA_VERSION.rpartition(".")[0]
-
 scala_config = use_extension(
     "@rules_scala//scala/extensions:config.bzl",
     "scala_config",
@@ -195,7 +194,7 @@ scala_deps.scalatest()
 
 maven = use_extension("@rules_jvm_external//:extensions.bzl", "maven")
 maven.install(
-    name = "openroad_maven",
+    name = "maven",
     artifacts = [
         "org.chipsalliance:chisel_2.13:7.0.0",
         "org.chipsalliance:chisel-plugin_2.13.16:7.0.0",
@@ -206,7 +205,7 @@ maven.install(
         "https://s01.oss.sonatype.org/content/repositories/snapshots",
     ],
 )
-use_repo(maven, "openroad_maven")
+use_repo(maven, "maven")
 
 http_archive = use_repo_rule("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
 

MODULE.bazel.lock

@@ -1,4 +1,5 @@
 load("@openroad_rules_python//python:pip.bzl", "compile_pip_requirements")
+load("@rules_cc//cc:cc_library.bzl", "cc_library")
 
 compile_pip_requirements(
     name = "requirements",

bazel/BUILD

@@ -22,7 +22,9 @@ def _tcl_encode_or_impl(ctx):
     # Only keep .tcl and .py files.
     allowed_extensions = (".tcl", ".py")
     filtered_sources = [
-        f for f in ctx.files.srcs if f.basename.endswith(allowed_extensions)
+        f
+        for f in ctx.files.srcs
+        if f.basename.endswith(allowed_extensions)
     ]
 
     ctx.actions.run(
@@ -41,13 +43,13 @@ tcl_encode = rule(
             doc = "The name of the extern string array value in the generated file.",
             mandatory = True,
         ),
-        "out": attr.string(
-            doc = "The name of the C++ source file generated by these rules.",
-        ),
         "namespace": attr.string(
             doc = "The namespace in the C++ source file generated.",
             mandatory = True,
         ),
+        "out": attr.string(
+            doc = "The name of the C++ source file generated by these rules.",
+        ),
         "srcs": attr.label_list(
             allow_empty = False,
             allow_files = [".tcl", ".py"],

bazel/tcl_encode_or.bzl

@@ -136,6 +136,7 @@ tcl_wrap_cc = rule(
             The root file must be explicitly provided. This is the file which will be passed to
             swig for generation.""",
         ),
+        "runtime_header": attr.string(),
         "srcs": attr.label_list(
             allow_empty = False,
             allow_files = [".i", ".swig", ".h", ".hpp", ".hh"],
@@ -147,7 +148,6 @@ tcl_wrap_cc = rule(
         "swig_options": attr.string_list(
             doc = "args to pass directly to the swig binary",
         ),
-        "runtime_header": attr.string(),
         "_swig": attr.label(
             default = "@org_swig//:swig_stable",
             allow_files = True,