[Buildroot] [PATCH 1/1] Added keepalived package

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Tue Feb 7 16:46:27 UTC 2017


Hello,

On Tue,  7 Feb 2017 18:20:58 +0200, Ilias Apalodimas wrote:
> Signed-off-by: Ilias Apalodimas <apalos at gmail.com>

Thanks for this patch. This looks mostly good, so great for a first
contribution.

A few comments:

 - You don't need to send a series with a cover letter for just a
   single patch. Just send the patch itself.

 - Make a change to the DEVELOPERS file to associate your name/email to
   this package. This way, you'll get notified when/if this package
   fails to build, or when someone sends a patch touching this package.

 - You're missing a hash file. See other packages and the manual for
   the format/details.

> diff --git a/package/keepalived/Config.in b/package/keepalived/Config.in
> new file mode 100644
> index 0000000..ee4ab7a
> --- /dev/null
> +++ b/package/keepalived/Config.in
> @@ -0,0 +1,13 @@
> +config BR2_PACKAGE_KEEPALIVED
> +	bool "keepalived"
> +	select BR2_PACKAGE_OPENSSL
> +	select BR2_PACKAGE_POPT

Are you sure this builds even for no-MMU platforms, static linking
scenario? It's pretty likely that keepalived uses fork() and therefore
will not work on no-MMU platforms.

Yann E. Morin has just submitted a script today to help checking a
package, you may want to use it. It will be merged soon, which will
make things easier.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com



More information about the buildroot mailing list