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

Re: Bug#1059880: RFS: chr/0.1.76-1 [ITP] -- terminal-based text editor



On Wed, 2024-01-31 at 22:07 +0100, Christoph Hueffelmann wrote:
> Hi all,
> 
> we have updated the changelog to only one entry and the required version 
> of TuiWidgets has now arrived in "testing".
> 
> We hope that all remarks are done and would like to see chr sponsored.
> 
> Thanks in advance,
> Christoph
> 
> On Tue, 2 Jan 2024 20:19:57 +0100 Christoph Hueffelmann <chr@istoph.de> 
> wrote:
> > Package: sponsorship-requests
> > Severity: wishlist
> > X-Debbugs-CC:textshell@uchuujin.de
> > 
> > Dear mentors,
> > 
> > I am looking for a sponsor for my package "chr", yes, it's a short name,
> > but it's an editor that you use often:
> > 
> >   * Package name     : chr
> >     Version          : 0.1.76-1
> >     Upstream contact : Christoph Hueffelmann<chr@istoph.de>
> >   * URL              :https://github.com/istoph/editor
> >   * License          : Expat, BSL-1.0
> >   * Vcs              :https://salsa.debian.org/chr/chr
> >     Section          : editors
> > 
> > The source builds the following binary packages:
> > 
> >    chr - terminal-based text editor
> >    chr-tiny - terminal-based text editor - without syntax highlighting
> > 
> > To access further information about this package, please visit the following URL:
> > 
> >    https://mentors.debian.net/package/chr/
> > 
> > Alternatively, you can download the package with 'dget' using this command:
> > 
> >    dget -xhttps://mentors.debian.net/debian/pool/main/c/chr/chr_0.1.76-1.dsc
> > 
> > Changes for the initial release:
> > 
> >   chr (0.1.76-1) unstable; urgency=medium
> >   .
> >     * Fix: missing ifdef SYNTAX_HIGHLIGHTING for file commands
> >   .
> >   chr (0.1.75-1) unstable; urgency=medium
> >   .
> >     * Initial release.
> >       (Closes: #1059361)
> > 
> > This package still depends on tuiwidgets/0.2.1-1, which has also waiting for
> > sponsorship (#1057974).
> > 
> > I don't know what the internal process is like, perhaps both can be sponsored
> > together?
> > 
> > Regards,
> > -- 
> >    Christoph Hueffelmann
> 

Hi Christoph,

Basic review.

As stated on 'mentors':

The second line of the initial release changelog entry is not really required.

Initial releases should be '-1' i.e. 0.1.76-1.

Additional:

You have an obsolete Build-Depends 'pkg-config' this should be changed to 'pkgconf'.

Regards

Phil

-- 
Playing the game for the games sake.

Web:

* Debian Wiki: https://wiki.debian.org/PhilWyett
* Website: https://kathenas.org
* Social Debian: https://pleroma.debian.social/kathenas/
* Social Instagram: https://www.instagram.com/kathenasorg/


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


Reply to: