From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] aiccu: remove package
Date: Sun, 10 Sep 2017 11:55:38 +0200 [thread overview]
Message-ID: <20170910095538.GE3536@scaer> (raw)
In-Reply-To: <20170910094317.1146-1-alexander.i.mukhin@gmail.com>
Alexander, All,
On 2017-09-10 12:43 +0300, Alexander Mukhin spake thusly:
> Signed-off-by: Alexander Mukhin <alexander.i.mukhin@gmail.com>
Thanks! :-)
Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Yet, a little nit-pick: why did you not add Michael in Cc of this patch,
since he's (was!) declared as developper for aiccu?
Also, I would have kept the reason in the commit log itself, because for
once, duplicating the info in the commit log and the help text does make
sense.
But no need to respin. If the committer sees fit, they'll add it back.
[--SNIP--]
> diff --git a/Config.in.legacy b/Config.in.legacy
> index 44303a42b..d4befa864 100644
> --- a/Config.in.legacy
> +++ b/Config.in.legacy
> @@ -145,6 +145,16 @@ endif
> ###############################################################################
> comment "Legacy options removed in 2017.11"
>
> +config BR2_PACKAGE_AICCU
> + bool "aiccu utility removed"
> + select BR2_LEGACY
> + help
> + As the SixXS project has ceased its operation on 2017-06-06,
> + the AICCU utility has no use anymore and has been removed.
> +
> + https://www.sixxs.net/sunset/
It's a bit sad they're not considering releasing the server part... :-/
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
next prev parent reply other threads:[~2017-09-10 9:55 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-09 9:25 [Buildroot] [PATCH] aiccu: delete package Alexander Mukhin
2017-09-09 10:09 ` Alexander Mukhin
2017-09-09 14:00 ` Thomas Petazzoni
2017-09-09 15:38 ` Yann E. MORIN
2017-09-10 9:43 ` [Buildroot] [PATCH] aiccu: remove package Alexander Mukhin
2017-09-10 9:55 ` Yann E. MORIN [this message]
2017-09-10 10:21 ` Alexander Mukhin
2017-09-11 9:27 ` Peter Korsgaard
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=20170910095538.GE3536@scaer \
--to=yann.morin.1998@free.fr \
--cc=buildroot@busybox.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox