[yocto] [meta-fsl-ppc master] inetutils_1.8.bbappend:task do_configure and do_compile:failed

[McClintock Matthew-B29882]

On Tue, Oct 30, 2012 at 1:55 AM,  <b40290 at freescale.com> wrote:
> From: Chunrong Guo <b40290 at freescale.com>
>
>      fix inetutils task do_configure:failed and
>                    task do_compile:failed

At a minimum give some hint what the errors were.

> Signed-off-by: Chunrong Guo <b40290 at freescale.com>
> ---
>  .../files/disable-pre-ANSI-compilers.patch         |   11 +++++++++++
>  recipes-append/inetutils/files/remove_gets.patch   |   13 +++++++++++++
>  recipes-append/inetutils/inetutils_1.8.bbappend    |    8 ++++++++
>  3 files changed, 32 insertions(+)
>  create mode 100644 recipes-append/inetutils/files/disable-pre-ANSI-compilers.patch
>  create mode 100644 recipes-append/inetutils/files/remove_gets.patch
>  create mode 100644 recipes-append/inetutils/inetutils_1.8.bbappend

I think the patches belong in meta-openembedded not in meta-fsl-ppc.
Am I missing something here?

-M

>
> diff --git a/recipes-append/inetutils/files/disable-pre-ANSI-compilers.patch b/recipes-append/inetutils/files/disable-pre-ANSI-compilers.patch
> new file mode 100644
> index 0000000..b945f3a
> --- /dev/null
> +++ b/recipes-append/inetutils/files/disable-pre-ANSI-compilers.patch
> @@ -0,0 +1,11 @@
> +--- inetutils-1.8.org/configure.ac     2012-10-23 02:05:20.516386701 -0500
> ++++ inetutils-1.8/configure.ac 2012-10-25 21:31:10.602591484 -0500
> +@@ -569,7 +569,7 @@
> + IU_CHECK_MEMBERS([struct stat.st_blksize])
> +
> + ### Checks for compiler characteristics.
> +-AM_C_PROTOTYPES dnl FIXME: Does inetutils even compile on pre-ANSI compilers?
> ++#AM_C_PROTOTYPES dnl FIXME: Does inetutils even compile on pre-ANSI compilers?
> + AC_C_CONST
> +
> + dnl See if `weak refs' are possible; these make it possible (with shared
> diff --git a/recipes-append/inetutils/files/remove_gets.patch b/recipes-append/inetutils/files/remove_gets.patch
> new file mode 100644
> index 0000000..f8081a2
> --- /dev/null
> +++ b/recipes-append/inetutils/files/remove_gets.patch
> @@ -0,0 +1,13 @@
> +--- inetutils-1.8.org/lib/stdio.in.h   2010-05-15 06:35:16.000000000 -0500
> ++++ inetutils-1.8/lib/stdio.in.h       2012-10-25 22:56:43.198607472 -0500
> +@@ -138,8 +138,10 @@
> + /* It is very rare that the developer ever has full control of stdin,
> +    so any use of gets warrants an unconditional warning.  Assume it is
> +    always declared, since it is required by C89.  */
> ++#if defined gets
> + #undef gets
> + _GL_WARN_ON_USE (gets, "gets is a security hole - use fgets instead");
> ++#endif
> +
> + #if @GNULIB_FOPEN@
> + # if @REPLACE_FOPEN@
> diff --git a/recipes-append/inetutils/inetutils_1.8.bbappend b/recipes-append/inetutils/inetutils_1.8.bbappend
> new file mode 100644
> index 0000000..c64f8cf
> --- /dev/null
> +++ b/recipes-append/inetutils/inetutils_1.8.bbappend
> @@ -0,0 +1,8 @@
> +FILESEXTRAPATHS_prepend_fsl := "${THISDIR}/files:"
> +
> +SRC_URI_append_fsl += "file://disable-pre-ANSI-compilers.patch \
> +                       file://remove_gets.patch \
> +                        "
> +
> +PR_fsl .= "+${DISTRO}"
> +
> --
> 1.7.9.7
>
>
> _______________________________________________
> yocto mailing list
> yocto at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto