Re: [OE-core] [PATCH 30/44] byacc: remove the recipe

2017-03-13 Thread Mark Hatle
On 3/13/17 8:18 AM, Alexander Kanavin wrote:
> On 03/10/2017 06:37 PM, Mark Hatle wrote:
>> byacc, until recently, was required by the LSB / LSB Test Suite.
>>
>> So I'd rather is be kept in oe-core through the 2.3 release.
> 
> I couldn't find it in the LSB 5 standard. Is it something from earlier 
> versions?

It is in the instructions for the LSB Test Suite.  Byacc was required until
fairly recently.  From what I've found it was needed up until about September of
last year.  (There may be people doing CGL certification and such that still
need to use older versions of the LSB Test Suite for the time being.)

I have no objections (at this time) to remove it in 2.4, but I think it's too
later in 2.3 to remove/move it on someone.

--Mark

> Alex
> 

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 30/44] byacc: remove the recipe

2017-03-13 Thread Alexander Kanavin

On 03/10/2017 06:37 PM, Mark Hatle wrote:

byacc, until recently, was required by the LSB / LSB Test Suite.

So I'd rather is be kept in oe-core through the 2.3 release.


I couldn't find it in the LSB 5 standard. Is it something from earlier 
versions?


Alex

--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 30/44] byacc: remove the recipe

2017-03-10 Thread Mark Hatle
byacc, until recently, was required by the LSB / LSB Test Suite.

So I'd rather is be kept in oe-core through the 2.3 release.

--Mark

On 3/10/17 5:24 AM, Alexander Kanavin wrote:
> It was used only by rpm5 and will be moved to meta-oe
> 
> Signed-off-by: Alexander Kanavin 
> ---
>  meta/recipes-extended/byacc/byacc.inc  |  22 ---
>  .../0001-byacc-do-not-reorder-CC-and-CFLAGS.patch  | 161 
> -
>  meta/recipes-extended/byacc/byacc/byacc-open.patch |  25 
>  meta/recipes-extended/byacc/byacc_20161202.bb  |  12 --
>  .../packagegroup-core-full-cmdline.bb  |   1 -
>  5 files changed, 221 deletions(-)
>  delete mode 100644 meta/recipes-extended/byacc/byacc.inc
>  delete mode 100644 
> meta/recipes-extended/byacc/byacc/0001-byacc-do-not-reorder-CC-and-CFLAGS.patch
>  delete mode 100644 meta/recipes-extended/byacc/byacc/byacc-open.patch
>  delete mode 100644 meta/recipes-extended/byacc/byacc_20161202.bb
> 
> diff --git a/meta/recipes-extended/byacc/byacc.inc 
> b/meta/recipes-extended/byacc/byacc.inc
> deleted file mode 100644
> index adb07193d73..000
> --- a/meta/recipes-extended/byacc/byacc.inc
> +++ /dev/null
> @@ -1,22 +0,0 @@
> -SUMMARY = "Berkeley LALR Yacc parser generator"
> -DESCRIPTION = "A parser generator utility that reads a grammar specification 
> from a file and generates an LR(1) \
> -parser for it.  The parsers consist of a set of LALR(1) parsing tables and a 
> driver routine written in the C \
> -programming language."
> -SECTION = "devel"
> -LICENSE = "PD"
> -
> -SRC_URI = "ftp://invisible-island.net/byacc/byacc-${PV}.tgz \
> -   file://byacc-open.patch \
> -   file://0001-byacc-do-not-reorder-CC-and-CFLAGS.patch"
> -
> -EXTRA_OECONF += "--program-transform-name='s,^,b,'"
> -
> -BBCLASSEXTEND = "native"
> -
> -inherit autotools
> -
> -do_configure() {
> - install -m 0755 ${STAGING_DATADIR_NATIVE}/gnu-config/config.guess ${S}
> - install -m 0755 ${STAGING_DATADIR_NATIVE}/gnu-config/config.sub ${S}
> - oe_runconf
> -}
> diff --git 
> a/meta/recipes-extended/byacc/byacc/0001-byacc-do-not-reorder-CC-and-CFLAGS.patch
>  
> b/meta/recipes-extended/byacc/byacc/0001-byacc-do-not-reorder-CC-and-CFLAGS.patch
> deleted file mode 100644
> index 7cd2510c8ed..000
> --- 
> a/meta/recipes-extended/byacc/byacc/0001-byacc-do-not-reorder-CC-and-CFLAGS.patch
> +++ /dev/null
> @@ -1,161 +0,0 @@
> -Subject: byacc: do not reorder $CC and $CFLAGS
> -
> -byacc tries to process $CC and decide which part should belong to CC and 
> which
> -part should below to CFLAGS and then do reordering. It doesn't make much 
> sense
> -for OE. And it doesn't do its work correctly. Some options are dropped.
> -
> -Delete all these stuff so that we could have all options we need.
> -
> -Upstream-Status: Inappropriate [OE Specific]
> -
> -Signed-off-by: Chen Qi 
> 
> - aclocal.m4 |   1 -
> - configure  | 119 
> -
> - 2 files changed, 120 deletions(-)
> -
> -diff --git a/aclocal.m4 b/aclocal.m4
> -index 917a848..62ef241 100644
>  a/aclocal.m4
> -+++ b/aclocal.m4
> -@@ -1021,7 +1021,6 @@ CF_GCC_VERSION
> - CF_ACVERSION_CHECK(2.52,
> - [AC_PROG_CC_STDC],
> - [CF_ANSI_CC_REQD])
> --CF_CC_ENV_FLAGS
> - ])dnl
> - dnl 
> ---
> - dnl CF_PROG_GROFF version: 2 updated: 2015/07/04 11:16:27
> -diff --git a/configure b/configure
> -index 9707e50..4f0497c 100755
>  a/configure
> -+++ b/configure
> -@@ -1946,125 +1946,6 @@ esac
> - # This should have been defined by AC_PROG_CC
> - : ${CC:=cc}
> - 
> --echo "$as_me:1949: checking \$CC variable" >&5
> --echo $ECHO_N "checking \$CC variable... $ECHO_C" >&6
> --case "$CC" in
> --(*[\ \  ]-*)
> --echo "$as_me:1953: result: broken" >&5
> --echo "${ECHO_T}broken" >&6
> --{ echo "$as_me:1955: WARNING: your environment misuses the CC variable 
> to hold CFLAGS/CPPFLAGS options" >&5
> --echo "$as_me: WARNING: your environment misuses the CC variable to hold 
> CFLAGS/CPPFLAGS options" >&2;}
> --# humor him...
> --cf_flags=`echo "$CC" | sed -e 's/^.*[   ]\(-[^  ]\)/\1/'`
> --CC=`echo "$CC " | sed -e 's/[   ]-[^].*$//' -e 's/[ ]*$//'`
> --for cf_arg in $cf_flags
> --do
> --case "x$cf_arg" in
> --(x-[IUDfgOW]*)
> --
> --cf_fix_cppflags=no
> --cf_new_cflags=
> --cf_new_cppflags=
> --cf_new_extra_cppflags=
> --
> --for cf_add_cflags in $cf_flags
> --do
> --case $cf_fix_cppflags in
> --(no)
> --case $cf_add_cflags in
> --(-undef|-nostdinc*|-I*|-D*|-U*|-E|-P|-C)
> --case $cf_add_cflags in
> --(-D*)
> --cf_tst_cflags=`echo ${cf_add_cflags} |sed -e 
> 's/^-D[^=]*='\''\"[^"]*//'`
> --
> --test "x${cf_add_cflags}" != "x${cf_tst_cflags}" \
> --&& test -z "${cf_tst_cflags}" \
> --&& c