All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages
@ 2013-11-12 13:26 Mischa Jonker
  2013-11-12 13:26 ` [Buildroot] [PATCH 1/7] gdb: arc: Use git instead of website Mischa Jonker
                   ` (7 more replies)
  0 siblings, 8 replies; 14+ messages in thread
From: Mischa Jonker @ 2013-11-12 13:26 UTC (permalink / raw)
  To: buildroot

Hi,

This set of patches gets rid of the Synopsys website for downloading
ARC-specific versions of binutils, gdb, gcc, uClibc. Instead, our github
site is used, and the git-protocol where possible (all packages except gcc).

Although our toolchain release is not completely finished, and a couple
of issues are still being worked on, I would like to get some feedback already
on this patch set: after all, it's easy to update the git ID's associated
with the final release when that is ready (hopefully within a week or two).

The new toolchain release fixes a number of auto-builder issues.

Thanks,

Mischa

Mischa Jonker (7):
  gdb: arc: Use git instead of website
  binutils: arc: Use git instead of website
  gcc: arc: Download from github instead of Synopsys website
  uclibc: arc: Use git instead of website
  gcc: remove ARC-specific patches
  uclibc: remove ARC-specific patches
  Revert "arc: Add option for ARC-specific download site"

 arch/Config.in.arc                                 |    4 -
 package/binutils/Config.in.host                    |    2 +-
 package/binutils/binutils.mk                       |    3 +-
 package/gcc/4.8-arc/100-uclibc-conf.patch          |   15 --
 package/gcc/4.8-arc/301-missing-execinfo_h.patch   |   13 --
 package/gcc/4.8-arc/302-c99-snprintf.patch         |   13 --
 .../gcc/4.8-arc/305-libmudflap-susv3-legacy.patch  |   49 -------
 package/gcc/4.8-arc/disable_gc_sections.patch      |  145 --------------------
 package/gcc/4.8-arc/fix_branch_out_of_range.patch  |   30 ----
 package/gcc/Config.in.host                         |    2 +-
 package/gcc/gcc.mk                                 |   11 +-
 package/gdb/Config.in.host                         |    2 +-
 package/gdb/gdb.mk                                 |    9 +-
 package/uclibc/0.9.33-arc/fix-fstat-for-be.patch   |   32 -----
 .../uclibc/0.9.33-arc/fix-susv3-legacy-bcmp.patch  |   37 -----
 package/uclibc/0.9.33-arc/fix-susv3-legacy.patch   |   36 -----
 .../0.9.33-arc/uclibc-arc-add-__kernel_long.patch  |   29 ----
 package/uclibc/Config.in                           |    9 +-
 package/uclibc/uclibc.mk                           |    5 +-
 19 files changed, 23 insertions(+), 423 deletions(-)
 delete mode 100644 package/gcc/4.8-arc/100-uclibc-conf.patch
 delete mode 100644 package/gcc/4.8-arc/301-missing-execinfo_h.patch
 delete mode 100644 package/gcc/4.8-arc/302-c99-snprintf.patch
 delete mode 100644 package/gcc/4.8-arc/305-libmudflap-susv3-legacy.patch
 delete mode 100644 package/gcc/4.8-arc/disable_gc_sections.patch
 delete mode 100644 package/gcc/4.8-arc/fix_branch_out_of_range.patch
 delete mode 100644 package/uclibc/0.9.33-arc/fix-fstat-for-be.patch
 delete mode 100644 package/uclibc/0.9.33-arc/fix-susv3-legacy-bcmp.patch
 delete mode 100644 package/uclibc/0.9.33-arc/fix-susv3-legacy.patch
 delete mode 100644 package/uclibc/0.9.33-arc/uclibc-arc-add-__kernel_long.patch

-- 
1.7.9.5

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

