Bug#941016: ITP: gnome-firmware-updater -- GTK tool to upgrade, downgrade, and reinstall firmware on devices supported by fwupd
Hi Mike,
Thank you for a detailed review!
> Hi Jesper,
> (Cc:ing Martin Wimpress from Ubuntu MATE)
>
>
> > I'm ok with that. Thank you for taking your time, I will do my best to
> > help. I've applied some upstream patches that fix
> > the manpage issue and i've also taken care of the lintian issues.
> > looking forward to hearing your review comments soon!
>
> Here comes the review:
>
> 1.
> src:pkg name: upstream names it gnome-firmware-updater, so why do you
> name it gnome-firmware
>
> 2.
> bin:pkg name: same as with src:pkg name
>
> 3.
> debian/control:
>
> - SYNOPSIS: gtk front end for fwupd -> GTK
> - LONG_DESCRIPTION: indent the itemizations by two characters
> - LONG_DESCRIPTION: why don't you use the wording from
> upstream's README.md file? Please do.
>
> https://gitlab.gnome.org/hughsie/gnome-firmware-updater/blob/master/README.md
> - Build-Depends: please add a comma after the last pkg name, it
> makes diffs look nicer
> one the line below Build-Depends gets changed (e.g. with another pkg name)
> - Vcs-*: fields are missing, we should put the pkg on salsa under
> the debian/ folder
> - Depends: field, I recommend having one package per row.
> - Please use appropriate upper case and lower case letters
>
> debian/copyright:
>
> - personally, I don't like the "Files: *" debian/copyright style,
> but ftpmaster
> wave it through, so...
> - Source: field: no URL with a release number here, I'd put the
> releases/ folder
> here as URL or even better, the GitLab repo.
>
> A general thing: you did good, but for other packages you might
> maintain... I normally
> use all licenses appearing in the upstream code for debian/.
>
>
> debian/changelog:
>
> - my spelling: "Initial release to Debian. (Closes: #941016)."
> -> thus, using full-stops
>
> 4.
> The pristine-tar branch is not on salsa, I cannot build without that
> branch using gbp-buildpackage.
>
> There are three options now:
>
> (a) you work on those bits and pieces and I take a second look
> (b) you are ok with me adding some commits
> (c) a mixture of (a) and (b)
>
> We should created an empty Git repo (gnome-firmware-updater) under
> salsa.debian.org/debian (I can do that, if you agree) and push a clone
> of your Git repo there. Please let me know if you are ok with
> continueing using that repository as the official packaging VCS
> location.
>
> Another option would be placing the packaging Git repo onto the Debian
> GNOME team's namespace on GitLab, but gnome-firmware-updater has
> "gnome" in its name, but as I learned from upstream, it is a suitable
> add-on tool for all GTK-based desktop environments, so a "neutral"
> spot like the /debian team folder salsa is probably the better way to
> go.
>
> Also, Martin Wimpress from Ubuntu MATE and I would like to add
> ourselves as uploaders. Furthermore, I recommend adding the Debian
> GNOME team and the Debian+Ubuntu MATE team as uploaders, too. In old
> times, most Debian packages had been 1-person-maintained, which turned
> out to be a bad concept in case devs went missing in action (MIA).
> Allowing team uploads to packages makes Debian a much more flexible
> place to be at. Please let me know, if you are ok with this, too.
>
> Thanks for your work!
> Mike
Regarding the naming of the package, I've based it on the Fedora/openSUSE package and this upstream commit:
https://gitlab.gnome.org/hughsie/gnome-firmware-updater/commit/55705a55e930e4ecb14fc5ae122bbbad9042f03b
It seems like this is the current and future package name so I suggest keeping it.
If you're okay with it, I'd be happy to take the time and fix these issues myself, that way I can learn how to do these
things properly (so, option a). I'll try to do it as soon as possible.
I'm okay with moving the repo under /debian/gnome-firmware but perhaps I could tidy up a bit first. I plan to use the
dep14 structure like they do with the gnome-shell package. Perhaps we can move it after the initial packaging is complete?
Are you okay with having a second look once I have a new revision ready? I'll let you know when it's done. I'll make sure
to add you and Martin as uploaders by then.
Best regards,
Jesper Derander
Reply to: