#1014473 vcswatch: track identity of committers of unuploaded commits

#1014473#5
Date:
2022-07-06 16:14:57 UTC
From:
To:
It would be great if vcswatch could track who the committers were of the
commits have not yet made it into the archive.

I'm curious about this as a way to track the number of janitor commits that
have not yet been uploaded, and to find packages I should help upload.

(Interested in working on a patch for vcswatch that does this)

#1014473#8
Date:
2022-07-22 10:53:35 UTC
From:
To:
Re: Jelmer Vernooij

Hi Jelmer,

sorry for the latish reply.

We are already extracting the not-yet-uploaded part of the git log
which is part of https://qa.debian.org/data/vcswatch/vcswatch.json.gz :

{
  "package_version": "0.03-2",
  "merge_requests": 0,
  "dumb_http": null,
  "issues": null,
  "commits": 16,
  "valid_checkout": 1,
  "tag": "debian/0.03-1.1",
  "url": "https://salsa.debian.org/perl-team/modules/packages/libio-socket-portstate-perl.git",
  "error": null,
  "avatar": null,
  "browser": "https://salsa.debian.org/perl-team/modules/packages/libio-socket-portstate-perl",
  "changelog_distribution": "UNRELEASED",
  "last_scan": "2022-07-18 03:14:08+00",
  "debian_dir": 1,
  "changelog_version": "0.03-2",
  "changelog": "libio-socket-portstate-perl (0.03-2) UNRELEASED; urgency=medium\n\n  [ Salvatore Bonaccorso ]\n  * Update Vcs-Browser URL to cgit web frontend\n  * debian/control: Use HTTPS transport protocol for Vcs-Git URI\n\n  [ gregor herrmann ]\n  * debian/copyright: change Copyright-Format 1.0 URL to HTTPS.\n  * Remove Piotr Roszatycki from Uploaders. Thanks for your work!\n\n  [ Salvatore Bonaccorso ]\n  * Update Vcs-* headers for switch to salsa.debian.org\n\n  [ gregor herrmann ]\n  * debian/watch: use uscan version 4.\n\n -- Salvatore Bonaccorso <carnil@debian.org>  Sat, 16 Aug 2014 09:32:32 +0200",
  "ci_url": "https://salsa.debian.org/perl-team/modules/packages/libio-socket-portstate-perl/-/pipelines",
  "vcs": "Git",
  "hash": "838c0802e0fa352208c1f8cad54a3b5cc1c83571",
  "branch": "master",
  "package": "libio-socket-portstate-perl",
  "ci_status": null,
  "edited_by": null,
  "next_scan": "2022-07-27 01:41:00+00",
  "status": "UNREL",
  "vcslog": "commit 838c0802e0fa352208c1f8cad54a3b5cc1c83571\nAuthor: Jelmer Vernooij <jelmer@jelmer.uk>\nDate:   Wed Jun 15 01:07:50 2022 +0100\n\n    Revert removal of .gbp.conf and .travis.yml.\n\ncommit 943cf41e6f49d494f52416595bdd44b53f767327\nAuthor: Jelmer Vernooij <jelmer@jelmer.uk>\nDate:   Wed Jun 15 01:04:05 2022 +0100\n\n  "
}

(abridged paste)

Does that work? I guess we could try extracting the authors
(committers?) into a proper json structure if that helps.

Helmut was approaching me about extracting even more fields from git,
Maintainer, Uploaders, Homepage, updated Vcs info, debian/watch, and
expose that for an easier feedback into the packages file without
requiring new uploads. That will likely happen shortly. (Mentioning
that here since it seems similar.)

Sure!

Christoph

