[pkg-go] Fwd: Code review: golang-github-danverbraganza-varcaser

Anthony Fok foka at debian.org
Tue Jul 17 16:09:01 BST 2018


Hi Tong,

> Have you got around to take a look at my review request?

No, not in depth.  Sorry, I have been really busy in real life, and I
was hoping other pkg-go team members could help you review the
package.  ;-)

Anyhow, I took a quick look at
https://salsa.debian.org/go-team/packages/golang-github-danverbraganza-varcaser
which you committed two weeks ago and noticed these two problems:

 1. The pristine-tar is the default branch (either you or gbp pushed
the branches in the wrong order)

 2. There is no debian/0.0_git20151108.ce61ec4-1 tag

which indicates to me that you are not running the gbp commands in the
right order, or some other issues.

So, I went ahead and removed
https://salsa.debian.org/go-team/packages/golang-github-danverbraganza-varcaser
altogether so you can make a proper build and push to it properly
again.  Instructions below:

On Tue, Jul 17, 2018 at 2:23 AM, Anthony Fok <foka at debian.org> wrote:
> On Thu, Jul 5, 2018 at 2:24 PM, Tong Sun <pkgoyq.xpt at neverbox.com> wrote:
>> Hi,
>>
>> Can anyone delete the  https://salsa.debian.org/go-team/packages/ffcvt  repo
>> for me please?
>
> It has been deleted.  Please try and see if you can push your
> repository now, but note the following CAVEAT:
>
> Your first "gbp push" should be done AFTER you have successfully built
> the package a signed tag
> (e.g. debian/1.0.0-1) with the following command (or equivalent):
>
>     gbp buildpackage --git-pbuilder --git-sign-tags --git-tag
>
> For better or for worse, "gbp push" starts by pushing the debian
> version tag first, and if it doesn't find it, it will proceed to push
> the upstream tag (which the upstream branch contains).
> And Salsa (running GitLab) treats whatever it receives first as the
> default branch.
>
> If you don't want that to happen, and still want to upload your
> work-in-progress package to Salsa, I recommend that you manually push
> your "master" or "debian/sid" branch first by using "git push -u
> origin master" or "git push -u origin debian/sid".
>
> gbp is a great tool, but it is not omnipotent, and it is definitely
> not a replacement for good old "git" in many situations.

Cheers,
Anthony



More information about the Pkg-go-maintainers mailing list