All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: Ismael Luceno <ismael@iodev.co.uk>
Cc: buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH] package/pounce: New package
Date: Mon, 14 Aug 2023 13:59:17 +0200	[thread overview]
Message-ID: <20230814115917.GD421096@scaer> (raw)
In-Reply-To: <20230814113637.28134-1-ismael@iodev.co.uk>

Ismael, All,

On 2023-08-14 13:36 +0200, Ismael Luceno spake thusly:
> Simple IRC bouncer.
> 
> Signed-off-by: Ismael Luceno <ismael@iodev.co.uk>

You did not take into account a single one of the comments I made in
my previous review:
    https://lore.kernel.org/buildroot/20230712193831.GI188780@scaer/

Please read that review and act upon the comments before you submit
again.

If you have some concerns or questions about some of what I said, please
address those by replying to my review mail.

Regards,
Yann E. MORIN.

> ---
>  DEVELOPERS                 |  1 +
>  package/Config.in          |  1 +
>  package/pounce/Config.in   | 18 ++++++++++++++++++
>  package/pounce/pounce.hash |  3 +++
>  package/pounce/pounce.mk   | 14 ++++++++++++++
>  5 files changed, 37 insertions(+)
>  create mode 100644 package/pounce/Config.in
>  create mode 100644 package/pounce/pounce.hash
>  create mode 100644 package/pounce/pounce.mk
> 
> diff --git a/DEVELOPERS b/DEVELOPERS
> index 6ffa3ee693a8..4dad3d474ad0 100644
> --- a/DEVELOPERS
> +++ b/DEVELOPERS
> @@ -1338,6 +1338,7 @@ F:	package/exempi/
>  N:	Ismael Luceno <ismael@iodev.co.uk>
>  F:	package/axel/
>  F:	package/mawk/
> +F:	package/pounce/
>  
>  N:	Jagan Teki <jagan@amarulasolutions.com>
>  F:	board/amarula/
> diff --git a/package/Config.in b/package/Config.in
> index 54cddc39140d..783683dd5201 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -2458,6 +2458,7 @@ endif
>  	source "package/phytool/Config.in"
>  	source "package/pimd/Config.in"
>  	source "package/pixiewps/Config.in"
> +	source "package/pounce/Config.in"
>  	source "package/pound/Config.in"
>  	source "package/pppd/Config.in"
>  	source "package/pptp-linux/Config.in"
> diff --git a/package/pounce/Config.in b/package/pounce/Config.in
> new file mode 100644
> index 000000000000..83bb66643652
> --- /dev/null
> +++ b/package/pounce/Config.in
> @@ -0,0 +1,18 @@
> +config BR2_PACKAGE_POUNCE
> +	bool "pounce"
> +	select BR2_PACKAGE_LIBRESSL
> +	help
> +	  pounce is a multi-client, TLS-only IRC bouncer. It maintains a
> +	  persistent connection to an IRC server, acting as a proxy and
> +	  buffer for a number of clients. When a client connects, any
> +	  messages received since it last disconnected will be relayed to
> +	  it. Unlike some other bouncers, pounce uses a single buffer for
> +	  all IRC messages, which acts as a queue from which each client
> +	  reads messages independently.
> +
> +	  pounce speaks regular modern IRC to both servers and clients, using
> +	  the server-time extension to indicate when messages originally
> +	  occurred. Clients identify themselves to pounce by their IRC
> +	  usernames.
> +
> +	  https://git.causal.agency/pounce/
> diff --git a/package/pounce/pounce.hash b/package/pounce/pounce.hash
> new file mode 100644
> index 000000000000..a827bd389f40
> --- /dev/null
> +++ b/package/pounce/pounce.hash
> @@ -0,0 +1,3 @@
> +# Locally computed
> +sha256  97f245556b1cc940553fca18f4d7d82692e6c11a30f612415e5e391e5d96604e  pounce-3.1.tar.gz
> +sha256  8ceb4b9ee5adedde47b31e975c1d90c73ad27b6b165a1dcd80c7c545eb65b903  LICENSE
> diff --git a/package/pounce/pounce.mk b/package/pounce/pounce.mk
> new file mode 100644
> index 000000000000..b0a28efd91bd
> --- /dev/null
> +++ b/package/pounce/pounce.mk
> @@ -0,0 +1,14 @@
> +################################################################################
> +#
> +# pounce
> +#
> +################################################################################
> +
> +POUNCE_VERSION = 3.1
> +POUNCE_SITE = https://git.causal.agency/pounce/snapshot
> +POUNCE_SOURCE = pounce-$(POUNCE_VERSION).tar.gz
> +POUNCE_LICENSE = GPL-3.0+
> +POUNCE_LICENSE_FILES = LICENSE
> +POUNCE_DEPENDENCIES = libressl
> +
> +$(eval $(autotools-package))
> -- 
> 2.41.0
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2023-08-14 11:59 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-08-14 11:36 [Buildroot] [PATCH] package/pounce: New package Ismael Luceno
2023-08-14 11:59 ` Yann E. MORIN [this message]
2023-08-14 12:12   ` Ismael Luceno
  -- strict thread matches above, loose matches on Subject: below --
2023-07-08 19:36 Ismael Luceno
2023-07-12 19:38 ` Yann E. MORIN
2023-07-08 19:35 Ismael Luceno

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230814115917.GD421096@scaer \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@buildroot.org \
    --cc=ismael@iodev.co.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.