发版工具写进 manifest 注释的补丁目录和仓库里实际沿用的目录对不上

Đang mở Phù hợp với người mới
#50 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
78/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
release

Hướng nghiên cứu

Lỗi nằm ở tools/pubrel/release_tools.py: đoạn mã snapshot gần dòng 301 ghi lại dir của mỗi chuỗi patch thành tên kho lưu trữ cộng với số build, còn annotated_manifest gần dòng 985 ghi giá trị đã ghi lại đó vào chú thích của manifest. Hãy so sánh các giá trị này với các mục dir thực tế trong patches/series.json và làm cho chú thích sử dụng thư mục nơi chuỗi thực sự nằm. Thêm một test cho annotated_manifest trong đó một chuỗi giữ nguyên thư mục cũ, sau đó kiểm tra xem ba thư mục trong chú thích có tồn tại dưới patches/ hay không.

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

Mô tả

bug P3

现象

发版工具(#47 搬进来的 tools/pubrel/release_tools.py)给源码包里的 manifest 写注释时,补丁目录写的不是仓库里实际的目录:

  • 抓快照时(约 301 行)每个补丁系列的 dir 一律改成「仓库名 + / + 本次构建号」,比如 gfxstream/mas36、crashpad/mas36;
  • annotated_manifest(约 985 行)把它写进注释:补丁系列 patches/<dir> = 基准 + N 个提交;
  • 但没变的系列,仓库里沿用的是原来的目录:patches/series.json 里是 gfxstream/mas35、crashpad/module-version22。

0.35.0、0.36.0 的 manifest 注释都是这样,照注释去 patches/ 找会找不到。只影响注释,补丁内容、回放和哈希都不受影响。

要做的

注释里写系列实际所在的目录,和 patches/series.json 的 dir 一致。改完给 annotated_manifest 补一个测试:有一个系列沿用旧目录时,注释写的是旧目录。

验收

  • 用 0.36.0 的输入重新生成 manifest,注释里的三个目录都能在 patches/ 下找到;
  • 已发布版本的 manifest 不改(只改以后的)。

这条在 #47 合并前后修都可以;合并前修的话直接改在那个 PR 里。

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

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 ModAVD/ModAVD

Tất cả issue của ModAVD/ModAVD

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.