On Wed, Apr 8, 2026 at 7:50 PM Andrew Lee <[email protected]> wrote:
>
> Hi Shengjing,
>
> Thank you for your intpu.
>
> On Wed, Apr 8, 2026 at 1:02 PM Shengjing Zhu <[email protected]> wrote:
> > I haven't touched it for some time so it may be inaccurate. There is a
> > `curl` script in debian/ dir. You can first build the package in an
> > online environment, and the regenerate.sh script in
> > execute_after_dh_auto_configure will invoke that `curl` script to
> > download missing source files.
>
> The `execute_after_dh_auto_configure` target actully does:
> - find and delete `*.pb.go` files
> - invoke `scripts/regenerate.sh` to regenerate `*.pb.go` files from
> exist .proto files under `d/missing-sources/` without download new
> files from upstream.
>
> I checked
> debian/patches/0001-Adapt-regenerate.sh-for-packaging.patch contains
> lines with `curl`. And then `scripts/regenerate.sh` after the patch
> applied during `dh_auto_configure` contains no `curl` lines.
>
> I found your related changes here:
>   
> https://salsa.debian.org/go-team/packages/golang-google-grpc/-/commit/49b9240cbb8a28b40a85304b8c701b5eafd9dcc9
>
> It seems the part of `scripts/regenerate.sh` design for downloading
> from upstream is broken in `execute_after_dh_auto_configure` target.
> However, it's also hard for me to understnad the single commit if it's
> not really broken?
>

Looks like it's broken. Maybe I found the script changed to use git
and the old curl[1] script did work. And I haven't figured out how to
adapt that as well.

[1] 
https://salsa.debian.org/go-team/packages/golang-google-grpc/-/blob/debian/sid/debian/curl?ref_type=heads

-- 
Shengjing Zhu

Reply via email to