#1014473#11
Date:
2022-08-08 15:14:44 UTC
From:
To:
Re: To Jelmer Vernooij
"controlfile" and "upstream_metadata" in the json:

  {
    "browser": "https://salsa.debian.org/debian/kanyremote",
    "vcslog": "commit 2461c1171fc9103e2fd9ec946208fe5e1bc2deb7\nAuthor: Philipp Huebner <debalance@debian.org>\n
Date:   Sat May 28 00:32:06 2022 +0200\n\n    Updated years in debian/copyright\n\ncommit a72ff16979f78545d0eb83
09370e47782536e4fd\nAuthor: Debian Janitor <janitor@jelmer.uk>\nDate:   Fri Sep 24 05:04:54 2021 +0000\n\n    Re
move obsolete field Name from debian/upstream/metadata (already present in machine-readable debian/copyright).\n
    \n    Changes-By: lintian-brush\n\ncommit 77bcda14a9bc556ecc7d5029ddf82282ff0c3303\nAuthor: Debian Janitor <
janitor@jelmer.uk>\nDate:   Fri Sep 24 05:04:43 2021 +0000\n\n    Trim trailing whitespace.\n    \n    Changes-B
y: lintian-brush\n    Fixes: lintian: trailing-whitespace\n    See-also: https://lintian.debian.org/tags/trailin
g-whitespace.html",
    "ci_url": "https://salsa.debian.org/debian/kanyremote/-/pipelines",
    "last_scan": "2022-08-04 18:18:12+00",
    "issues": null,
    "url": "https://salsa.debian.org/debian/kanyremote.git",
    "valid_checkout": 1,
    "changelog_version": "8.1-1.2",
    "watchfile": "version=4\nhttps://sf.net/anyremote/kanyremote-(.*)\\.tar\\.gz",
    "package": "kanyremote",
    "changelog_distribution": "UNRELEASED",
    "branch": "master",
    "merge_requests": 1,
    "vcs": "Git",
    "dumb_http": null,
    "controlfile": "Source: kanyremote\nSection: kde\nPriority: optional\nMaintainer: Philipp Huebner <debalance
@debian.org>\nBuild-Depends: debhelper-compat (= 13), dh-python, python3-all\nStandards-Version: 4.5.1\nRules-Re
quires-Root: no\nHomepage: http://anyremote.sourceforge.net\nVcs-Git: https://salsa.debian.org/debian/kanyremote
.git\nVcs-Browser: https://salsa.debian.org/debian/kanyremote\n\nPackage: kanyremote\nArchitecture: all\nDepends
: ${misc:Depends},\n         ${python3:Depends},\n         anyremote (>= 6.7),\n         python3-bluez (>= 0.9.1
),\n         python3-pyqt5\nRecommends: bluez\nDescription: KDE frontend for anyRemote\n kAnyRemote package is K
DE GUI frontend for anyRemote.\n (http://anyremote.sourceforge.net/). The overall goal of this project is to\n p
rovide remote control service on Linux through Bluetooth, InfraRed, Wi-Fi\n or TCP/IP connection.",
    "edited_at": null,
    "edited_by": null,
    "hash": "2461c1171fc9103e2fd9ec946208fe5e1bc2deb7",
    "debian_dir": 1,
    "changelog": "kanyremote (8.1-1.2) UNRELEASED; urgency=medium\n\n  * Trim trailing whitespace.\n  * Remove o
bsolete field Name from debian/upstream/metadata (already present in\n    machine-readable debian/copyright).\n\
n -- Debian Janitor <janitor@jelmer.uk>  Fri, 24 Sep 2021 05:04:43 -0000",
    "next_scan": "2022-08-12 12:11:00+00",
    "commits": 3,
    "package_version": "8.1-1.1",
    "ci_status": null,
    "status": "NEW",
    "upstream_metadata": "Bug-Database: https://sourceforge.net/p/anyremote/discussion/\nBug-Submit: https://sourceforge.net/p/anyremote/discussion/\nChangelog: https://sourceforge.net/p/anyremote/code/HEAD/tree/kanyremote/trunk/ChangeLog\nRepository: svn://svn.code.sf.net/p/anyremote/code/kanyremote/trunk\nRepository-Browse: https://sourceforge.net/p/anyremote/code/HEAD/tree/kanyremote/\nRegistration: https://sourceforge.net/user/registration\nContact: anyremote@mail.ru\nDocumentation: http://anyremote.sourceforge.net/docs.html\nFAQ: http://anyremote.sourceforge.net/faq.html",
    "avatar": "https://salsa.debian.org/uploads/-/system/project/avatar/1272/anyremote.png",
    "tag": "debian/8.1-1.1",
    "error": null
  },

Christoph

#1014473#16
Date:
2022-08-11 12:37:51 UTC
From:
To:
Thanks!

It would indeed be great if we could also extract the authors into a proper JSON structure so downstream consumers like the janitor don't have to implement that themselves for each VCS type.

#1014473#21
Date:
2023-09-20 17:09:11 UTC
From:
To:
review Mycelium Previ the Software wouldn't ha@gmail.com




Sent via the Samsung Galaxy Note9, an AT&T 5G Evolution capable smartphone

#1014473#26
Date:
2023-09-20 17:59:56 UTC
From:
To:
And not a PR to [Laravel Passport](https://github.com/laravel/passport)? This was [attempted](https://github.com/laravel/passport/pull/1433) in the past but it was decided to not be changed so this package




Sent via the Samsung Galaxy Note9, an AT&T 5G Evolution capable smartphone