<div dir="auto">Sorry to top post. I am usually on CentOS 7 or whatever rtbf64c is running which is also CentOS 7 I think. I am not in a position to check that this evening.<div dir="auto"><br></div><div dir="auto">What do we need to check to narrow down the difference?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mar 30, 2017 7:15 PM, "Chris Johns" <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 31/03/2017 09:24, Chris Johns wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I am getting this on master (4.12) after a bootstrap on FreeBSD 11.0 so<br>
I am wondering if perl has changed something which effects us.<br>
</blockquote>
<br>
It is not perl or ampolish3 related so the subject has changed.<br>
<br>
It is looking like a mismatch between a command line sort on some hosts and python's sorted where the sb-bootstrap command is being run.<br>
<br>
I have run the sb-bootstrap and the bootstrap script on FreeBSD 11.0 and I get the same results.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Anyone else seeing this?<br>
<br>
</blockquote>
<br>
Joel and Kevin?<br>
<br>
What hosts and what commands did you use to create these patches?<br>
<br>
It looks like the python based sb-bootstrap is doing a case sensitive sort while some hosts are doing a case insensitive sort.<br>
<br>
On FreeBSD 11.0 and 10.3 I get:<br>
<br>
$ echo "bf537Stamp/bsp_specs<br>
> TLL6527M/bsp_specs<br>
> eZKit533/bsp_specs" | sort<br>
TLL6527M/bsp_specs<br>
bf537Stamp/bsp_specs<br>
eZKit533/bsp_specs<br>
$ python<br>
Python 2.7.13 (default, Jan 12 2017, 01:19:30)<br>
[GCC 4.2.1 Compatible FreeBSD Clang 3.8.0 (tags/RELEASE_380/final 262564)] on freebsd11<br>
Type "help", "copyright", "credits" or "license" for more information.<br>
>>> b=['bf537Stamp/bsp_specs', 'TLL6527M/bsp_specs', 'eZKit533/bsp_specs']<br>
>>> sorted(b)<br>
['TLL6527M/bsp_specs', 'bf537Stamp/bsp_specs', 'eZKit533/bsp_specs']<br>
<br>
These results show the sort and python match.<br>
<br>
I have looked into POSIX and sort and the sort order defaults are locale related and the bootstrap script forces C so there looks like a difference between FreeBSD and your hosts in what the defaults are.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I do not want to push a change if conflicts with other hosts.<br>
</blockquote>
<br>
Unless I hear back I think a patch to revert the acinclude.m4 change is ok.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
ruru rtems.git $ git diff<br>
diff --git a/c/src/lib/libbsp/bfin/acincl<wbr>ude.m4<br>
b/c/src/lib/libbsp/bfin/acincl<wbr>ude.m4<br>
index 828fd894dd..ab6082ef47 100644<br>
--- a/c/src/lib/libbsp/bfin/acincl<wbr>ude.m4<br>
+++ b/c/src/lib/libbsp/bfin/acincl<wbr>ude.m4<br>
@@ -2,12 +2,12 @@<br>
AC_DEFUN([RTEMS_CHECK_BSPDIR]<wbr>,<br>
[<br>
case "$1" in<br>
+ TLL6527M )<br>
+ AC_CONFIG_SUBDIRS([TLL6527M]);<wbr>;<br>
bf537Stamp )<br>
AC_CONFIG_SUBDIRS([<wbr>bf537Stamp]);;<br>
eZKit533 )<br>
AC_CONFIG_SUBDIRS([eZKit533])<wbr>;;<br>
- TLL6527M )<br>
- AC_CONFIG_SUBDIRS([TLL6527M]);<wbr>;<br>
*)<br>
AC_MSG_ERROR([Invalid BSP]);;<br>
esac<br>
</blockquote>
<br>
This change is hidden in the mega patch to remove texinfo ..<br>
<br>
<a href="https://git.rtems.org/rtems/commit/?id=48a7fa31f918a6fc88719b3c9393a9ba2829f42a" rel="noreferrer" target="_blank">https://git.rtems.org/rtems/co<wbr>mmit/?id=48a7fa31f918a6fc88719<wbr>b3c9393a9ba2829f42a</a><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
diff --git a/c/src/lib/libbsp/powerpc/aci<wbr>nclude.m4<br>
b/c/src/lib/libbsp/powerpc/aci<wbr>nclude.m4<br>
index edc67b5414..4c83475dd8 100644<br>
--- a/c/src/lib/libbsp/powerpc/aci<wbr>nclude.m4<br>
+++ b/c/src/lib/libbsp/powerpc/aci<wbr>nclude.m4<br>
@@ -32,12 +32,12 @@ AC_DEFUN([RTEMS_CHECK_BSPDIR],<br>
AC_CONFIG_SUBDIRS([t32mppc]);<wbr>;<br>
tqm8xx )<br>
AC_CONFIG_SUBDIRS([tqm8xx]);;<br>
+ virtex )<br>
+ AC_CONFIG_SUBDIRS([virtex]);;<br>
virtex4 )<br>
AC_CONFIG_SUBDIRS([virtex4]);<wbr>;<br>
virtex5 )<br>
AC_CONFIG_SUBDIRS([virtex5]);<wbr>;<br>
- virtex )<br>
- AC_CONFIG_SUBDIRS([virtex]);;<br>
*)<br>
AC_MSG_ERROR([Invalid BSP]);;<br>
esac<br>
</blockquote>
<br>
And in this patch from Kevin ...<br>
<br>
<a href="https://git.rtems.org/rtems/commit/?id=1c6926c11f2e5efcb166c668b097d64a0321d66e" rel="noreferrer" target="_blank">https://git.rtems.org/rtems/co<wbr>mmit/?id=1c6926c11f2e5efcb166c<wbr>668b097d64a0321d66e</a><br>
<br>
Chris<br>
</blockquote></div></div>