Packaging: the two release lanes disagree on shipping the GPL license text — team decision needed
Maintainer thường phản hồi trong vòng 1 ngày
Một pull request liên quan đã được merge.
- #787 của @Amperstrand — đã merge
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 25/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- go
- Lĩnh vực
- build-system
Hướng nghiên cứu
Read packaging/local-build-ipk.sh and packaging/Makefile to verify the stated packaging difference. The issue explicitly leaves the policy choice to the team, so wait for a decision before changing either lane; done means implementing the chosen outcome and updating equivalence handling if needed.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Migrated from Amperstrand/tollgate-module-basic-go#92 as part of the fork retirement — OpenTollGate/tollgate-module-basic-go is the single source of truth; the fork issue carries the original discussion and any lab evidence. Retirement plan: see the tracking issue linked in the migration comment there.]
Factual divergence (no recommendation attached — decision for the team)
Surfaces from the SDK-vs-SDK-free equivalence spike on main @ d0cb92f (context: upstream OpenTollGate#371, #383).
The two packaging lanes stage the license text differently:
| Lane | License handling |
|---|---|
SDK-free .ipk (packaging/local-build-ipk.sh) |
Installs the GPL-3.0 text into the payload as /usr/share/doc/tollgate-wrt/LICENSE |
SDK .apk (packaging/Makefile, the CI apk lane) |
Metadata only (PKG_LICENSE:=GPL-3.0-only, PKG_LICENSE_FILES:=LICENSE); no license text inside the package |
Consequences (why it needs an explicit decision rather than a silent fix):
- Content divergence between formats:
.ipkartifacts ship the license text;.apkartifacts built by CI do not. Same version, different contents. - Blocks byte-equivalence between the lanes — this single file (~35 KB GPL text) is the only content difference the spike found between the two staging lists (everything else matches).
- GPL-distribution relevance: shipping the text alongside the binary is the compliance-conservative direction; OpenWrt feed packages commonly rely on metadata instead. Both positions are defensible — this is precisely not a call for an agent to make.
Possible outcomes (listed neutrally):
- Add the install line to
packaging/Makefile→ both lanes ship it (.apkgrows ~35 KB) - Remove the install from
local-build-ipk.sh→ neither ships the text, metadata only - Keep as-is and document the divergence (equivalence tooling then treats it as a known, allowed delta)
Until decided, the equivalence tooling mirrors the SDK lane (the designated oracle) and reports this delta loudly with a link here.
- Ngôn ngữ chính
- Go
- Star
- 12
- Fork
- 14
- Merge trung bình
- 3 ngày 21 giờ
- Pull request đã merge (30 ngày)
- 269
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của OpenTollGate/tollgate-module-basic-go
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
OpenTollGate/tollgate-module-basic-go#1135 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
OpenTollGate/tollgate-module-basic-go#1122 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
changelog: dependency-security sweep missing from [Unreleased] [Amperstrand fork #105 review debt]Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
OpenTollGate/tollgate-module-basic-go#1055 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
OpenTollGate/tollgate-module-basic-go#1027 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
OpenTollGate/tollgate-module-basic-go#1026 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của OpenTollGate/tollgate-module-basic-go
Issue tương tự
-
Dokumentasjon på nais defaultsĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
agent-butler-finding bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
jordansmall/spindrift#6161 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
open-telemetry/opentelemetry-go-compile-instrumentation#1635 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
C-bug O-community X-blathers-untriaged
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 84/100
cockroachdb/cockroach#176210 · 1 bình luận ·
-
VLAgent `spec.k8sCollector.includePodLabels: false` has no effectCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
VictoriaMetrics/operator#2772 ·
Maintainer thường phản hồi trong vòng 1 ngày