Re: [PATCH] configure: Add RTEMS_TOOL_CHAIN_ERROR

2017-01-17 Thread Sebastian Huber
On 13/01/17 15:54, Joel Sherrill wrote: On Fri, Jan 13, 2017 at 4:47 AM, Sebastian Huber > wrote: On 13/01/17 09:48, Sebastian Huber wrote: On 12/01/17 22:44, Joel Sherrill wrote: I repeat that I would like to purge old tes

Re: [PATCH] configure: Add RTEMS_TOOL_CHAIN_ERROR

2017-01-13 Thread Joel Sherrill
On Fri, Jan 13, 2017 at 4:47 AM, Sebastian Huber wrote: > On 13/01/17 09:48, Sebastian Huber wrote: > >> On 12/01/17 22:44, Joel Sherrill wrote: >> >>> I repeat that I would like to purge old tests that result in >>> conditionals in >>> source code we no longer need. So any HAVE_XXX that are a si

Re: [PATCH] configure: Add RTEMS_TOOL_CHAIN_ERROR

2017-01-13 Thread Sebastian Huber
On 13/01/17 09:48, Sebastian Huber wrote: On 12/01/17 22:44, Joel Sherrill wrote: I repeat that I would like to purge old tests that result in conditionals in source code we no longer need. So any HAVE_XXX that are a side effect of transitioning newlib additions should be removed. I removed t

Re: [PATCH] configure: Add RTEMS_TOOL_CHAIN_ERROR

2017-01-13 Thread Sebastian Huber
On 12/01/17 22:44, Joel Sherrill wrote: I repeat that I would like to purge old tests that result in conditionals in source code we no longer need. So any HAVE_XXX that are a side effect of transitioning newlib additions should be removed. I removed three tests today which I added in the last

Re: [PATCH] configure: Add RTEMS_TOOL_CHAIN_ERROR

2017-01-12 Thread Sebastian Huber
On 13/01/17 04:52, Chris Johns wrote: On 13/01/2017 08:44, Joel Sherrill wrote: On Thu, Jan 12, 2017 at 3:34 PM, Chris Johns mailto:chr...@rtems.org>> wrote: On 13/01/2017 00:37, Joel Sherrill wrote: I am OK with this and think it will give a more helpful message. I am OK

Re: [PATCH] configure: Add RTEMS_TOOL_CHAIN_ERROR

2017-01-12 Thread Chris Johns
On 13/01/2017 08:44, Joel Sherrill wrote: On Thu, Jan 12, 2017 at 3:34 PM, Chris Johns mailto:chr...@rtems.org>> wrote: On 13/01/2017 00:37, Joel Sherrill wrote: I am OK with this and think it will give a more helpful message. I am OK with the change however I would like the

Re: [PATCH] configure: Add RTEMS_TOOL_CHAIN_ERROR

2017-01-12 Thread Joel Sherrill
On Thu, Jan 12, 2017 at 3:34 PM, Chris Johns wrote: > On 13/01/2017 00:37, Joel Sherrill wrote: > >> I am OK with this and think it will give a more helpful message. >> > > I am OK with the change however I would like the patch to wait until > docs.rtems.org is up and a permanent link can be used

Re: [PATCH] configure: Add RTEMS_TOOL_CHAIN_ERROR

2017-01-12 Thread Chris Johns
On 13/01/2017 00:37, Joel Sherrill wrote: I am OK with this and think it will give a more helpful message. I am OK with the change however I would like the patch to wait until docs.rtems.org is up and a permanent link can be used for +AC_DEFUN([RTEMS_TOOL_CHAIN_ERROR],AC_MSG_ERROR(

Re: [PATCH] configure: Add RTEMS_TOOL_CHAIN_ERROR

2017-01-12 Thread Joel Sherrill
I am OK with this and think it will give a more helpful message. Did you notice any other grossly out of date, unneeded tests that generated conditionals for the code? We have been very nice about not hard forcing people to update and things like the sys/cpuset.h tests should just be removed now

[PATCH] configure: Add RTEMS_TOOL_CHAIN_ERROR

2017-01-12 Thread Sebastian Huber
Stop the build with a common error message in case the tool chain is seriously out of date. --- cpukit/configure.ac | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/cpukit/configure.ac b/cpukit/configure.ac index 4410bf1..4aad929 100644 --- a/cpukit