All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] eglibc-use-option-groups.patch: Various fixups
@ 2014-11-23 17:44 Juro Bystricky
  2015-01-22 20:52 ` Juro Bystricky
  0 siblings, 1 reply; 4+ messages in thread
From: Juro Bystricky @ 2014-11-23 17:44 UTC (permalink / raw)
  To: openembedded-core

Build suppport for mutibyte character handling only when
__OPTION_EGLIBC_LOCALE_CODE evaluates to 1.
Fixes missing .out suffix for several tests to be built.
Fixes building of locales needed for several tests. Do not use
cross-localedef to build locales. Use localedef built with the
newly built libgc instead.

Fixes:
[YOCTO #6809] [YOCTO #6796] [YOCTO #6797]

Signed-off-by: Juro Bystricky <jurobystricky@hotmail.com>
---
 .../glibc/glibc/eglibc-use-option-groups.patch     | 43 ++--------------------
 1 file changed, 4 insertions(+), 39 deletions(-)

diff --git a/meta/recipes-core/glibc/glibc/eglibc-use-option-groups.patch b/meta/recipes-core/glibc/glibc/eglibc-use-option-groups.patch
index 40c7832..0e2f0c6 100644
--- a/meta/recipes-core/glibc/glibc/eglibc-use-option-groups.patch
+++ b/meta/recipes-core/glibc/glibc/eglibc-use-option-groups.patch
@@ -2434,24 +2434,7 @@ Index: git/localedata/Makefile
  
  ifeq ($(run-built-tests),yes)
  # We have to generate locales
-@@ -143,9 +161,13 @@
- $(addprefix $(objpfx),$(CTYPE_FILES)): %: \
-   gen-locale.sh $(common-objpfx)locale/localedef Makefile \
-   $(addprefix charmaps/,$(CHARMAPS)) $(addprefix locales/,$(LOCALE_SRCS))
--	@$(SHELL) gen-locale.sh $(common-objpfx) \
--		  '$(built-program-cmd-before-env)' '$(run-program-env)' \
--		  '$(built-program-cmd-after-env)' $@; \
-+	@$(SHELL) gen-locale.sh $(common-objpfx)	\
-+		 '$(if $(cross-localedef), 		\
-+		       $(cross-localedef),		\
-+		       $(built-program-cmd-before-env)  \
-+		       $(run-program-env)		\
-+		       $(built-program-cmd-after-env))' \
-+		       $@; \
- 	$(evaluate-test)
- 
- $(addsuffix .out,$(addprefix $(objpfx),$(tests))): %: \
-@@ -213,6 +235,11 @@
+@@ -213,6 +231,11 @@
  
  include SUPPORTED
  
@@ -4783,7 +4766,7 @@ Index: git/posix/fnmatch.c
  #  endif
  
 -#  if (HAVE_MBSTATE_T && HAVE_MBSRTOWCS) || _LIBC
-+#  if (HAVE_MBSTATE_T && HAVE_MBSRTOWCS) || (_LIBC && __OPTION_EGLIBC_LOCALE_CODE)
++#  if (HAVE_MBSTATE_T && HAVE_MBSRTOWCS && _LIBC && __OPTION_EGLIBC_LOCALE_CODE)
  /* In this case we are implementing the multibyte character handling.  */
  #   define HANDLE_MULTIBYTE	1
  #  endif
@@ -5065,8 +5048,8 @@ Index: git/posix/Makefile
  		 $(objpfx)bug-glob2-mem.out $(objpfx)tst-vfork3-mem.out \
  		 $(objpfx)tst-fnmatch-mem.out $(objpfx)bug-regex36-mem.out
 +ifeq (y,$(OPTION_POSIX_REGEXP_GLIBC))
-+tests-special += $(objpfx)bug-regex14-mem $(objpfx)tst-rxspencer-no-utf8-mem \
-+  		 $(objpfx)tst-pcre-mem $(objpfx)tst-boost-mem
++tests-special += $(objpfx)bug-regex14-mem.out $(objpfx)tst-rxspencer-no-utf8-mem.out \
++  		 $(objpfx)tst-pcre-mem.out $(objpfx)tst-boost-mem.out
 +endif
 +
  xtests-special += $(objpfx)bug-ga2-mem.out
@@ -16458,24 +16441,6 @@ Index: git/sysdeps/nptl/small-macros-fns.c
 +libc_hidden_def (__libc_lock_unlock_recursive_fn);
 +
 +#endif /*defined _LIBC && (!defined NOT_IN_libc || defined IS_IN_libpthread)*/
-Index: git/include/libc-symbols.h
-===================================================================
---- git.orig/include/libc-symbols.h	2014-08-29 20:00:47.144070587 -0700
-+++ git/include/libc-symbols.h	2014-08-29 20:01:15.248070587 -0700
-@@ -60,8 +60,11 @@
- /* Define these macros for the benefit of portable GNU code that wants to check
-    them.  Of course, STDC_HEADERS is never false when building libc!  */
- #define STDC_HEADERS	1
--#define HAVE_MBSTATE_T	1
--#define HAVE_MBSRTOWCS	1
-+
-+#if __OPTION_EGLIBC_LOCALE_CODE
-+# define HAVE_MBSTATE_T	1
-+# define HAVE_MBSRTOWCS	1
-+#endif
- #define HAVE_LIBINTL_H	1
- #define HAVE_WCTYPE_H	1
- #define HAVE_ISWCTYPE	1
 Index: git/crypt/crypt_common.c
 ===================================================================
 --- /dev/null	1970-01-01 00:00:00.000000000 +0000
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] eglibc-use-option-groups.patch: Various fixups
  2014-11-23 17:44 [PATCH] eglibc-use-option-groups.patch: Various fixups Juro Bystricky
@ 2015-01-22 20:52 ` Juro Bystricky
  2015-01-22 22:02   ` Peter Seebach
  0 siblings, 1 reply; 4+ messages in thread
From: Juro Bystricky @ 2015-01-22 20:52 UTC (permalink / raw)
  To: openembedded-core

[-- Attachment #1: Type: text/plain, Size: 4302 bytes --]

ping.


> From: jurobystricky@hotmail.com
> To: openembedded-core@lists.openembedded.org
> CC: sgw@linux.intel.com; peter.seebach@windriver.com
> Subject: [PATCH] eglibc-use-option-groups.patch: Various fixups
> Date: Sun, 23 Nov 2014 09:44:17 -0800
> 
> Build suppport for mutibyte character handling only when
> __OPTION_EGLIBC_LOCALE_CODE evaluates to 1.
> Fixes missing .out suffix for several tests to be built.
> Fixes building of locales needed for several tests. Do not use
> cross-localedef to build locales. Use localedef built with the
> newly built libgc instead.
> 
> Fixes:
> [YOCTO #6809] [YOCTO #6796] [YOCTO #6797]
> 
> Signed-off-by: Juro Bystricky <jurobystricky@hotmail.com>
> ---
>  .../glibc/glibc/eglibc-use-option-groups.patch     | 43 ++--------------------
>  1 file changed, 4 insertions(+), 39 deletions(-)
> 
> diff --git a/meta/recipes-core/glibc/glibc/eglibc-use-option-groups.patch b/meta/recipes-core/glibc/glibc/eglibc-use-option-groups.patch
> index 40c7832..0e2f0c6 100644
> --- a/meta/recipes-core/glibc/glibc/eglibc-use-option-groups.patch
> +++ b/meta/recipes-core/glibc/glibc/eglibc-use-option-groups.patch
> @@ -2434,24 +2434,7 @@ Index: git/localedata/Makefile
>   
>   ifeq ($(run-built-tests),yes)
>   # We have to generate locales
> -@@ -143,9 +161,13 @@
> - $(addprefix $(objpfx),$(CTYPE_FILES)): %: \
> -   gen-locale.sh $(common-objpfx)locale/localedef Makefile \
> -   $(addprefix charmaps/,$(CHARMAPS)) $(addprefix locales/,$(LOCALE_SRCS))
> --	@$(SHELL) gen-locale.sh $(common-objpfx) \
> --		  '$(built-program-cmd-before-env)' '$(run-program-env)' \
> --		  '$(built-program-cmd-after-env)' $@; \
> -+	@$(SHELL) gen-locale.sh $(common-objpfx)	\
> -+		 '$(if $(cross-localedef), 		\
> -+		       $(cross-localedef),		\
> -+		       $(built-program-cmd-before-env)  \
> -+		       $(run-program-env)		\
> -+		       $(built-program-cmd-after-env))' \
> -+		       $@; \
> - 	$(evaluate-test)
> - 
> - $(addsuffix .out,$(addprefix $(objpfx),$(tests))): %: \
> -@@ -213,6 +235,11 @@
> +@@ -213,6 +231,11 @@
>   
>   include SUPPORTED
>   
> @@ -4783,7 +4766,7 @@ Index: git/posix/fnmatch.c
>   #  endif
>   
>  -#  if (HAVE_MBSTATE_T && HAVE_MBSRTOWCS) || _LIBC
> -+#  if (HAVE_MBSTATE_T && HAVE_MBSRTOWCS) || (_LIBC && __OPTION_EGLIBC_LOCALE_CODE)
> ++#  if (HAVE_MBSTATE_T && HAVE_MBSRTOWCS && _LIBC && __OPTION_EGLIBC_LOCALE_CODE)
>   /* In this case we are implementing the multibyte character handling.  */
>   #   define HANDLE_MULTIBYTE	1
>   #  endif
> @@ -5065,8 +5048,8 @@ Index: git/posix/Makefile
>   		 $(objpfx)bug-glob2-mem.out $(objpfx)tst-vfork3-mem.out \
>   		 $(objpfx)tst-fnmatch-mem.out $(objpfx)bug-regex36-mem.out
>  +ifeq (y,$(OPTION_POSIX_REGEXP_GLIBC))
> -+tests-special += $(objpfx)bug-regex14-mem $(objpfx)tst-rxspencer-no-utf8-mem \
> -+  		 $(objpfx)tst-pcre-mem $(objpfx)tst-boost-mem
> ++tests-special += $(objpfx)bug-regex14-mem.out $(objpfx)tst-rxspencer-no-utf8-mem.out \
> ++  		 $(objpfx)tst-pcre-mem.out $(objpfx)tst-boost-mem.out
>  +endif
>  +
>   xtests-special += $(objpfx)bug-ga2-mem.out
> @@ -16458,24 +16441,6 @@ Index: git/sysdeps/nptl/small-macros-fns.c
>  +libc_hidden_def (__libc_lock_unlock_recursive_fn);
>  +
>  +#endif /*defined _LIBC && (!defined NOT_IN_libc || defined IS_IN_libpthread)*/
> -Index: git/include/libc-symbols.h
> -===================================================================
> ---- git.orig/include/libc-symbols.h	2014-08-29 20:00:47.144070587 -0700
> -+++ git/include/libc-symbols.h	2014-08-29 20:01:15.248070587 -0700
> -@@ -60,8 +60,11 @@
> - /* Define these macros for the benefit of portable GNU code that wants to check
> -    them.  Of course, STDC_HEADERS is never false when building libc!  */
> - #define STDC_HEADERS	1
> --#define HAVE_MBSTATE_T	1
> --#define HAVE_MBSRTOWCS	1
> -+
> -+#if __OPTION_EGLIBC_LOCALE_CODE
> -+# define HAVE_MBSTATE_T	1
> -+# define HAVE_MBSRTOWCS	1
> -+#endif
> - #define HAVE_LIBINTL_H	1
> - #define HAVE_WCTYPE_H	1
> - #define HAVE_ISWCTYPE	1
>  Index: git/crypt/crypt_common.c
>  ===================================================================
>  --- /dev/null	1970-01-01 00:00:00.000000000 +0000
> -- 
> 1.9.1
> 
 		 	   		  

[-- Attachment #2: Type: text/html, Size: 5093 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] eglibc-use-option-groups.patch: Various fixups
  2015-01-22 20:52 ` Juro Bystricky
@ 2015-01-22 22:02   ` Peter Seebach
  2015-01-23 15:16     ` Burton, Ross
  0 siblings, 1 reply; 4+ messages in thread
From: Peter Seebach @ 2015-01-22 22:02 UTC (permalink / raw)
  To: Juro Bystricky; +Cc: openembedded-core

On Thu, 22 Jan 2015 12:52:37 -0800
Juro Bystricky <jurobystricky@hotmail.com> wrote:

> ping.

My memory (which is notoriously unreliable) was that we'd merged these
changes, but it may be we did thim in our local tree.

> 
> > From: jurobystricky@hotmail.com
> > To: openembedded-core@lists.openembedded.org
> > CC: sgw@linux.intel.com; peter.seebach@windriver.com
> > Subject: [PATCH] eglibc-use-option-groups.patch: Various fixups
> > Date: Sun, 23 Nov 2014 09:44:17 -0800
> > 
> > Build suppport for mutibyte character handling only when
> > __OPTION_EGLIBC_LOCALE_CODE evaluates to 1.
> > Fixes missing .out suffix for several tests to be built.
> > Fixes building of locales needed for several tests. Do not use
> > cross-localedef to build locales. Use localedef built with the
> > newly built libgc instead.
> > 
> > Fixes:
> > [YOCTO #6809] [YOCTO #6796] [YOCTO #6797]
> > 
> > Signed-off-by: Juro Bystricky <jurobystricky@hotmail.com>
> > ---
> >  .../glibc/glibc/eglibc-use-option-groups.patch     | 43 ++--------------------
> >  1 file changed, 4 insertions(+), 39 deletions(-)
> > 
> > diff --git a/meta/recipes-core/glibc/glibc/eglibc-use-option-groups.patch b/meta/recipes-core/glibc/glibc/eglibc-use-option-groups.patch
> > index 40c7832..0e2f0c6 100644
> > --- a/meta/recipes-core/glibc/glibc/eglibc-use-option-groups.patch
> > +++ b/meta/recipes-core/glibc/glibc/eglibc-use-option-groups.patch
> > @@ -2434,24 +2434,7 @@ Index: git/localedata/Makefile
> >   
> >   ifeq ($(run-built-tests),yes)
> >   # We have to generate locales
> > -@@ -143,9 +161,13 @@
> > - $(addprefix $(objpfx),$(CTYPE_FILES)): %: \
> > -   gen-locale.sh $(common-objpfx)locale/localedef Makefile \
> > -   $(addprefix charmaps/,$(CHARMAPS)) $(addprefix locales/,$(LOCALE_SRCS))
> > --	@$(SHELL) gen-locale.sh $(common-objpfx) \
> > --		  '$(built-program-cmd-before-env)' '$(run-program-env)' \
> > --		  '$(built-program-cmd-after-env)' $@; \
> > -+	@$(SHELL) gen-locale.sh $(common-objpfx)	\
> > -+		 '$(if $(cross-localedef), 		\
> > -+		       $(cross-localedef),		\
> > -+		       $(built-program-cmd-before-env)  \
> > -+		       $(run-program-env)		\
> > -+		       $(built-program-cmd-after-env))' \
> > -+		       $@; \
> > - 	$(evaluate-test)
> > - 
> > - $(addsuffix .out,$(addprefix $(objpfx),$(tests))): %: \
> > -@@ -213,6 +235,11 @@
> > +@@ -213,6 +231,11 @@
> >   
> >   include SUPPORTED
> >   
> > @@ -4783,7 +4766,7 @@ Index: git/posix/fnmatch.c
> >   #  endif
> >   
> >  -#  if (HAVE_MBSTATE_T && HAVE_MBSRTOWCS) || _LIBC
> > -+#  if (HAVE_MBSTATE_T && HAVE_MBSRTOWCS) || (_LIBC && __OPTION_EGLIBC_LOCALE_CODE)
> > ++#  if (HAVE_MBSTATE_T && HAVE_MBSRTOWCS && _LIBC && __OPTION_EGLIBC_LOCALE_CODE)
> >   /* In this case we are implementing the multibyte character handling.  */
> >   #   define HANDLE_MULTIBYTE	1
> >   #  endif
> > @@ -5065,8 +5048,8 @@ Index: git/posix/Makefile
> >   		 $(objpfx)bug-glob2-mem.out $(objpfx)tst-vfork3-mem.out \
> >   		 $(objpfx)tst-fnmatch-mem.out $(objpfx)bug-regex36-mem.out
> >  +ifeq (y,$(OPTION_POSIX_REGEXP_GLIBC))
> > -+tests-special += $(objpfx)bug-regex14-mem $(objpfx)tst-rxspencer-no-utf8-mem \
> > -+  		 $(objpfx)tst-pcre-mem $(objpfx)tst-boost-mem
> > ++tests-special += $(objpfx)bug-regex14-mem.out $(objpfx)tst-rxspencer-no-utf8-mem.out \
> > ++  		 $(objpfx)tst-pcre-mem.out $(objpfx)tst-boost-mem.out
> >  +endif
> >  +
> >   xtests-special += $(objpfx)bug-ga2-mem.out
> > @@ -16458,24 +16441,6 @@ Index: git/sysdeps/nptl/small-macros-fns.c
> >  +libc_hidden_def (__libc_lock_unlock_recursive_fn);
> >  +
> >  +#endif /*defined _LIBC && (!defined NOT_IN_libc || defined IS_IN_libpthread)*/
> > -Index: git/include/libc-symbols.h
> > -===================================================================
> > ---- git.orig/include/libc-symbols.h	2014-08-29 20:00:47.144070587 -0700
> > -+++ git/include/libc-symbols.h	2014-08-29 20:01:15.248070587 -0700
> > -@@ -60,8 +60,11 @@
> > - /* Define these macros for the benefit of portable GNU code that wants to check
> > -    them.  Of course, STDC_HEADERS is never false when building libc!  */
> > - #define STDC_HEADERS	1
> > --#define HAVE_MBSTATE_T	1
> > --#define HAVE_MBSRTOWCS	1
> > -+
> > -+#if __OPTION_EGLIBC_LOCALE_CODE
> > -+# define HAVE_MBSTATE_T	1
> > -+# define HAVE_MBSRTOWCS	1
> > -+#endif
> > - #define HAVE_LIBINTL_H	1
> > - #define HAVE_WCTYPE_H	1
> > - #define HAVE_ISWCTYPE	1
> >  Index: git/crypt/crypt_common.c
> >  ===================================================================
> >  --- /dev/null	1970-01-01 00:00:00.000000000 +0000
> > -- 
> > 1.9.1
> > 
>  		 	   		  


-- 
Listen, get this.  Nobody with a good compiler needs to be justified.


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] eglibc-use-option-groups.patch: Various fixups
  2015-01-22 22:02   ` Peter Seebach
@ 2015-01-23 15:16     ` Burton, Ross
  0 siblings, 0 replies; 4+ messages in thread
From: Burton, Ross @ 2015-01-23 15:16 UTC (permalink / raw)
  To: Peter Seebach; +Cc: openembedded-core

[-- Attachment #1: Type: text/plain, Size: 371 bytes --]

On 22 January 2015 at 22:02, Peter Seebach <peter.seebach@windriver.com>
wrote:

> > ping.
>
> My memory (which is notoriously unreliable) was that we'd merged these
> changes, but it may be we did thim in our local tree.
>

Yeah they're not in master - I think I merged and then backed them out
locally to bisect some breakage.  I'm testing here now.

Ross

[-- Attachment #2: Type: text/html, Size: 802 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-01-23 15:16 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-11-23 17:44 [PATCH] eglibc-use-option-groups.patch: Various fixups Juro Bystricky
2015-01-22 20:52 ` Juro Bystricky
2015-01-22 22:02   ` Peter Seebach
2015-01-23 15:16     ` Burton, Ross

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.