Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Remove USE_EXTERNAL_CLI11: it downloads CLI11 2.3.2 while the vendored header is 2.5.0

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#175 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
85/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
cmake, cpp
Lĩnh vực
build-system

Hướng nghiên cứu

Đọc tùy chọn USE_EXTERNAL_CLI11 và khối điều kiện trong CMakeLists.txt, sau đó kiểm tra cmake/CLI11.cmake cùng các tham chiếu được tìm thấy bằng hai lệnh git grep trong issue. Xóa tùy chọn và module, rồi chạy các lệnh build, ctest và kiểm tra định dạng đã liệt kê. Công việc hoàn tất khi header CLI11 được vendored được sử dụng và không còn tham chiếu nào đến tùy chọn hoặc module.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

build good first issue severity/minor

ctrace parses its command line with the CLI11 header vendored in external/CLI11.hpp, version 2.5.0. With -DUSE_EXTERNAL_CLI11=ON (CMakeLists.txt:115-120), CMake downloads CLI11 v2.3.2 instead (cmake/CLI11.cmake): an older version that CI never builds, while the release license check reads the vendored header (CMakeLists.txt:209-214). The option gives two possible versions of the same dependency, and nothing needs it.

To do: remove the option and keep the vendored header.

How:

  • delete cmake/CLI11.cmake;
  • delete the USE_EXTERNAL_CLI11 option and its if block in CMakeLists.txt (lines 115 to 120);
  • run git grep -n USE_EXTERNAL_CLI11 and git grep -n CLI11.cmake to check that nothing else refers to them.

The wiki's build page also lists the option; a maintainer will update it.

Check:

cmake --build build --parallel 4
ctest --test-dir build --output-on-failure
./scripts/format-check.sh

Acceptance

  • cmake/CLI11.cmake is gone, and git grep USE_EXTERNAL_CLI11 finds nothing.
  • The README build configures and builds, and ctest passes.

From the technical debt register (docs/quality/technical-debt.md, D8).

Ngôn ngữ chính
C++
Star
5
Fork
4
Merge trung bình
1 giờ 10 phút
Pull request đã merge (30 ngày)
79

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của CoreTrace/coretrace

Tất cả issue của CoreTrace/coretrace

Issue tương tự

Thêm issue về C++

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.