* [Buildroot] [PATCH 1/7] gdb: arc: Use git instead of website
  2013-11-12 13:26 [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages Mischa Jonker
@ 2013-11-12 13:26 ` Mischa Jonker
  2013-11-12 21:37   ` Thomas Petazzoni
  2013-11-12 13:26 ` [Buildroot] [PATCH 2/7] binutils: " Mischa Jonker
                   ` (6 subsequent siblings)
  7 siblings, 1 reply; 14+ messages in thread
From: Mischa Jonker @ 2013-11-12 13:26 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Mischa Jonker <mjonker@synopsys.com>
---
 package/gdb/Config.in.host |    2 +-
 package/gdb/gdb.mk         |    9 +++++----
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/package/gdb/Config.in.host b/package/gdb/Config.in.host
index 128aed9..d4e8ee7 100644
--- a/package/gdb/Config.in.host
+++ b/package/gdb/Config.in.host
@@ -58,6 +58,6 @@ config BR2_GDB_VERSION
 	default "7.3.1"    if BR2_GDB_VERSION_7_3
 	default "7.4.1"    if BR2_GDB_VERSION_7_4
 	default "7.5.1"    if BR2_GDB_VERSION_7_5
-	default "7.5.1-arc" if BR2_arc
+	default "8673f98"  if BR2_arc
 
 endif
diff --git a/package/gdb/gdb.mk b/package/gdb/gdb.mk
index 9cf90e3..317d78c 100644
--- a/package/gdb/gdb.mk
+++ b/package/gdb/gdb.mk
@@ -21,15 +21,16 @@ GDB_VERSION = 7.5.1
 endif
 endif
 
-ifeq ($(BR2_arc),y)
-GDB_SITE = $(BR2_ARC_SITE)
-endif
-
 ifeq ($(GDB_VERSION),6.7.1-avr32-2.1.5)
 GDB_SITE = ftp://www.at91.com/pub/buildroot/
 endif
 
+ifeq ($(BR2_arc),y)
+GDB_SITE = git://github.com/foss-for-synopsys-dwc-arc-processors/gdb.git
+GDB_SOURCE = gdb-$(GDB_VERSION).tar.gz
+else
 GDB_SOURCE = gdb-$(GDB_VERSION).tar.bz2
+endif
 GDB_LICENSE = GPLv2+ LGPLv2+ GPLv3+ LGPLv3+
 GDB_LICENSE_FILES = COPYING COPYING.LIB COPYING3 COPYING3.LIB
 
-- 
1.7.9.5

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

* [Buildroot] [PATCH 2/7] binutils: arc: Use git instead of website
  2013-11-12 13:26 [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages Mischa Jonker
  2013-11-12 13:26 ` [Buildroot] [PATCH 1/7] gdb: arc: Use git instead of website Mischa Jonker
@ 2013-11-12 13:26 ` Mischa Jonker
  2013-11-12 13:26 ` [Buildroot] [PATCH 3/7] gcc: arc: Download from github instead of Synopsys website Mischa Jonker
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 14+ messages in thread
From: Mischa Jonker @ 2013-11-12 13:26 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Mischa Jonker <mjonker@synopsys.com>
---
 package/binutils/Config.in.host |    2 +-
 package/binutils/binutils.mk    |    3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/package/binutils/Config.in.host b/package/binutils/Config.in.host
index 8f19597..77daffc 100644
--- a/package/binutils/Config.in.host
+++ b/package/binutils/Config.in.host
@@ -44,7 +44,7 @@ endchoice
 config BR2_BINUTILS_VERSION
 	string
 	default "2.18-avr32-1.0.1" if BR2_BINUTILS_VERSION_2_18_AVR32_1_0_1
-	default "2.23.2-arc"	if BR2_arc
+	default "0c824ba511"	if BR2_arc
 	default "2.20.1"	if BR2_BINUTILS_VERSION_2_20_1
 	default "2.21"		if BR2_BINUTILS_VERSION_2_21
 	default "2.21.1"	if BR2_BINUTILS_VERSION_2_21_1
diff --git a/package/binutils/binutils.mk b/package/binutils/binutils.mk
index f1723b2..53364cc 100644
--- a/package/binutils/binutils.mk
+++ b/package/binutils/binutils.mk
@@ -27,7 +27,8 @@ ifeq ($(ARCH),avr32)
 BINUTILS_SITE = ftp://www.at91.com/pub/buildroot
 endif
 ifeq ($(BR2_arc),y)
-BINUTILS_SITE = $(BR2_ARC_SITE)
+BINUTILS_SITE = git://github.com/foss-for-synopsys-dwc-arc-processors/binutils.git
+BINUTILS_SOURCE = binutils-$(BINUTILS_VERSION).tar.gz
 endif
 BINUTILS_EXTRA_CONFIG_OPTIONS = $(call qstrip,$(BR2_BINUTILS_EXTRA_CONFIG_OPTIONS))
 BINUTILS_INSTALL_STAGING = YES
-- 
1.7.9.5

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

* [Buildroot] [PATCH 3/7] gcc: arc: Download from github instead of Synopsys website
  2013-11-12 13:26 [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages Mischa Jonker
  2013-11-12 13:26 ` [Buildroot] [PATCH 1/7] gdb: arc: Use git instead of website Mischa Jonker
  2013-11-12 13:26 ` [Buildroot] [PATCH 2/7] binutils: " Mischa Jonker
@ 2013-11-12 13:26 ` Mischa Jonker
  2013-11-12 13:26 ` [Buildroot] [PATCH 4/7] uclibc: arc: Use git instead of website Mischa Jonker
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 14+ messages in thread
From: Mischa Jonker @ 2013-11-12 13:26 UTC (permalink / raw)
  To: buildroot

The github release mechanism only supports .tar.gz, hence the ugly
GCC_UNPACK variable.

Signed-off-by: Mischa Jonker <mjonker@synopsys.com>
---
 package/gcc/Config.in.host |    2 +-
 package/gcc/gcc.mk         |   11 +++++++----
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/package/gcc/Config.in.host b/package/gcc/Config.in.host
index 41c1213..b762411 100644
--- a/package/gcc/Config.in.host
+++ b/package/gcc/Config.in.host
@@ -90,7 +90,7 @@ config BR2_GCC_VERSION
 	default "4.6.4"     if BR2_GCC_VERSION_4_6_X
 	default "4.7.3"     if BR2_GCC_VERSION_4_7_X
 	default "4.8.2"     if BR2_GCC_VERSION_4_8_X
-	default "4.8-arc"   if BR2_GCC_VERSION_4_8_ARC
+	default "arc-4.8-R2-rc1"   if BR2_GCC_VERSION_4_8_ARC
 	default BR2_GCC_SNAP_DATE if BR2_GCC_VERSION_SNAP
 
 config BR2_EXTRA_GCC_CONFIG_OPTIONS
diff --git a/package/gcc/gcc.mk b/package/gcc/gcc.mk
index 41600ef..79fe4ef 100644
--- a/package/gcc/gcc.mk
+++ b/package/gcc/gcc.mk
@@ -19,13 +19,16 @@ ifneq ($(GCC_SNAP_DATE),)
 GCC_SITE = ftp://gcc.gnu.org/pub/gcc/snapshots/$(GCC_SNAP_DATE)/
 else ifeq ($(findstring avr32,$(GCC_VERSION)),avr32)
 GCC_SITE = ftp://www.at91.com/pub/buildroot/
-else ifeq ($(findstring arc,$(GCC_VERSION)),arc)
-GCC_SITE = $(BR2_ARC_SITE)
+else ifeq ($(BR2_arc),y)
+GCC_SITE = http://github.com/foss-for-synopsys-dwc-arc-processors/gcc/archive/
+GCC_SOURCE = $(GCC_VERSION).tar.gz
+GCC_UNPACK = $(ZCAT)
 else
 GCC_SITE = $(BR2_GNU_MIRROR:/=)/gcc/gcc-$(GCC_VERSION)
 endif
 
-GCC_SOURCE = gcc-$(GCC_VERSION).tar.bz2
+GCC_UNPACK ?= $(BZCAT)
+GCC_SOURCE ?= gcc-$(GCC_VERSION).tar.bz2
 
 #
 # Xtensa special hook
@@ -60,7 +63,7 @@ endef
 #
 
 define HOST_GCC_EXTRACT_CMDS
-	$(BZCAT) $(DL_DIR)/$(GCC_SOURCE) | \
+	$(GCC_UNPACK) $(DL_DIR)/$(GCC_SOURCE) | \
 		$(TAR) $(TAR_STRIP_COMPONENTS)=1 -C $(@D) \
 		--exclude='libjava/*' \
 		--exclude='libgo/*' \
-- 
1.7.9.5

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

* [Buildroot] [PATCH 4/7] uclibc: arc: Use git instead of website
  2013-11-12 13:26 [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages Mischa Jonker
                   ` (2 preceding siblings ...)
  2013-11-12 13:26 ` [Buildroot] [PATCH 3/7] gcc: arc: Download from github instead of Synopsys website Mischa Jonker
@ 2013-11-12 13:26 ` Mischa Jonker
  2013-11-12 13:26 ` [Buildroot] [PATCH 5/7] gcc: remove ARC-specific patches Mischa Jonker
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 14+ messages in thread
From: Mischa Jonker @ 2013-11-12 13:26 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Mischa Jonker <mjonker@synopsys.com>
---
 package/uclibc/Config.in |    9 +++------
 package/uclibc/uclibc.mk |    5 +++--
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/package/uclibc/Config.in b/package/uclibc/Config.in
index 82dd3aa..251927f 100644
--- a/package/uclibc/Config.in
+++ b/package/uclibc/Config.in
@@ -20,12 +20,9 @@ choice
 		bool "uClibc 0.9.33.x"
 		depends on !(BR2_arc || BR2_avr32 || BR2_xtensa)
 
-	config BR2_UCLIBC_VERSION_0_9_33_ARC
-		bool "uClibc 0.9.33.x-arc"
-		depends on BR2_arc
-
 	config BR2_UCLIBC_VERSION_SNAPSHOT
 		bool "daily snapshot"
+		depends on !(BR2_arc)
 
 endchoice
 
@@ -41,7 +38,7 @@ config BR2_UCLIBC_VERSION_STRING
 	default 0.9.31.1	if BR2_UCLIBC_VERSION_0_9_31
 	default 0.9.32.1	if BR2_UCLIBC_VERSION_0_9_32
 	default 0.9.33.2	if BR2_UCLIBC_VERSION_0_9_33
-	default 0.9.33-arc	if BR2_UCLIBC_VERSION_0_9_33_ARC
+	default "56eaa0"	if BR2_arc
 	default BR2_USE_UCLIBC_SNAPSHOT	if BR2_UCLIBC_VERSION_SNAPSHOT
 
 config BR2_UCLIBC_CONFIG
@@ -49,7 +46,7 @@ config BR2_UCLIBC_CONFIG
 	default "package/uclibc/uClibc-0.9.31.config" if BR2_UCLIBC_VERSION_0_9_31
 	default "package/uclibc/uClibc-0.9.32.config" if BR2_UCLIBC_VERSION_0_9_32
 	default "package/uclibc/uClibc-0.9.33.config" if BR2_UCLIBC_VERSION_0_9_33
-	default "package/uclibc/uClibc-snapshot.config" if BR2_UCLIBC_VERSION_0_9_33_ARC
+	default "package/uclibc/uClibc-snapshot.config" if BR2_arc
 	default "package/uclibc/uClibc-snapshot.config" if BR2_UCLIBC_VERSION_SNAPSHOT
 	help
 	  Some people may wish to use their own modified uClibc configuration
diff --git a/package/uclibc/uclibc.mk b/package/uclibc/uclibc.mk
index 20d3bb6..29933cc 100644
--- a/package/uclibc/uclibc.mk
+++ b/package/uclibc/uclibc.mk
@@ -9,8 +9,9 @@ UCLIBC_SOURCE = uClibc-$(UCLIBC_VERSION).tar.bz2
 
 ifeq ($(BR2_UCLIBC_VERSION_SNAPSHOT),y)
 UCLIBC_SITE = http://www.uclibc.org/downloads/snapshots
-else ifeq ($(findstring arc,$(UCLIBC_VERSION)),arc)
-UCLIBC_SITE = $(BR2_ARC_SITE)
+else ifeq ($(BR2_arc),y)
+UCLIBC_SITE = git://github.com/foss-for-synopsys-dwc-arc-processors/uClibc.git
+UCLIBC_SOURCE = uClibc-$(UCLIBC_VERSION).tar.gz
 else
 UCLIBC_SITE = http://www.uclibc.org/downloads
 UCLIBC_SOURCE = uClibc-$(UCLIBC_VERSION).tar.xz
-- 
1.7.9.5

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

* [Buildroot] [PATCH 5/7] gcc: remove ARC-specific patches
  2013-11-12 13:26 [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages Mischa Jonker
                   ` (3 preceding siblings ...)
  2013-11-12 13:26 ` [Buildroot] [PATCH 4/7] uclibc: arc: Use git instead of website Mischa Jonker
@ 2013-11-12 13:26 ` Mischa Jonker
  2013-11-12 17:03   ` Arnout Vandecappelle
  2013-11-12 13:26 ` [Buildroot] [PATCH 6/7] uclibc: " Mischa Jonker
                   ` (2 subsequent siblings)
  7 siblings, 1 reply; 14+ messages in thread
From: Mischa Jonker @ 2013-11-12 13:26 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Mischa Jonker <mjonker@synopsys.com>
---
 package/gcc/4.8-arc/100-uclibc-conf.patch          |   15 --
 package/gcc/4.8-arc/301-missing-execinfo_h.patch   |   13 --
 package/gcc/4.8-arc/302-c99-snprintf.patch         |   13 --
 .../gcc/4.8-arc/305-libmudflap-susv3-legacy.patch  |   49 -------
 package/gcc/4.8-arc/disable_gc_sections.patch      |  145 --------------------
 package/gcc/4.8-arc/fix_branch_out_of_range.patch  |   30 ----
 6 files changed, 265 deletions(-)
 delete mode 100644 package/gcc/4.8-arc/100-uclibc-conf.patch
 delete mode 100644 package/gcc/4.8-arc/301-missing-execinfo_h.patch
 delete mode 100644 package/gcc/4.8-arc/302-c99-snprintf.patch
 delete mode 100644 package/gcc/4.8-arc/305-libmudflap-susv3-legacy.patch
 delete mode 100644 package/gcc/4.8-arc/disable_gc_sections.patch
 delete mode 100644 package/gcc/4.8-arc/fix_branch_out_of_range.patch

diff --git a/package/gcc/4.8-arc/100-uclibc-conf.patch b/package/gcc/4.8-arc/100-uclibc-conf.patch
deleted file mode 100644
index d56bf0a..0000000
--- a/package/gcc/4.8-arc/100-uclibc-conf.patch
+++ /dev/null
@@ -1,15 +0,0 @@
-Index: gcc-4.8.0/contrib/regression/objs-gcc.sh
-===================================================================
---- gcc-4.8.0.orig/contrib/regression/objs-gcc.sh	2009-04-09 17:00:19.000000000 +0200
-+++ gcc-4.8.0/contrib/regression/objs-gcc.sh	2013-03-23 17:39:04.000000000 +0100
-@@ -106,6 +106,10 @@
-  then
-   make all-gdb all-dejagnu all-ld || exit 1
-   make install-gdb install-dejagnu install-ld || exit 1
-+elif [ $H_REAL_TARGET = $H_REAL_HOST -a $H_REAL_TARGET = i686-pc-linux-uclibc ]
-+ then
-+  make all-gdb all-dejagnu all-ld || exit 1
-+  make install-gdb install-dejagnu install-ld || exit 1
- elif [ $H_REAL_TARGET = $H_REAL_HOST ] ; then
-   make bootstrap || exit 1
-   make install || exit 1
diff --git a/package/gcc/4.8-arc/301-missing-execinfo_h.patch b/package/gcc/4.8-arc/301-missing-execinfo_h.patch
deleted file mode 100644
index 00efda2..0000000
--- a/package/gcc/4.8-arc/301-missing-execinfo_h.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-Index: gcc-4.8.0/boehm-gc/include/gc.h
-===================================================================
---- gcc-4.8.0.orig/boehm-gc/include/gc.h	2007-04-23 23:10:09.000000000 +0200
-+++ gcc-4.8.0/boehm-gc/include/gc.h	2013-03-23 17:39:20.000000000 +0100
-@@ -503,7 +503,7 @@
- #if defined(__linux__) || defined(__GLIBC__)
- # include <features.h>
- # if (__GLIBC__ == 2 && __GLIBC_MINOR__ >= 1 || __GLIBC__ > 2) \
--     && !defined(__ia64__)
-+     && !defined(__ia64__) && !defined(__UCLIBC__)
- #   ifndef GC_HAVE_BUILTIN_BACKTRACE
- #     define GC_HAVE_BUILTIN_BACKTRACE
- #   endif
diff --git a/package/gcc/4.8-arc/302-c99-snprintf.patch b/package/gcc/4.8-arc/302-c99-snprintf.patch
deleted file mode 100644
index cd4d2cc..0000000
--- a/package/gcc/4.8-arc/302-c99-snprintf.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-Index: gcc-4.8.0/libstdc++-v3/include/c_global/cstdio
-===================================================================
---- gcc-4.8.0.orig/libstdc++-v3/include/c_global/cstdio	2013-02-03 18:54:05.000000000 +0100
-+++ gcc-4.8.0/libstdc++-v3/include/c_global/cstdio	2013-03-23 17:39:32.000000000 +0100
-@@ -138,7 +138,7 @@
-   using ::vsprintf;
- } // namespace
- 
--#if _GLIBCXX_USE_C99
-+#if _GLIBCXX_USE_C99 || defined __UCLIBC__
- 
- #undef snprintf
- #undef vfscanf
diff --git a/package/gcc/4.8-arc/305-libmudflap-susv3-legacy.patch b/package/gcc/4.8-arc/305-libmudflap-susv3-legacy.patch
deleted file mode 100644
index 35d5f50..0000000
--- a/package/gcc/4.8-arc/305-libmudflap-susv3-legacy.patch
+++ /dev/null
@@ -1,49 +0,0 @@
-Index: gcc-4.8.0/libmudflap/mf-hooks2.c
-===================================================================
---- gcc-4.8.0.orig/libmudflap/mf-hooks2.c	2013-02-03 18:48:05.000000000 +0100
-+++ gcc-4.8.0/libmudflap/mf-hooks2.c	2013-03-23 17:39:43.000000000 +0100
-@@ -424,7 +424,7 @@
- {
-   TRACE ("%s\n", __PRETTY_FUNCTION__);
-   MF_VALIDATE_EXTENT(s, n, __MF_CHECK_WRITE, "bzero region");
--  bzero (s, n);
-+  memset (s, 0, n);
- }
- 
- 
-@@ -434,7 +434,7 @@
-   TRACE ("%s\n", __PRETTY_FUNCTION__);
-   MF_VALIDATE_EXTENT(src, n, __MF_CHECK_READ, "bcopy src");
-   MF_VALIDATE_EXTENT(dest, n, __MF_CHECK_WRITE, "bcopy dest");
--  bcopy (src, dest, n);
-+  memmove (dest, src, n);
- }
- 
- 
-@@ -444,7 +444,7 @@
-   TRACE ("%s\n", __PRETTY_FUNCTION__);
-   MF_VALIDATE_EXTENT(s1, n, __MF_CHECK_READ, "bcmp 1st arg");
-   MF_VALIDATE_EXTENT(s2, n, __MF_CHECK_READ, "bcmp 2nd arg");
--  return bcmp (s1, s2, n);
-+  return n == 0 ? 0 : memcmp (s1, s2, n);
- }
- 
- 
-@@ -453,7 +453,7 @@
-   size_t n = strlen (s);
-   TRACE ("%s\n", __PRETTY_FUNCTION__);
-   MF_VALIDATE_EXTENT(s, CLAMPADD(n, 1), __MF_CHECK_READ, "index region");
--  return index (s, c);
-+  return strchr (s, c);
- }
- 
- 
-@@ -462,7 +462,7 @@
-   size_t n = strlen (s);
-   TRACE ("%s\n", __PRETTY_FUNCTION__);
-   MF_VALIDATE_EXTENT(s, CLAMPADD(n, 1), __MF_CHECK_READ, "rindex region");
--  return rindex (s, c);
-+  return strrchr (s, c);
- }
- 
- /* XXX:  stpcpy, memccpy */
diff --git a/package/gcc/4.8-arc/disable_gc_sections.patch b/package/gcc/4.8-arc/disable_gc_sections.patch
deleted file mode 100644
index 46a3a26..0000000
--- a/package/gcc/4.8-arc/disable_gc_sections.patch
+++ /dev/null
@@ -1,145 +0,0 @@
-From be904b178fc0858ad41a3c2fbc13b388bd1ff465 Mon Sep 17 00:00:00 2001
-From: Mischa Jonker <mjonker@synopsys.com>
-Date: Mon, 15 Jul 2013 11:50:20 +0200
-Subject: [PATCH] libstdc++: Disable gc-sections to work around binutils bug
-
-This is to work around the following error message:
-BFD (GNU Binutils) 2.23.2 assertion fail elf32-arc.c:2140
-
-Signed-off-by: Mischa Jonker <mjonker@synopsys.com>
----
- libstdc++-v3/acinclude.m4 |    3 ---
- libstdc++-v3/configure    |   33 ---------------------------------
- 2 files changed, 36 deletions(-)
-
-diff --git a/libstdc++-v3/acinclude.m4 b/libstdc++-v3/acinclude.m4
-index e131214..aa2e375 100644
---- a/libstdc++-v3/acinclude.m4
-+++ b/libstdc++-v3/acinclude.m4
-@@ -296,9 +296,6 @@ AC_DEFUN([GLIBCXX_CHECK_LINKER_FEATURES], [
-       fi
-       rm -f conftest.c conftest.o conftest
-     fi
--    if test "$ac_gcsections" = "yes"; then
--      SECTION_LDFLAGS="-Wl,--gc-sections $SECTION_LDFLAGS"
--    fi
-     AC_MSG_RESULT($ac_gcsections)
- 
-     if test "$ac_test_CFLAGS" = set; then
-diff --git a/libstdc++-v3/configure b/libstdc++-v3/configure
-index ed049cf..d325e3e 100755
---- a/libstdc++-v3/configure
-+++ b/libstdc++-v3/configure
-@@ -20653,9 +20653,6 @@ rm -f core conftest.err conftest.$ac_objext \
-       fi
-       rm -f conftest.c conftest.o conftest
-     fi
--    if test "$ac_gcsections" = "yes"; then
--      SECTION_LDFLAGS="-Wl,--gc-sections $SECTION_LDFLAGS"
--    fi
-     { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_gcsections" >&5
- $as_echo "$ac_gcsections" >&6; }
- 
-@@ -27688,9 +27685,6 @@ rm -f core conftest.err conftest.$ac_objext \
-       fi
-       rm -f conftest.c conftest.o conftest
-     fi
--    if test "$ac_gcsections" = "yes"; then
--      SECTION_LDFLAGS="-Wl,--gc-sections $SECTION_LDFLAGS"
--    fi
-     { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_gcsections" >&5
- $as_echo "$ac_gcsections" >&6; }
- 
-@@ -33619,9 +33613,6 @@ rm -f core conftest.err conftest.$ac_objext \
-       fi
-       rm -f conftest.c conftest.o conftest
-     fi
--    if test "$ac_gcsections" = "yes"; then
--      SECTION_LDFLAGS="-Wl,--gc-sections $SECTION_LDFLAGS"
--    fi
-     { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_gcsections" >&5
- $as_echo "$ac_gcsections" >&6; }
- 
-@@ -45496,9 +45487,6 @@ rm -f core conftest.err conftest.$ac_objext \
-       fi
-       rm -f conftest.c conftest.o conftest
-     fi
--    if test "$ac_gcsections" = "yes"; then
--      SECTION_LDFLAGS="-Wl,--gc-sections $SECTION_LDFLAGS"
--    fi
-     { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_gcsections" >&5
- $as_echo "$ac_gcsections" >&6; }
- 
-@@ -45710,9 +45698,6 @@ rm -f core conftest.err conftest.$ac_objext \
-       fi
-       rm -f conftest.c conftest.o conftest
-     fi
--    if test "$ac_gcsections" = "yes"; then
--      SECTION_LDFLAGS="-Wl,--gc-sections $SECTION_LDFLAGS"
--    fi
-     { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_gcsections" >&5
- $as_echo "$ac_gcsections" >&6; }
- 
-@@ -46185,9 +46170,6 @@ rm -f core conftest.err conftest.$ac_objext \
-       fi
-       rm -f conftest.c conftest.o conftest
-     fi
--    if test "$ac_gcsections" = "yes"; then
--      SECTION_LDFLAGS="-Wl,--gc-sections $SECTION_LDFLAGS"
--    fi
-     { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_gcsections" >&5
- $as_echo "$ac_gcsections" >&6; }
- 
-@@ -52470,9 +52452,6 @@ rm -f core conftest.err conftest.$ac_objext \
-       fi
-       rm -f conftest.c conftest.o conftest
-     fi
--    if test "$ac_gcsections" = "yes"; then
--      SECTION_LDFLAGS="-Wl,--gc-sections $SECTION_LDFLAGS"
--    fi
-     { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_gcsections" >&5
- $as_echo "$ac_gcsections" >&6; }
- 
-@@ -58386,9 +58365,6 @@ rm -f core conftest.err conftest.$ac_objext \
-       fi
-       rm -f conftest.c conftest.o conftest
-     fi
--    if test "$ac_gcsections" = "yes"; then
--      SECTION_LDFLAGS="-Wl,--gc-sections $SECTION_LDFLAGS"
--    fi
-     { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_gcsections" >&5
- $as_echo "$ac_gcsections" >&6; }
- 
-@@ -58553,9 +58529,6 @@ rm -f core conftest.err conftest.$ac_objext \
-       fi
-       rm -f conftest.c conftest.o conftest
-     fi
--    if test "$ac_gcsections" = "yes"; then
--      SECTION_LDFLAGS="-Wl,--gc-sections $SECTION_LDFLAGS"
--    fi
-     { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_gcsections" >&5
- $as_echo "$ac_gcsections" >&6; }
- 
-@@ -58781,9 +58754,6 @@ rm -f core conftest.err conftest.$ac_objext \
-       fi
-       rm -f conftest.c conftest.o conftest
-     fi
--    if test "$ac_gcsections" = "yes"; then
--      SECTION_LDFLAGS="-Wl,--gc-sections $SECTION_LDFLAGS"
--    fi
-     { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_gcsections" >&5
- $as_echo "$ac_gcsections" >&6; }
- 
-@@ -64697,9 +64667,6 @@ rm -f core conftest.err conftest.$ac_objext \
-       fi
-       rm -f conftest.c conftest.o conftest
-     fi
--    if test "$ac_gcsections" = "yes"; then
--      SECTION_LDFLAGS="-Wl,--gc-sections $SECTION_LDFLAGS"
--    fi
-     { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_gcsections" >&5
- $as_echo "$ac_gcsections" >&6; }
- 
--- 
-1.7.9.5
-
diff --git a/package/gcc/4.8-arc/fix_branch_out_of_range.patch b/package/gcc/4.8-arc/fix_branch_out_of_range.patch
deleted file mode 100644
index 5340a3b..0000000
--- a/package/gcc/4.8-arc/fix_branch_out_of_range.patch
+++ /dev/null
@@ -1,30 +0,0 @@
-arc: Fix operand-out-of-range errors
-
-brcc_s instructions can generate operand-out-of-range errors. While a
-better solution is being discussed by the compiler team, this workaround
-ensures that the chances of running into this issue are low.
-
-Signed-off-by: Mischa Jonker <mjonker@synopsys.com>
-
-diff --git a/gcc/config/arc/arc.c b/gcc/config/arc/arc.c
-index ff602c0..b3ca4c4 100644
---- a/gcc/config/arc/arc.c
-+++ b/gcc/config/arc/arc.c
-@@ -6565,7 +6565,7 @@ estimate required size increase).
- 	      rtx *ccp = &XEXP (XVECEXP (pat, 0, 1), 0);
- 
- 	      offset = branch_dest (insn) - INSN_ADDRESSES (INSN_UID (insn));
--	      if ((offset >= -140 && offset < 140)
-+	      if ((offset >= -120 && offset < 120)
- 		  && rtx_equal_p (XEXP (op, 1), const0_rtx)
-  		  && compact_register_operand (XEXP (op, 0), VOIDmode)
- 		  && equality_comparison_operator (op, VOIDmode))
-@@ -6687,7 +6687,7 @@ estimate required size increase).
- 
- 		  if (op0 != cmp0)
-  		    cc_clob_rtx = gen_rtx_REG (CC_ZNmode, CC_REG);
-- 		  else if ((offset >= -140 && offset < 140)
-+		  else if ((offset >= -120 && offset < 120)
- 			   && rtx_equal_p (op1, const0_rtx)
- 			   && compact_register_operand (op0, VOIDmode)
- 			   && (GET_CODE (op) == EQ
-- 
1.7.9.5

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

* [Buildroot] [PATCH 6/7] uclibc: remove ARC-specific patches
  2013-11-12 13:26 [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages Mischa Jonker
                   ` (4 preceding siblings ...)
  2013-11-12 13:26 ` [Buildroot] [PATCH 5/7] gcc: remove ARC-specific patches Mischa Jonker
@ 2013-11-12 13:26 ` Mischa Jonker
  2013-11-12 13:26 ` [Buildroot] [PATCH 7/7] Revert "arc: Add option for ARC-specific download site" Mischa Jonker
  2013-11-12 14:59 ` [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages Spenser Gilliland
  7 siblings, 0 replies; 14+ messages in thread
From: Mischa Jonker @ 2013-11-12 13:26 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Mischa Jonker <mjonker@synopsys.com>
---
 package/uclibc/0.9.33-arc/fix-fstat-for-be.patch   |   32 -----------------
 .../uclibc/0.9.33-arc/fix-susv3-legacy-bcmp.patch  |   37 --------------------
 package/uclibc/0.9.33-arc/fix-susv3-legacy.patch   |   36 -------------------
 .../0.9.33-arc/uclibc-arc-add-__kernel_long.patch  |   29 ---------------
 4 files changed, 134 deletions(-)
 delete mode 100644 package/uclibc/0.9.33-arc/fix-fstat-for-be.patch
 delete mode 100644 package/uclibc/0.9.33-arc/fix-susv3-legacy-bcmp.patch
 delete mode 100644 package/uclibc/0.9.33-arc/fix-susv3-legacy.patch
 delete mode 100644 package/uclibc/0.9.33-arc/uclibc-arc-add-__kernel_long.patch

diff --git a/package/uclibc/0.9.33-arc/fix-fstat-for-be.patch b/package/uclibc/0.9.33-arc/fix-fstat-for-be.patch
deleted file mode 100644
index 2cdb517..0000000
--- a/package/uclibc/0.9.33-arc/fix-fstat-for-be.patch
+++ /dev/null
@@ -1,32 +0,0 @@
-From f394d65c74631a0e715bac582b09c302ee84114f Mon Sep 17 00:00:00 2001
-From: Mischa Jonker <mjonker@synopsys.com>
-Date: Tue, 23 Jul 2013 17:59:04 +0200
-Subject: [PATCH] libc/sysdeps: Fix common-generic stat.h for BE part 2
-
-Previous patch only fixed the build. This patch fixes the behavior at
-run-time as well.
-
-It fixes "Value too large for defined data type" messages caused by
-fstat syscall wrapper returning -EOVERFLOW.
-
-Signed-off-by: Mischa Jonker <mjonker@synopsys.com>
----
- libc/sysdeps/linux/common-generic/bits/stat.h |    2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/libc/sysdeps/linux/common-generic/bits/stat.h b/libc/sysdeps/linux/common-generic/bits/stat.h
-index 07716ab..945c408 100644
---- a/libc/sysdeps/linux/common-generic/bits/stat.h
-+++ b/libc/sysdeps/linux/common-generic/bits/stat.h
-@@ -63,8 +63,8 @@ struct stat
-     unsigned long long __pad4;
-     long __pad5;
-     long st_size;				/* Size of file, in bytes. */
--    int __pad6;
-     int st_blksize;				/* Optimal block size for I/O. */
-+    int __pad6;
-     long __pad7;
-     long st_blocks;				/* Number 512-byte blocks allocated */
- # endif /* __LITTLE_ENDIAN */
---
-1.7.9.5
diff --git a/package/uclibc/0.9.33-arc/fix-susv3-legacy-bcmp.patch b/package/uclibc/0.9.33-arc/fix-susv3-legacy-bcmp.patch
deleted file mode 100644
index e0884ea..0000000
--- a/package/uclibc/0.9.33-arc/fix-susv3-legacy-bcmp.patch
+++ /dev/null
@@ -1,37 +0,0 @@
-From 0594ba53b9b8d9a1ac409fd187e4d1ba8f2e7f2a Mon Sep 17 00:00:00 2001
-From: Vineet Gupta <vgupta@synopsys.com>
-Date: Thu, 8 Aug 2013 15:41:51 +0530
-Subject: [PATCH] ARC: SuSv3 legacy support: Add "bcmp" alias for memcmp
-
-Although uClibc provides this already as UCLIBC_SUSV3_LEGACY_MACROS,
-that however requires UCLIBC_SUSV3_LEGACY to be switched off, causing a
-bunch of other things to be not available (usleep)
-
-Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
----
- libc/string/arc/memcmp.S | 5 +++++
- 1 file changed, 5 insertions(+)
-
-diff --git a/libc/string/arc/memcmp.S b/libc/string/arc/memcmp.S
-index 0c7c345..cb25990 100644
---- a/libc/string/arc/memcmp.S
-+++ b/libc/string/arc/memcmp.S
-@@ -1,6 +1,7 @@
- /* Copyright (C) 2007 ARC International (UK) LTD */
- 
- #include <bits/asm.h>
-+#include <features.h>
- 
- #ifdef __LITTLE_ENDIAN__
- #define WORD2 r2
-@@ -115,3 +116,7 @@ ENTRY(memcmp)
- 	j_s.d	[blink]
- 	mov	r0,0
- ENDFUNC(memcmp)
-+
-+#ifdef __UCLIBC_SUSV3_LEGACY__
-+strong_alias(memcmp,bcmp)
-+#endif
--- 
-1.8.1.2
-
diff --git a/package/uclibc/0.9.33-arc/fix-susv3-legacy.patch b/package/uclibc/0.9.33-arc/fix-susv3-legacy.patch
deleted file mode 100644
index a768024..0000000
--- a/package/uclibc/0.9.33-arc/fix-susv3-legacy.patch
+++ /dev/null
@@ -1,36 +0,0 @@
-From 3124ace804337d32974392e0a73a09cdc0d60fc8 Mon Sep 17 00:00:00 2001
-From: Mischa Jonker <mjonker@synopsys.com>
-Date: Mon, 15 Jul 2013 13:06:11 +0200
-Subject: [PATCH] ARC: Add alias 'index' to strcmp for susv3 legacy
- compatibility
-
-This fixes build errors for 'keyutils'
-
-Signed-off-by: Mischa Jonker <mjonker@synopsys.com>
----
- libc/string/arc/strchr.S |    5 +++++
- 1 file changed, 5 insertions(+)
-
-diff --git a/libc/string/arc/strchr.S b/libc/string/arc/strchr.S
-index b9f28c0..8d51aa4 100644
---- a/libc/string/arc/strchr.S
-+++ b/libc/string/arc/strchr.S
-@@ -1,6 +1,7 @@
- /* Copyright (C) 2007 ARC International (UK) LTD */
- 
- #include <bits/asm.h>
-+#include <features.h>
- 
- /* ARC700 has a relatively long pipeline and branch prediction, so we want
-    to avoid branches that are hard to predict.  On the other hand, the
-@@ -115,3 +116,7 @@ ENTRY(strchr)
- 	mov.mi	r0,0
- #endif /* ENDIAN */
- ENDFUNC(strchr)
-+
-+#ifdef __UCLIBC_SUSV3_LEGACY__
-+strong_alias(strchr,index)
-+#endif
--- 
-1.7.9.5
-
diff --git a/package/uclibc/0.9.33-arc/uclibc-arc-add-__kernel_long.patch b/package/uclibc/0.9.33-arc/uclibc-arc-add-__kernel_long.patch
deleted file mode 100644
index caebac1..0000000
--- a/package/uclibc/0.9.33-arc/uclibc-arc-add-__kernel_long.patch
+++ /dev/null
@@ -1,29 +0,0 @@
-From 5164fb28e9205626211d0436933e6eb4960be582 Mon Sep 17 00:00:00 2001
-From: Mischa Jonker <mjonker@synopsys.com>
-Date: Wed, 10 Jul 2013 09:05:25 +0200
-Subject: [PATCH] ARC: libc/sysdeps: add __kernel_long and __kernel_ulong
-
-Linux 3.4 added these two types and updated various exported header
-files to use them. While other architectures were updated before,
-this was not the case for ARC.
-
-This fixes the build failure with Linux 3.10 headers.
-
-Signed-off-by: Mischa Jonker <mjonker@synopsys.com>
----
- libc/sysdeps/linux/arc/bits/kernel_types.h |    2 ++
- 1 file changed, 2 insertions(+)
-
-diff --git a/libc/sysdeps/linux/arc/bits/kernel_types.h b/libc/sysdeps/linux/arc/bits/kernel_types.h
-index fd52e1c..fc28fa1 100755
---- a/libc/sysdeps/linux/arc/bits/kernel_types.h
-+++ b/libc/sysdeps/linux/arc/bits/kernel_types.h
-@@ -39,6 +39,8 @@ typedef	__kernel_uid_t		__kernel_old_uid_t;
- typedef __kernel_gid_t		__kernel_old_gid_t;
- typedef long long		__kernel_loff_t;
- typedef unsigned int		__kernel_old_dev_t;
-+typedef long			__kernel_long_t;
-+typedef unsigned long		__kernel_ulong_t;
-
- typedef struct {
- #ifdef __USE_ALL
-- 
1.7.9.5

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

* [Buildroot] [PATCH 7/7] Revert "arc: Add option for ARC-specific download site"
  2013-11-12 13:26 [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages Mischa Jonker
                   ` (5 preceding siblings ...)
  2013-11-12 13:26 ` [Buildroot] [PATCH 6/7] uclibc: " Mischa Jonker
@ 2013-11-12 13:26 ` Mischa Jonker
  2013-11-12 14:59 ` [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages Spenser Gilliland
  7 siblings, 0 replies; 14+ messages in thread
From: Mischa Jonker @ 2013-11-12 13:26 UTC (permalink / raw)
  To: buildroot

All packages use github now.

This reverts commit 1445b7fd2e46db5d39ad5e2f5cc262574399f5f3.

Signed-off-by: Mischa Jonker <mjonker@synopsys.com>
---
 arch/Config.in.arc |    4 ----
 1 file changed, 4 deletions(-)

diff --git a/arch/Config.in.arc b/arch/Config.in.arc
index 9f68b46..60b59f0 100644
--- a/arch/Config.in.arc
+++ b/arch/Config.in.arc
@@ -12,7 +12,3 @@ config BR2_ENDIAN
 
 config BR2_GCC_TARGET_CPU
 	default "arc700"
-
-config BR2_ARC_SITE
-	string
-	default "http://www.synopsys.com/apps/arc-developer/"
-- 
1.7.9.5

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

* [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages
  2013-11-12 13:26 [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages Mischa Jonker
                   ` (6 preceding siblings ...)
  2013-11-12 13:26 ` [Buildroot] [PATCH 7/7] Revert "arc: Add option for ARC-specific download site" Mischa Jonker
@ 2013-11-12 14:59 ` Spenser Gilliland
  2013-11-12 15:07   ` Thomas De Schampheleire
  7 siblings, 1 reply; 14+ messages in thread
From: Spenser Gilliland @ 2013-11-12 14:59 UTC (permalink / raw)
  To: buildroot

On Nov 12, 2013 7:26 AM, "Mischa Jonker" <Mischa.Jonker@synopsys.com> wrote:
>
> Hi,
>
> This set of patches gets rid of the Synopsys website for downloading
> ARC-specific versions of binutils, gdb, gcc, uClibc. Instead, our github
> site is used, and the git-protocol where possible (all packages except
gcc).
>
> Although our toolchain release is not completely finished, and a couple
> of issues are still being worked on, I would like to get some feedback
already
> on this patch set: after all, it's easy to update the git ID's associated
> with the final release when that is ready (hopefully within a week or
two).
>
> The new toolchain release fixes a number of auto-builder issues.
>
> Thanks,
>
> Mischa
>
> Mischa Jonker (7):
>   gdb: arc: Use git instead of website
>   binutils: arc: Use git instead of website
>   gcc: arc: Download from github instead of Synopsys website
>   uclibc: arc: Use git instead of website
>   gcc: remove ARC-specific patches
>   uclibc: remove ARC-specific patches
>   Revert "arc: Add option for ARC-specific download site"
>
>  arch/Config.in.arc                                 |    4 -
>  package/binutils/Config.in.host                    |    2 +-
>  package/binutils/binutils.mk                       |    3 +-
>  package/gcc/4.8-arc/100-uclibc-conf.patch          |   15 --
>  package/gcc/4.8-arc/301-missing-execinfo_h.patch   |   13 --
>  package/gcc/4.8-arc/302-c99-snprintf.patch         |   13 --
>  .../gcc/4.8-arc/305-libmudflap-susv3-legacy.patch  |   49 -------
>  package/gcc/4.8-arc/disable_gc_sections.patch      |  145
--------------------
>  package/gcc/4.8-arc/fix_branch_out_of_range.patch  |   30 ----
>  package/gcc/Config.in.host                         |    2 +-
>  package/gcc/gcc.mk                                 |   11 +-
>  package/gdb/Config.in.host                         |    2 +-
>  package/gdb/gdb.mk                                 |    9 +-
>  package/uclibc/0.9.33-arc/fix-fstat-for-be.patch   |   32 -----
>  .../uclibc/0.9.33-arc/fix-susv3-legacy-bcmp.patch  |   37 -----
>  package/uclibc/0.9.33-arc/fix-susv3-legacy.patch   |   36 -----
>  .../0.9.33-arc/uclibc-arc-add-__kernel_long.patch  |   29 ----
>  package/uclibc/Config.in                           |    9 +-
>  package/uclibc/uclibc.mk                           |    5 +-
>  19 files changed, 23 insertions(+), 423 deletions(-)
>  delete mode 100644 package/gcc/4.8-arc/100-uclibc-conf.patch
>  delete mode 100644 package/gcc/4.8-arc/301-missing-execinfo_h.patch
>  delete mode 100644 package/gcc/4.8-arc/302-c99-snprintf.patch
>  delete mode 100644 package/gcc/4.8-arc/305-libmudflap-susv3-legacy.patch
>  delete mode 100644 package/gcc/4.8-arc/disable_gc_sections.patch
>  delete mode 100644 package/gcc/4.8-arc/fix_branch_out_of_range.patch
>  delete mode 100644 package/uclibc/0.9.33-arc/fix-fstat-for-be.patch
>  delete mode 100644 package/uclibc/0.9.33-arc/fix-susv3-legacy-bcmp.patch
>  delete mode 100644 package/uclibc/0.9.33-arc/fix-susv3-legacy.patch
>  delete mode 100644
package/uclibc/0.9.33-arc/uclibc-arc-add-__kernel_long.patch
>
> --
> 1.7.9.5
>
>

Just a quick comment.  When downloading from github do not use the git://
protocol use http:// as shown in section 6.2.8 of the buildroot manual.

Thanks,
Spenser _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20131112/5b3a82b2/attachment-0001.html>

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

* [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages
  2013-11-12 14:59 ` [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages Spenser Gilliland
@ 2013-11-12 15:07   ` Thomas De Schampheleire
  0 siblings, 0 replies; 14+ messages in thread
From: Thomas De Schampheleire @ 2013-11-12 15:07 UTC (permalink / raw)
  To: buildroot

On Tue, Nov 12, 2013 at 3:59 PM, Spenser Gilliland
<spenser@gillilanding.com> wrote:
>
> Just a quick comment.  When downloading from github do not use the git://
> protocol use http:// as shown in section 6.2.8 of the buildroot manual.
>

True.
We recently also talked about creating a utility function that wraps
all github URLs, and use https iso http.
See http://lists.busybox.net/pipermail/buildroot/2013-November/081714.html
and http://lists.busybox.net/pipermail/buildroot/2013-November/081823.html

If anyone is interested in implementing this, please do.

Best regards,
Thomas

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

* [Buildroot] [PATCH 5/7] gcc: remove ARC-specific patches
  2013-11-12 13:26 ` [Buildroot] [PATCH 5/7] gcc: remove ARC-specific patches Mischa Jonker
@ 2013-11-12 17:03   ` Arnout Vandecappelle
  0 siblings, 0 replies; 14+ messages in thread
From: Arnout Vandecappelle @ 2013-11-12 17:03 UTC (permalink / raw)
  To: buildroot

On 12/11/13 14:26, Mischa Jonker wrote:
> Signed-off-by: Mischa Jonker<mjonker@synopsys.com>
> ---
>   package/gcc/4.8-arc/100-uclibc-conf.patch          |   15 --
>   package/gcc/4.8-arc/301-missing-execinfo_h.patch   |   13 --
>   package/gcc/4.8-arc/302-c99-snprintf.patch         |   13 --
>   .../gcc/4.8-arc/305-libmudflap-susv3-legacy.patch  |   49 -------
>   package/gcc/4.8-arc/disable_gc_sections.patch      |  145 --------------------
>   package/gcc/4.8-arc/fix_branch_out_of_range.patch  |   30 ----
>   6 files changed, 265 deletions(-)
>   delete mode 100644 package/gcc/4.8-arc/100-uclibc-conf.patch
>   delete mode 100644 package/gcc/4.8-arc/301-missing-execinfo_h.patch
>   delete mode 100644 package/gcc/4.8-arc/302-c99-snprintf.patch
>   delete mode 100644 package/gcc/4.8-arc/305-libmudflap-susv3-legacy.patch
>   delete mode 100644 package/gcc/4.8-arc/disable_gc_sections.patch
>   delete mode 100644 package/gcc/4.8-arc/fix_branch_out_of_range.patch

  This one should probably be squashed with 3/7, because that is the 
reason that these patches are no longer used.

  Regards,
  Arnout

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

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

* [Buildroot] [PATCH 1/7] gdb: arc: Use git instead of website
  2013-11-12 13:26 ` [Buildroot] [PATCH 1/7] gdb: arc: Use git instead of website Mischa Jonker
@ 2013-11-12 21:37   ` Thomas Petazzoni
  2013-11-12 21:40     ` Mischa Jonker
  0 siblings, 1 reply; 14+ messages in thread
From: Thomas Petazzoni @ 2013-11-12 21:37 UTC (permalink / raw)
  To: buildroot

Dear Mischa Jonker,

On Tue, 12 Nov 2013 14:26:14 +0100, Mischa Jonker wrote:
> Signed-off-by: Mischa Jonker <mjonker@synopsys.com>
> ---
>  package/gdb/Config.in.host |    2 +-
>  package/gdb/gdb.mk         |    9 +++++----
>  2 files changed, 6 insertions(+), 5 deletions(-)
> 
> diff --git a/package/gdb/Config.in.host b/package/gdb/Config.in.host
> index 128aed9..d4e8ee7 100644
> --- a/package/gdb/Config.in.host
> +++ b/package/gdb/Config.in.host
> @@ -58,6 +58,6 @@ config BR2_GDB_VERSION
>  	default "7.3.1"    if BR2_GDB_VERSION_7_3
>  	default "7.4.1"    if BR2_GDB_VERSION_7_4
>  	default "7.5.1"    if BR2_GDB_VERSION_7_5
> -	default "7.5.1-arc" if BR2_arc
> +	default "8673f98"  if BR2_arc

Is there any reason why for gdb and binutils we don't have a "choice"
option for the ARC version? I know it's a bit silly because for ARC
that would be the only choice, but that would make things more similar
to what we do for gcc, no?

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

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

* [Buildroot] [PATCH 1/7] gdb: arc: Use git instead of website
  2013-11-12 21:37   ` Thomas Petazzoni
@ 2013-11-12 21:40     ` Mischa Jonker
  2013-11-12 21:52       ` Thomas Petazzoni
  0 siblings, 1 reply; 14+ messages in thread
From: Mischa Jonker @ 2013-11-12 21:40 UTC (permalink / raw)
  To: buildroot

Hi Thomas,

> > -	default "7.5.1-arc" if BR2_arc
> > +	default "8673f98"  if BR2_arc
> 
> Is there any reason why for gdb and binutils we don't have a "choice"
> option for the ARC version? I know it's a bit silly because for ARC that
> would be the only choice, but that would make things more similar to what we
> do for gcc, no?

I think that was decided as part of the review process back then. To be honest, I forgot to remove the choice in gcc now, so I didn't use this opportunity to make it consistent. Now that I'll make a v2 anyway, please let me know what your preference is: to choose or not to choose.

Thanks,

Mischa

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

* [Buildroot] [PATCH 1/7] gdb: arc: Use git instead of website
  2013-11-12 21:40     ` Mischa Jonker
@ 2013-11-12 21:52       ` Thomas Petazzoni
  0 siblings, 0 replies; 14+ messages in thread
From: Thomas Petazzoni @ 2013-11-12 21:52 UTC (permalink / raw)
  To: buildroot

Dear Mischa Jonker,

On Tue, 12 Nov 2013 21:40:12 +0000, Mischa Jonker wrote:

> > Is there any reason why for gdb and binutils we don't have a "choice"
> > option for the ARC version? I know it's a bit silly because for ARC that
> > would be the only choice, but that would make things more similar to what we
> > do for gcc, no?
> 
> I think that was decided as part of the review process back then. To be honest, I forgot to remove the choice in gcc now, so I didn't use this opportunity to make it consistent. Now that I'll make a v2 anyway, please let me know what your preference is: to choose or not to choose.

The problem is that the gcc choice is helpful to be able to do the
"select BR2_GCC_NEEDS_MPC". If you remove the gcc version choice, you
would have to add a "select BR2_GCC_NEEDS_MPC if BR2_arc" under the
main gcc option. Not a big deal, but I'm wondering if I don't prefer
having a single choice option.

Let's hear what others have to say, I don't have a really strong
opinion on the matter, except that I'd like to have consistency between
gcc/binutils/uClibc on the matter, and also across architectures: ARC,
AVR32 and Microblaze (all needing special versions) should use the same
strategy. Of course, I'm not asking you to fix all of them, just want
to figure out a strategy that we will apply to all of these cases.

Best regards,

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

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

end of thread, other threads:[~2013-11-12 21:52 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-11-12 13:26 [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages Mischa Jonker
2013-11-12 13:26 ` [Buildroot] [PATCH 1/7] gdb: arc: Use git instead of website Mischa Jonker
2013-11-12 21:37   ` Thomas Petazzoni
2013-11-12 21:40     ` Mischa Jonker
2013-11-12 21:52       ` Thomas Petazzoni
2013-11-12 13:26 ` [Buildroot] [PATCH 2/7] binutils: " Mischa Jonker
2013-11-12 13:26 ` [Buildroot] [PATCH 3/7] gcc: arc: Download from github instead of Synopsys website Mischa Jonker
2013-11-12 13:26 ` [Buildroot] [PATCH 4/7] uclibc: arc: Use git instead of website Mischa Jonker
2013-11-12 13:26 ` [Buildroot] [PATCH 5/7] gcc: remove ARC-specific patches Mischa Jonker
2013-11-12 17:03   ` Arnout Vandecappelle
2013-11-12 13:26 ` [Buildroot] [PATCH 6/7] uclibc: " Mischa Jonker
2013-11-12 13:26 ` [Buildroot] [PATCH 7/7] Revert "arc: Add option for ARC-specific download site" Mischa Jonker
2013-11-12 14:59 ` [Buildroot] [PATCH 0/7] RFC: Move to github as source for ARC-specific packages Spenser Gilliland
2013-11-12 15:07   ` Thomas De Schampheleire

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.