[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1016624: marked as done (RFS: psi-notify/1.3.0-1 -- Alert when your machine is becoming oversaturated)

Your message dated Thu, 04 Aug 2022 14:17:55 +0800
with message-id <7b89d183a3e63900ef6f04e84705f521a2bf7ac6.camel@debian.org>
and subject line Re: Bug#1016624: RFS: psi-notify/1.3.0-1 -- Alert when your machine is becoming oversaturated
has caused the Debian Bug report #1016624,
regarding RFS: psi-notify/1.3.0-1 -- Alert when your machine is becoming oversaturated
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
1016624: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1016624
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "psi-notify":

 * Package name    : psi-notify
   Version         : 1.3.0-1
   Upstream Author : Chris Down <chris@chrisdown.name>
 * URL             : https://github.com/cdown/psi-notify
 * License         : Expat
 * Vcs             : https://salsa.debian.org/michel/psi-notify
   Section         : utils

The source builds the following binary packages:

  psi-notify - Alert when your machine is becoming oversaturated

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

  https://mentors.debian.net/package/psi-notify/

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

  dget -x https://mentors.debian.net/debian/pool/main/p/psi-notify/psi-notify_1.3.0-1.dsc

Changes since the last upload:

 psi-notify (1.3.0-1) unstable; urgency=low
 .
   * Update to 1.3.0
   * add `make install`
   * `README.md`: update link to desktop notifications spec
   * `README.md`: add links to Debian and Ubuntu packages and Repology

Regards,

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

Attachment: signature.asc
Description: PGP signature


--- End Message ---
--- Begin Message --- 
On Wed, 2022-08-03 at 21:24 -0500, Michel Alexandre Salim wrote:

>  psi-notify (1.3.0-1) unstable; urgency=low

Uploaded.

Some things you might want to fix or talk to upstream/etc about:

Please add some patch headers using DEP-3, especially Forwarded:

https://dep-team.pages.debian.net/deps/dep3/

I suggest using pkg-config to find the install dir for the systemd user
service instead of hardcoding the path to the user service install dir.

Some parts of the test suite use bash but could use POSIX sh,
or be easily rewritten to support POSIX sh.

I note that the code uses PATH_MAX, even though on Linux PATH_MAX is
essentially meaningless. I'd encourage switching to malloc.

   https://insanecoding.blogspot.com/2007/11/pathmax-simply-isnt.html

There is one lintian false positive you could report:

   P: psi-notify source: very-long-line-length-in-source-file 1846 > 512 [demo.gif:4272]

There are some tools that print valid warnings for this package:

   $ lintian
   $ cppcheck -j1 --quiet --enable=warning .
   $ cme check dpkg
   $ yamllint .travis.yml

-- 
bye,
pabs

https://wiki.debian.org/PaulWise

Attachment: signature.asc
Description: This is a digitally signed message part


--- End Message ---
Reply to: