Bug#1037174: RFS: damo/1.8.4-1 [ITP] -- Data Access Monitoring Operator

2023-06-14 Thread Michel Alexandre Salim
Hi Adam,

Thanks for all the feedback! I've addressed them all and updated to
1.8.5 which has been released in the meantime. Added comments inline - and
the additional question at the end.

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

https://salsa.debian.org/python-team/packages/damo/-/tree/44d7ecd2bf2af277e2b93be82c32ffcb242f5d17

On Mon, Jun 12, 2023 at 05:15:24PM +0200, Adam Borowski wrote:
> On Tue, Jun 06, 2023 at 09:45:25PM -0500, Michel Alexandre Salim wrote:
> 
> Hi!
> The commands you run for % targets will fail badly if ran in parallel, and
> that's the default these days.  Please run that only once, eg from
> override_dh_auto_configure (as that's a target that's run early).
> 
Fixed, thanks

> Other nice to have bits:
>  * the description doesn't say what DAMON is, even "Data Access Monitoring
>Operator" doesn't shed much light
Added description, thank you!

>  * a command-line tool really should have a man page
Added a manpage using help2man and checked in the generator script

One question: is it alright to mark the architecture here as "all"? The
code itself is pure Python, but it depends on functionality of the Linux
kernel. `linux-all` is not a valid target, and if I mark it as
`linux-any` it will... cause buildd to unnecessarily build this on each
architecture, right?

Best regards,

-- 
Michel Alexandre Salim
identities: https://keyoxide.org/5dce2e7e9c3b1cffd335c1d78b229d2f7ccc04f2


signature.asc
Description: PGP signature


Bug#1037174: RFS: damo/1.8.4-1 [ITP] -- Data Access Monitoring Operator

2023-06-12 Thread Adam Borowski
On Tue, Jun 06, 2023 at 09:45:25PM -0500, Michel Alexandre Salim wrote:
>  * Package name : damo
>Version  : 1.8.4-1
>Upstream contact : SeongJae Park 
>  * URL  : https://damonitor.github.io/
>  * Vcs  : https://salsa.debian.org/python-team/packages/damo

>  damo (1.8.4-1) unstable; urgency=medium
>  .
>* Initial release. (Closes: #1037157)

Hi!
The commands you run for % targets will fail badly if ran in parallel, and
that's the default these days.  Please run that only once, eg from
override_dh_auto_configure (as that's a target that's run early).

Other nice to have bits:
 * the description doesn't say what DAMON is, even "Data Access Monitoring
   Operator" doesn't shed much light
 * a command-line tool really should have a man page

(but only the first part is a blocker)


Meow!
-- 
⢀⣴⠾⠻⢶⣦⠀
⣾⠁⢠⠒⠀⣿⡁ ʀᴜꜱꜱɪᴀɴᴇꜱ ᴇᴜɴᴛ ᴅᴏᴍᴜꜱ
⢿⡄⠘⠷⠚⠋⠀
⠈⠳⣄



Bug#1037174: RFS: damo/1.8.4-1 [ITP] -- Data Access Monitoring Operator

2023-06-06 Thread Michel Alexandre Salim
Package: sponsorship-requests
Severity: wishlist

Dear mentors,

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

 * Package name : damo
   Version  : 1.8.4-1
   Upstream contact : SeongJae Park 
 * URL  : https://damonitor.github.io/
 * License  : GPL-2
 * Vcs  : https://salsa.debian.org/python-team/packages/damo
   Section  : devel

The source builds the following binary packages:

  damo - Data Access Monitoring Operator

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

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

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

  dget -x https://mentors.debian.net/debian/pool/main/d/damo/damo_1.8.4-1.dsc

Changes for the initial release:

 damo (1.8.4-1) unstable; urgency=medium
 .
   * Initial release. (Closes: #1037157)

Regards,

-- 
Michel Alexandre Salim
identities: https://keyoxide.org/5dce2e7e9c3b1cffd335c1d78b229d2f7ccc04f2


signature.asc
Description: PGP signature