Hi Max, On Fri, 4 Jan 2019 at 11:25, Max Filippov <[email protected]> wrote: > On Thu, Jan 3, 2019 at 9:17 PM Firoz Khan <[email protected]> wrote: > > On Fri, 4 Jan 2019 at 08:28, Max Filippov <[email protected]> wrote: > > > On Wed, Jan 2, 2019 at 7:29 AM Firoz Khan <[email protected]> wrote: > > This was intentional as I was trying compile other one architecture with the > > previous implementation, it was giving the error. So I modified without > > space > > after comma. > > I have tested that presence or absence of space does not affect the build > for xtensa, and normally we put space after a comma, so I've mentioned that. > > Since this script is anyway removed in the next patch of the series it doesn't > really matter whether the spaces are present or not.
Yes, you are right, there is no effect in xtensa but if I'm right there is a problem in powerpc architecture (not remembering exactly which arch) with this space after comma. BTW, the script from all architecture will be removed and keep it in scripts directory is our plan. So to satisfy powerpc (again I'm not sure which arch) I just did this change. Thanks for your feedback. Firoz
