Re: [PATCH] Use a hash file for GCC only

2017-03-08 Thread Chris Johns 

On 08/03/2017 17:10, Sebastian Huber wrote:



On 07/03/17 23:24, Chris Johns wrote:

On 03/03/2017 02:23, Gedare Bloom wrote:

In a way, I do like that this shows the different gcc-newlib versions
we use. It might be nice to add a bit of organization to this hashes
file and/or documentation of the procedure for adding/removing
entries.

Maybe it makes more sense to have a separate file for each version
being used, e.g. a gcc-4.8.3 hash file, gcc-4.9.3, gcc-6.0.1, etc?
That should make it easier to maintain the hashes.

I'll leave it to Chris to decide if this is a suitable compromise.



This seems to have stalled and I have had to push the change to fix
builds on FreeBSD 11.0 so reverting the change is not possible.

Please apply this patch. We can look at further improvement as future
refactoring work.


After the comments in this thread I sent an alternative patch that
removes the hash file:


I am sorry, I must have missed it.



https://lists.rtems.org/pipermail/devel/2017-March/017102.html

Which one do you like?



This is fine.

Chris
___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Re: [PATCH] bsps: Provide for every BSP

2017-03-08 Thread Chris Johns 

On 08/03/2017 17:12, Sebastian Huber wrote:



On 07/03/17 22:05, Chris Johns wrote:

On 08/03/2017 01:35, Sebastian Huber wrote:

As a background information, I port currently the FDT(4) support from
FreeBSD:

https://www.freebsd.org/cgi/man.cgi?query=fdt&apropos=0&sektion=0&manpath=FreeBSD+11.0-RELEASE+and+Ports&arch=default&format=html


This allows the use of unmodified FreeBSD drivers on PowerPC and ARM
platforms in case the FDT is used.


Nice.

What are the differences?


Difference between ARM and PowerPC? None that I am aware of.



I was meaning this code and the upstream project.



Are standard blobs built with any FDT compiler usage?


I think the blob FDT itself is platform independent.



Great.

Patch looks fine.

Chris
___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

[PATCH] Fix rtems-test-check with a BSD sed.

2017-03-08 Thread Chris Johns 
BSD sed does not support '\t' and treated '[ \t]' as 3 characters. This patch
uses a standard method of supporting blank spaces.
---
 testsuites/automake/test-subdirs.am |  2 +-
 tools/build/rtems-test-check| 10 +-
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/testsuites/automake/test-subdirs.am 
b/testsuites/automake/test-subdirs.am
index 380aae1619..f83f03541f 100644
--- a/testsuites/automake/test-subdirs.am
+++ b/testsuites/automake/test-subdirs.am
@@ -16,7 +16,7 @@ all-local:
fi; \

tdata="$(top_srcdir)/../../c/src/lib/libbsp/$(RTEMS_CPU)/$(RTEMS_BSP_FAMILY)/make/custom/$(RTEMS_BSP)-testsuite.tcfg";
 \
if test -f "$$tdata"; then \
- 
vtdata="$(RTEMS_CPU)/$(RTEMS_BSP_FAMILY)/make/custom/$(RTEMS_BSP)-testsuite.cfg";
 \
+ 
vtdata="$(RTEMS_CPU)/$(RTEMS_BSP_FAMILY)/make/custom/$(RTEMS_BSP)-testsuite.tcfg";
 \
 else \
  vtdata="all tests"; \
fi; \
diff --git a/tools/build/rtems-test-check b/tools/build/rtems-test-check
index 108b7881c2..923af49306 100755
--- a/tools/build/rtems-test-check
+++ b/tools/build/rtems-test-check
@@ -1,6 +1,6 @@
 #! /bin/sh
 #
-# Copyright 2014, 2016 Chris Johns 
+# Copyright 2014, 2016, 2017 Chris Johns 
 # All rights reserved
 #
 
@@ -74,21 +74,21 @@ if test -f $testdata; then
   state=$(echo $line | sed -e "s/:.*//g")
   case $state in
 include)
-  inf=$(echo $line | sed -e "s/include://g" -e 's/^[ \t]//;s/[ 
\t]$//')
+  inf=$(echo $line | sed -e 's/include://g;s/[[:blank:]]//g')
   if test -f $includepath/$inf; then
 ntd="$includepath/$inf $ntd"
   fi
   ;;
 exclude)
-  line=$(echo $line | sed -e "s/exclude://g" -e 's/^[ \t]//;s/[ 
\t]$//')
+  line=$(echo $line | sed -e 's/exclude://g;s/[[:blank:]]//g')
   excluded_tests="${excluded_tests} $line"
   ;;
 expected-fail)
-  line=$(echo $line | sed -e "s/expected-fail://g" -e 's/^[ 
\t]//;s/[ \t]$//')
+  line=$(echo $line | sed -e 
's/expected-fail://g;s/[[:blank:]]//g')
   expected_fails="${expected_fails} $line"
   ;;
 indeterminate)
-  line=$(echo $line | sed -e "s/indeterminate://g" -e 's/^[ 
\t]//;s/[ \t]$//')
+  line=$(echo $line | sed -e 
's/indeterminate://g;s/[[:blank:]]//g')
   indeterminates="${indeterminates} $line"
   ;;
 *)
-- 
2.11.0

___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

[PATCH] Fix rtems-test-check with a BSD sed.

2017-03-08 Thread Chris Johns 
I have tested this patch in FreeBSD 11.0 and it builds a range of
memory limited BSPs. Without the patch the last t of cdtest was
being removed.

Could someone on Linux please test the patch and report back so I can
push it to master and 4.11.

Thanks
Chris
___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel