Re: [PATCH v3] libgloss: arm: break newlib dependency

Corinna Vinschen <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <Y759/1F0TCsV3G/[email protected]>
On Jan 11 01:01, Mike Frysinger wrote:
> The libgloss port has been reaching back into newlib internals for a
> single header whose contents have been frozen for almost a decade.
> To break this backwards libgloss->newlib dependency, move the acle
> header to the srcroot include/ so everyone can use the same copy.
> ---
> v3
> - use -idirafter with srcroot
> 
>  .../arm-acle-compat.h                         |   0
>  libgloss/Makefile.am                          |   3 +
>  libgloss/Makefile.in                          |   2 +
>  libgloss/arm/Makefile.in                      |   2 -
>  libgloss/arm/arm.h                            |   2 +-
>  libgloss/config/default.mh                    |   2 +-
>  newlib/Makefile.am                            |   3 +-
>  newlib/Makefile.in                            |   4 +-
>  newlib/libc/machine/arm/aeabi_memcpy-armv7a.S |   2 +-
>  newlib/libc/machine/arm/aeabi_memmove-thumb.S |   2 +-
>  newlib/libc/machine/arm/aeabi_memset-thumb.S  |   2 +-
>  newlib/libc/machine/arm/arm_asm.h             |   2 +-
>  newlib/libc/machine/arm/machine/acle-compat.h | 182 ------------------
>  newlib/libc/machine/arm/memchr-stub.c         |   2 +-
>  newlib/libc/machine/arm/memchr.S              |   2 +-
>  newlib/libc/machine/arm/memcpy-stub.c         |   2 +-
>  newlib/libc/machine/arm/memcpy.S              |   2 +-
>  newlib/libc/machine/arm/setjmp.S              |   2 +-
>  newlib/libc/machine/arm/strcmp.S              |   2 +-
>  newlib/libc/machine/arm/strlen-armv7.S        |   2 +-
>  newlib/libc/machine/arm/strlen-thumb2-Os.S    |   2 +-
>  newlib/libc/machine/arm/strlen.S              |   2 +-
>  newlib/libc/sys/arm/arm.h                     |   2 +-
>  23 files changed, 25 insertions(+), 203 deletions(-)
>  rename newlib/libc/machine/arm/acle-compat.h => include/arm-acle-compat.h (100%)
>  delete mode 100644 newlib/libc/machine/arm/machine/acle-compat.h

LGTM.


Thanks,
Corinna
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.