Bug#1050814: RFS: qbootctl/0.1.2-1 [ITP] -- utility to control Quacomm A/B boot slots

2023-09-02 Thread Roger Shimizu
Thanks, Dmitry, for the prompt fix/update!

On Sat, Sep 2, 2023 at 2:09 PM Dmitry Baryshkov  wrote:
>
> On Sat, 2 Sept 2023 at 11:23, Roger Shimizu  wrote:
> > Thanks for your ITP & RFS!
> >
> > Some nitpicking:
> > * debian/README.Debian:
> >   Please add more description, or remove this file.
>
> Ack. I've updated the package  to drop this file (and also to include
> the qbootctl.service file to mark the boot as successful
> automatically).
>
> > * debian/include/scsi/scsi_bsg_ufs.h
> >   This header is not upstreamed yet?
>
> This header comes from the Linux kernel itself. It is a part of uABI.
>
> >   Is it possible to use existing debian packaged header?
>
> Unfortunately, see https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1050368

Above ticket is in pending status, so the next upload should fix the issue.
So please update this package accordingly then.

> > * Do you know whether this package works for QCOM LE / LU baseline systems?
>
> Most likely, the Linux kernel API and the bootloader are the same. But
> I don't think anybody tested it with LE / LU.

OK. We can test it easily when it hit Debian archive.

I'll upload it later.
But hope you can also fix the lintian in the future:

W: qbootctl: no-manual-page [usr/bin/qbootctl]
I: qbootctl: package-supports-alternative-init-but-no-init.d-script
[lib/systemd/system/qbootctl.service]
I: qbootctl: systemd-service-file-missing-documentation-key
[lib/systemd/system/qbootctl.service]

Thank you for your contribution to Debian!

Cheers,
Roger

>
> --
> With best wishes
> Dmitry



Bug#1050814: RFS: qbootctl/0.1.2-1 [ITP] -- utility to control Quacomm A/B boot slots

2023-09-02 Thread Dmitry Baryshkov
On Sat, 2 Sept 2023 at 11:23, Roger Shimizu  wrote:
> Thanks for your ITP & RFS!
>
> Some nitpicking:
> * debian/README.Debian:
>   Please add more description, or remove this file.

Ack. I've updated the package  to drop this file (and also to include
the qbootctl.service file to mark the boot as successful
automatically).

> * debian/include/scsi/scsi_bsg_ufs.h
>   This header is not upstreamed yet?

This header comes from the Linux kernel itself. It is a part of uABI.

>   Is it possible to use existing debian packaged header?

Unfortunately, see https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1050368

> * Do you know whether this package works for QCOM LE / LU baseline systems?

Most likely, the Linux kernel API and the bootloader are the same. But
I don't think anybody tested it with LE / LU.

-- 
With best wishes
Dmitry



Bug#1050814: RFS: qbootctl/0.1.2-1 [ITP] -- utility to control Quacomm A/B boot slots

2023-09-02 Thread Roger Shimizu
control: owner -1 !
control: tags -1 +moreinfo

Dear Dmitry,

Thanks for your ITP & RFS!

Some nitpicking:
* debian/README.Debian:
  Please add more description, or remove this file.

* debian/include/scsi/scsi_bsg_ufs.h
  This header is not upstreamed yet?
  Is it possible to use existing debian packaged header?

* Do you know whether this package works for QCOM LE / LU baseline systems?

Cheers,
Roger



Bug#1050814: RFS: qbootctl/0.1.2-1 [ITP] -- utility to control Quacomm A/B boot slots

2023-08-29 Thread Dmitry Baryshkov
Package: sponsorship-requests
Severity: wishlist
X-Debbugs-Cc: debian-on-mobile-maintain...@alioth-lists.debian.net

Dear mentors,

I am looking for a sponsor for my package "qbootctl":

 * Package name : qbootctl
   Version  : 0.1.2-1
   Upstream contact : Caleb Connolly 
 * URL  : https://gitlab.com/sdm845-mainline/qbootctl
 * License  : GPL-2-Linux-syscall-note, BSD-3-clause, BSD-3-Clause, 
GPL-3+
 * Vcs  : https://salsa.debian.org/lumag/qbootctl
   Section  : utils

The source builds the following binary packages:

  qbootctl - utility to control Quacomm A/B boot slots

To access further information about this package, please visit the following 
URL:

  https://mentors.debian.net/package/qbootctl/

Alternatively, you can download the package with 'dget' using this command:

  dget -x 
https://mentors.debian.net/debian/pool/main/q/qbootctl/qbootctl_0.1.2-1.dsc

Changes for the initial release:

 qbootctl (0.1.2-1) unstable; urgency=medium
 .
   * Initial release. (Closes: #1050354)

-- 
With best wishes
Dmitry