Merge c2df863e5d into 07fd3e685a

.bcr/presubmit.yml

@@ -1,24 +1,25 @@
 # Copyright 2025 Arthur Sonzogni. All rights reserved.
 # Use of this source code is governed by the MIT license that can be found in
 # the LICENSE file.
-matrix:
+bcr_test_module:
-  platform:
+  module_path: "."
-  - centos7
+  matrix:
-  - debian10
+    platform: [
-  - ubuntu2004
+      "debian11",
-  - macos
+      "macos",
-  - windows
+      "macos-arm64",
-  bazel: [6.x, 7.x, 8.x]
+      "ubuntu2204",
-tasks:
+      "windows",
-  verify_targets:
+    ]
-    name: Build and test.
+    bazel: [
-    platform: ${{ platform }}
+      6.x,
-    bazel: ${{ bazel }}
+      7.x,
-    build_targets:
+      8.x,
-    - '@ftxui//:ftxui'
+    ]
-    - '@ftxui//:screen'
+  tasks:
-    - '@ftxui//:dom'
+    run_tests:
-    - '@ftxui//:component'
+      name: "Run test module"
-    test_targets:
+      platform: ${{ platform }}
-    - '@ftxui//:tests'
+      bazel: ${{ bazel }}
-
+      test_targets:
+        - "//..."

.bcr/source.template.json

@@ -1,5 +1,5 @@
 {
   "integrity": "",
-  "strip_prefix": "",
+  "strip_prefix": "{REPO}-{VERSION}",
-  "url": "https://github.com/ArthurSonzogni/FTXUI/releases/download/{TAG}/source.tar.gz"
+  "url": "https://github.com/{OWNER}/{REPO}/releases/download/{TAG}/{REPO}-{TAG}.tar.gz"
 }

.github/workflows/build.yaml

@@ -12,7 +12,7 @@ on:
 jobs:
 
   test_bazel:
-    name: "Bazel, ${{ matrix.compiler }}, ${{ matrix.os }}"
+    name: "Test Bazel"
     strategy:
       fail-fast: false
       matrix:
@@ -44,7 +44,7 @@ jobs:
         run: bazel run tests
 
   test_cmake:
-    name: "CMake, ${{ matrix.compiler }}, ${{ matrix.os }}"
+    name: "Tests CMake"
     strategy:
       fail-fast: false
       matrix:
@@ -173,7 +173,7 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
   # Build artifact for the release
-  package_compiled:
+  package:
     name: "Build packages"
     needs: release
     strategy:
@@ -217,29 +217,6 @@ jobs:
           upload_url: ${{ needs.release.outputs.upload_url }}
           asset_path: ${{ matrix.asset_path }}
           overwrite: true
-
-  # Build "source" artifact for the release. This is the same as the github
-  # "source" archive, but with a stable URL. This is useful for the Bazel
-  # Central Repository.
-  package_source:
-    name: "Build source package"
-    needs: release
-    runs-on: ubuntu-latest
-    steps:
-      - name: "Checkout repository"
-        uses: actions/checkout@v3
-
-      - name: "Create source package"
-        run: >
-          git archive --format=tar.gz --prefix=ftxui/ -o source.tar.gz HEAD
-
-      - name: "Upload source package"
-        uses: shogo82148/actions-upload-release-asset@v1
-        with:
-          upload_url: ${{ needs.release.outputs.upload_url }}
-          asset_path: source.tar.gz
-          overwrite: true
-
   
   documentation:
     if: github.ref == 'refs/heads/main'

.github/workflows/publish_to_bcr.yaml

@@ -1,12 +1,12 @@
 on:
-  # On new releases:
+  # Run the publish workflow after a successful release.
   workflow_call:
     inputs:
       tag_name:
         required: true
         type: string
 
-  # On manual trigger:
+  # Allow manual triggering of the workflow.
   workflow_dispatch:
     inputs:
       tag_name:
@@ -15,9 +15,10 @@ on:
 
 jobs:    
   publish:
-    uses: bazel-contrib/publish-to-bcr/.github/workflows/publish.yaml@v0.0.4
+    uses: bazel-contrib/publish-to-bcr/.github/workflows/publish.yaml@[version]
     with:
       tag_name: ${{ inputs.tag_name }}
+      # GitHub repository which is a fork of the upstream where the Pull Request will be opened.
       registry_fork: ArthurSonzogni/bazel-central-registry
     permissions:
       attestations: write

BUILD.bazel

@@ -3,9 +3,13 @@
 # the LICENSE file.
 
 # TODO:
-# - Build benchmark.
+# - Windows/MSVC support.
-# - Build fuzzers.
+#   - Pass /utf-8 to MSVC users depending on FTXUI.
-# - Build documentation.
+#   - Pass "FTXUI_MICROSOFT_TERMINAL_FALLBACK" to windows users.
+# - Support building benchmark.
+# - Support building examples.
+# - Support building fuzzer.
+# - Support building documentation.
 # - Enable the two tests timing out.
 # - Support WebAssembly
 
@@ -13,7 +17,7 @@ load("@rules_cc//cc:defs.bzl", "cc_binary", "cc_library", "cc_test")
 load(":bazel/ftxui.bzl", "ftxui_cc_library")
 load(":bazel/ftxui.bzl", "generate_examples")
 load(":bazel/ftxui.bzl", "cpp20")
-load(":bazel/ftxui.bzl", "windows_copts")
+load(":bazel/ftxui.bzl", "msvc_copts")
 load(":bazel/ftxui.bzl", "pthread_linkopts")
 
 package(default_visibility = ["//visibility:public"])
@@ -241,7 +245,7 @@ cc_test(
         "include",
         "src",
     ],
-    copts = cpp20() + windows_copts(),
+    copts = cpp20() + msvc_copts(),
     deps = [
         "//:ftxui",
         "@googletest//:gtest_main",

bazel/ftxui.bzl

@@ -20,42 +20,23 @@ def cpp20():
         "//conditions:default": ["-std=c++20"],
     })
 
-# Microsoft terminal is a bit buggy ¯\_(ツ)_/¯ and MSVC uses bad defaults.
+def msvc_copts():
-def windows_copts():
     MSVC_COPTS = [
-      # Microsoft Visual Studio must decode sources files as UTF-8.
+      # Force Microsoft Visual Studio to decode sources files in UTF-8.
       "/utf-8",
 
-      # Microsoft Visual Studio must interpret the codepoint using unicode.
+      # Force Microsoft Visual Studio to interpret the source files as
+      # Unicode.
       "/DUNICODE",
       "/D_UNICODE",
 
       # Fallback for Microsoft Terminal.
-      # This
-      # - Replace missing font symbols by others.
-      # - Reduce screen position pooling frequency to deals against a Microsoft
-      #   race condition. This was fixed in 2020, but clients never not updated.
-      #   - https://github.com/microsoft/terminal/pull/7583
-      #   - https://github.com/ArthurSonzogni/FTXUI/issues/136
       "/DFTXUI_MICROSOFT_TERMINAL_FALLBACK",
     ]
-
-    WINDOWS_COPTS = [
-      # Fallback for Microsoft Terminal.
-      # This
-      # - Replace missing font symbols by others.
-      # - Reduce screen position pooling frequency to deals against a Microsoft
-      #   race condition. This was fixed in 2020, but clients never not updated.
-      #   - https://github.com/microsoft/terminal/pull/7583
-      #   - https://github.com/ArthurSonzogni/FTXUI/issues/136
-      "-DFTXUI_MICROSOFT_TERMINAL_FALLBACK",
-    ];
     
     return select({
-        # MSVC:
         "@rules_cc//cc/compiler:msvc-cl": MSVC_COPTS,
         "@rules_cc//cc/compiler:clang-cl": MSVC_COPTS,
-        "@platforms//os:windows": WINDOWS_COPTS,
         "//conditions:default": [],
     })
 
@@ -88,12 +69,10 @@ def ftxui_cc_library(
             "include",
             "src",
         ],
-        copts = cpp17() + windows_copts(),
+        copts = cpp17() + msvc_copts(),
         visibility = ["//visibility:public"],
     )
 
-# Compile all the examples in the examples/ directory.
-# This is useful to check the Bazel is synchronized with CMake definitions.
 def generate_examples():
     cpp_files = native.glob(["examples/**/*.cpp"])
 
@@ -111,5 +90,5 @@ def generate_examples():
             name = name,
             srcs = [src],
             deps = ["//:component"],
-            copts = cpp20() + windows_copts(),
+            copts = cpp20() + msvc_copts(),
         )