Bug#1034666: marked as done (unblock: mirrorbits/0.5.1+git20220308.9189dc7+ds1-1+b1)
Your message dated Sat, 22 Apr 2023 11:17:49 +0200
with message-id <ZEOmPaN3XlOgroKl@ramacher.at>
and subject line Re: Bug#1034666: unblock: mirrorbits/0.5.1+git20220308.9189dc7+ds1-1+b1
has caused the Debian Bug report #1034666,
regarding unblock: mirrorbits/0.5.1+git20220308.9189dc7+ds1-1+b1
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)
--
1034666: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1034666
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock
X-Debbugs-Cc: mirrorbits@packages.debian.org
Control: affects -1 + src:mirrorbits
Please unblock package mirrorbits
This changeset fixes the build of mirrorbits.
[ Reason ]
This is to fix FTBFS reported at https://bugs.debian.org/1026674. FTBFS
was due to changes in build dependencies, the package needed some minor
adjustments.
[ Impact ]
mirrorbits won't be included in bookworm
[ Tests ]
N/A
[ Risks ]
Low risk, this is a leaf package
[ Checklist ]
[x] all changes are documented in the d/changelog
[x] I reviewed all changes and I approve them
[x] attach debdiff against the package in testing
[ Other info ]
The package has not been uploaded to unstable yet.
unblock mirrorbits/0.5.1+git20220308.9189dc7+ds1-1+b1
diff -Nru mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/changelog mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/changelog
--- mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/changelog 2022-07-18 19:55:10.000000000 +0700
+++ mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/changelog 2023-04-21 11:20:56.000000000 +0700
@@ -1,3 +1,11 @@
+mirrorbits (0.5.1+git20220308.9189dc7+ds1-2) unstable; urgency=medium
+
+ * Replace transitional golang-goprotobuf-dev package
+ * Drop patch aimed at compatibility with golang-goprotobuf-dev 1.5.x
+ * Fix FTBFS with libprotobuf-dev 3.21.9 (Closes: #1026674)
+
+ -- Arnaud Rebillout <arnaudr@kali.org> Fri, 21 Apr 2023 11:20:56 +0700
+
mirrorbits (0.5.1+git20220308.9189dc7+ds1-1) unstable; urgency=medium
* Drop patches, applied upstream
diff -Nru mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/control mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/control
--- mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/control 2022-07-18 19:55:10.000000000 +0700
+++ mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/control 2023-04-21 11:20:56.000000000 +0700
@@ -10,6 +10,7 @@
dh-sequence-golang,
golang-any,
golang-github-coreos-go-systemd-dev,
+ golang-github-golang-protobuf-1-3-dev,
golang-github-gomodule-redigo-dev,
golang-github-howeyc-gopass-dev,
golang-github-op-go-logging-dev,
@@ -19,8 +20,8 @@
golang-golang-x-net-dev,
golang-google-grpc-dev,
golang-gopkg-tylerb-graceful.v1-dev,
- golang-goprotobuf-dev,
golang-yaml.v2-dev,
+ protoc-gen-go-1-3,
Standards-Version: 4.6.0
Vcs-Browser: https://salsa.debian.org/debian/mirrorbits
Vcs-Git: https://salsa.debian.org/debian/mirrorbits.git
diff -Nru mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/Add-Go-import-path-to-.proto-file.patch mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/Add-Go-import-path-to-.proto-file.patch
--- mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/Add-Go-import-path-to-.proto-file.patch 2022-07-18 19:55:10.000000000 +0700
+++ mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/Add-Go-import-path-to-.proto-file.patch 1970-01-01 08:00:00.000000000 +0800
@@ -1,52 +0,0 @@
-From: Arnaud Rebillout <arnaudr@kali.org>
-Date: Mon, 11 Jul 2022 14:59:38 +0200
-Subject: Add Go import path to .proto file
-MIME-Version: 1.0
-Content-Type: text/plain; charset=UTF-8
-Content-Transfer-Encoding: 8bit
-
-This is required for golang-goprotobuf-dev 1.5.2-1 (currently in Debian
-experimental), otherwise protoc fails with:
-
- protoc -I rpc --go_out=plugins=grpc:rpc rpc/rpc.proto
- protoc-gen-go: unable to determine Go import path for "rpc.proto"
-
- Please specify either:
- . a "go_package" option in the .proto source file, or
- . a "M" argument on the command line.
-
- See https://developers.google.com/protocol-buffers/docs/reference/go-generated#package for more information.
-
- --go_out: protoc-gen-go: Plugin failed with status code 1.
-
-The fancy value '.;rpc' comes from:
-https://github.com/golang/protobuf/issues/1102#issuecomment-619240905
-
-Actually the option should specify the full Go path, howeverif I do that,
-protoc generates the file rpc.pb.go in a completely different location, and the
-value of 'package' in rpc.pb.go becomes a full path, rather than just "rpc".
-
-I'm aiming at minimal changes here (because of my poor Go skills), so I prefer
-to use this trick in order to keep the generated file unchanged, as much as
-possible.
-
-Forwarded: not-needed, Debian-specific
----
- rpc/rpc.proto | 4 +++-
- 1 file changed, 3 insertions(+), 1 deletion(-)
-
-diff --git a/rpc/rpc.proto b/rpc/rpc.proto
-index fed9eb3..b5b9eae 100644
---- a/rpc/rpc.proto
-+++ b/rpc/rpc.proto
-@@ -3,6 +3,8 @@ syntax = "proto3";
- import "google/protobuf/empty.proto";
- import "google/protobuf/timestamp.proto";
-
-+option go_package = ".;rpc";
-+
- service CLI {
- rpc GetVersion (google.protobuf.Empty) returns (VersionReply) {}
- rpc Upgrade (google.protobuf.Empty) returns (google.protobuf.Empty) {}
---
-2.36.1
diff -Nru mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/series mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/series
--- mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/series 2022-07-18 19:55:10.000000000 +0700
+++ mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/series 1970-01-01 08:00:00.000000000 +0800
@@ -1 +0,0 @@
-Add-Go-import-path-to-.proto-file.patch
diff -Nru mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/rules mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/rules
--- mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/rules 2022-07-18 19:55:10.000000000 +0700
+++ mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/rules 2023-04-21 11:20:56.000000000 +0700
@@ -16,7 +16,7 @@
execute_after_dh_auto_configure:
cd _build/src/$(PKG) && \
find -name "*.pb.go" -delete -printf "removed %p\n" && \
- protoc -I rpc --go_out=plugins=grpc:rpc rpc/rpc.proto
+ protoc -I rpc --go_out=plugins=grpc,Mgoogle/protobuf/empty.proto=github.com/golang/protobuf/ptypes/empty,Mgoogle/protobuf/timestamp.proto=github.com/golang/protobuf/ptypes/timestamp:rpc rpc/rpc.proto
override_dh_auto_build:
dh_auto_build -- $(GO_BUILDFLAGS) -ldflags '$(GO_LDFLAGS)'
--- End Message ---
--- Begin Message ---
Hi Arnaud
On 2023-04-21 14:45:18 +0700, Arnaud Rebillout wrote:
> Package: release.debian.org
> Severity: normal
> User: release.debian.org@packages.debian.org
> Usertags: unblock
> X-Debbugs-Cc: mirrorbits@packages.debian.org
> Control: affects -1 + src:mirrorbits
>
> Please unblock package mirrorbits
>
> This changeset fixes the build of mirrorbits.
>
> [ Reason ]
>
> This is to fix FTBFS reported at https://bugs.debian.org/1026674. FTBFS
> was due to changes in build dependencies, the package needed some minor
> adjustments.
>
> [ Impact ]
>
> mirrorbits won't be included in bookworm
The package was removed from bookworm in January and we stopped allowing
new packages migrating to bookworm with the start of the soft freeze in
February. I'm afraid it's too late.
Please consider providing a backport once bookworm is released.
Cheers
>
> [ Tests ]
>
> N/A
>
> [ Risks ]
>
> Low risk, this is a leaf package
>
> [ Checklist ]
> [x] all changes are documented in the d/changelog
> [x] I reviewed all changes and I approve them
> [x] attach debdiff against the package in testing
>
> [ Other info ]
>
> The package has not been uploaded to unstable yet.
>
> unblock mirrorbits/0.5.1+git20220308.9189dc7+ds1-1+b1
> diff -Nru mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/changelog mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/changelog
> --- mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/changelog 2022-07-18 19:55:10.000000000 +0700
> +++ mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/changelog 2023-04-21 11:20:56.000000000 +0700
> @@ -1,3 +1,11 @@
> +mirrorbits (0.5.1+git20220308.9189dc7+ds1-2) unstable; urgency=medium
> +
> + * Replace transitional golang-goprotobuf-dev package
> + * Drop patch aimed at compatibility with golang-goprotobuf-dev 1.5.x
> + * Fix FTBFS with libprotobuf-dev 3.21.9 (Closes: #1026674)
> +
> + -- Arnaud Rebillout <arnaudr@kali.org> Fri, 21 Apr 2023 11:20:56 +0700
> +
> mirrorbits (0.5.1+git20220308.9189dc7+ds1-1) unstable; urgency=medium
>
> * Drop patches, applied upstream
> diff -Nru mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/control mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/control
> --- mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/control 2022-07-18 19:55:10.000000000 +0700
> +++ mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/control 2023-04-21 11:20:56.000000000 +0700
> @@ -10,6 +10,7 @@
> dh-sequence-golang,
> golang-any,
> golang-github-coreos-go-systemd-dev,
> + golang-github-golang-protobuf-1-3-dev,
> golang-github-gomodule-redigo-dev,
> golang-github-howeyc-gopass-dev,
> golang-github-op-go-logging-dev,
> @@ -19,8 +20,8 @@
> golang-golang-x-net-dev,
> golang-google-grpc-dev,
> golang-gopkg-tylerb-graceful.v1-dev,
> - golang-goprotobuf-dev,
> golang-yaml.v2-dev,
> + protoc-gen-go-1-3,
> Standards-Version: 4.6.0
> Vcs-Browser: https://salsa.debian.org/debian/mirrorbits
> Vcs-Git: https://salsa.debian.org/debian/mirrorbits.git
> diff -Nru mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/Add-Go-import-path-to-.proto-file.patch mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/Add-Go-import-path-to-.proto-file.patch
> --- mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/Add-Go-import-path-to-.proto-file.patch 2022-07-18 19:55:10.000000000 +0700
> +++ mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/Add-Go-import-path-to-.proto-file.patch 1970-01-01 08:00:00.000000000 +0800
> @@ -1,52 +0,0 @@
> -From: Arnaud Rebillout <arnaudr@kali.org>
> -Date: Mon, 11 Jul 2022 14:59:38 +0200
> -Subject: Add Go import path to .proto file
> -MIME-Version: 1.0
> -Content-Type: text/plain; charset=UTF-8
> -Content-Transfer-Encoding: 8bit
> -
> -This is required for golang-goprotobuf-dev 1.5.2-1 (currently in Debian
> -experimental), otherwise protoc fails with:
> -
> - protoc -I rpc --go_out=plugins=grpc:rpc rpc/rpc.proto
> - protoc-gen-go: unable to determine Go import path for "rpc.proto"
> -
> - Please specify either:
> - . a "go_package" option in the .proto source file, or
> - . a "M" argument on the command line.
> -
> - See https://developers.google.com/protocol-buffers/docs/reference/go-generated#package for more information.
> -
> - --go_out: protoc-gen-go: Plugin failed with status code 1.
> -
> -The fancy value '.;rpc' comes from:
> -https://github.com/golang/protobuf/issues/1102#issuecomment-619240905
> -
> -Actually the option should specify the full Go path, howeverif I do that,
> -protoc generates the file rpc.pb.go in a completely different location, and the
> -value of 'package' in rpc.pb.go becomes a full path, rather than just "rpc".
> -
> -I'm aiming at minimal changes here (because of my poor Go skills), so I prefer
> -to use this trick in order to keep the generated file unchanged, as much as
> -possible.
> -
> -Forwarded: not-needed, Debian-specific
> ----
> - rpc/rpc.proto | 4 +++-
> - 1 file changed, 3 insertions(+), 1 deletion(-)
> -
> -diff --git a/rpc/rpc.proto b/rpc/rpc.proto
> -index fed9eb3..b5b9eae 100644
> ---- a/rpc/rpc.proto
> -+++ b/rpc/rpc.proto
> -@@ -3,6 +3,8 @@ syntax = "proto3";
> - import "google/protobuf/empty.proto";
> - import "google/protobuf/timestamp.proto";
> -
> -+option go_package = ".;rpc";
> -+
> - service CLI {
> - rpc GetVersion (google.protobuf.Empty) returns (VersionReply) {}
> - rpc Upgrade (google.protobuf.Empty) returns (google.protobuf.Empty) {}
> ---
> -2.36.1
> diff -Nru mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/series mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/series
> --- mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/series 2022-07-18 19:55:10.000000000 +0700
> +++ mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/patches/series 1970-01-01 08:00:00.000000000 +0800
> @@ -1 +0,0 @@
> -Add-Go-import-path-to-.proto-file.patch
> diff -Nru mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/rules mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/rules
> --- mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/rules 2022-07-18 19:55:10.000000000 +0700
> +++ mirrorbits-0.5.1+git20220308.9189dc7+ds1/debian/rules 2023-04-21 11:20:56.000000000 +0700
> @@ -16,7 +16,7 @@
> execute_after_dh_auto_configure:
> cd _build/src/$(PKG) && \
> find -name "*.pb.go" -delete -printf "removed %p\n" && \
> - protoc -I rpc --go_out=plugins=grpc:rpc rpc/rpc.proto
> + protoc -I rpc --go_out=plugins=grpc,Mgoogle/protobuf/empty.proto=github.com/golang/protobuf/ptypes/empty,Mgoogle/protobuf/timestamp.proto=github.com/golang/protobuf/ptypes/timestamp:rpc rpc/rpc.proto
>
> override_dh_auto_build:
> dh_auto_build -- $(GO_BUILDFLAGS) -ldflags '$(GO_LDFLAGS)'
--
Sebastian Ramacher
--- End Message ---
Reply to: