[U-Boot] [PATCH v2] MAKEALL: fix get_target_arch() to adjust to '-' in Status field

[U-Boot] [PATCH v2] MAKEALL: fix get_target_arch() to adjust to '-' in Status field

[Masahiro Yamada]

Since the Kconfig conversion, boards.cfg scanned by MAKEALL is
generated by tools/genboardscfg.py.  Every board is supposed to have
its own MAINTAINERS that contains maintainer and status information,
but, in fact, MAINTAINERS is missing from some boards.

For such boards, the first field, Status, is filled with '-'.
It causes a problem for "set" command, which ignores '-' in its
arguments.  Consequently, get_target_arch() returns a wrong field
and MAKEALL fails to get a correct toolchain.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
  - Improve git description

---

 MAKEALL | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/MAKEALL b/MAKEALL
index cd3b6c7..3808a59 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -551,13 +551,7 @@ get_target_maintainers() {
 get_target_arch() {
 	local target=$1
 
-	# Automatic mode
-	local line=`awk '\$7 == "'"$target"'" { print \$0 }' boards.cfg`
-
-	if [ -z "${line}" ] ; then echo "" ; return ; fi
-
-	set ${line}
-	echo "$2"
+	awk '$7 == "'$target'" { print $2 }' boards.cfg
 }
 
 list_target() {
-- 
1.9.1

[U-Boot] [PATCH v2] MAKEALL: fix get_target_arch() to adjust to '-' in Status field

[Simon Glass]

Hi Masahiro,

On 30 March 2015 at 05:59, Masahiro Yamada
<yamada.masahiro@socionext.com> wrote:
> Since the Kconfig conversion, boards.cfg scanned by MAKEALL is
> generated by tools/genboardscfg.py.  Every board is supposed to have
> its own MAINTAINERS that contains maintainer and status information,
> but, in fact, MAINTAINERS is missing from some boards.
>
> For such boards, the first field, Status, is filled with '-'.
> It causes a problem for "set" command, which ignores '-' in its
> arguments.  Consequently, get_target_arch() returns a wrong field
> and MAKEALL fails to get a correct toolchain.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
>   - Improve git description

Should we perhaps apply the patch to deprecate MAKEALL?

Regards,
Simon

[U-Boot] [PATCH v2] MAKEALL: fix get_target_arch() to adjust to '-' in Status field

[Tom Rini]

On Tue, Mar 31, 2015 at 08:39:57PM -0600, Simon Glass wrote:

> Hi Masahiro,
> 
> On 30 March 2015 at 05:59, Masahiro Yamada
> <yamada.masahiro@socionext.com> wrote:
> > Since the Kconfig conversion, boards.cfg scanned by MAKEALL is
> > generated by tools/genboardscfg.py.  Every board is supposed to have
> > its own MAINTAINERS that contains maintainer and status information,
> > but, in fact, MAINTAINERS is missing from some boards.
> >
> > For such boards, the first field, Status, is filled with '-'.
> > It causes a problem for "set" command, which ignores '-' in its
> > arguments.  Consequently, get_target_arch() returns a wrong field
> > and MAKEALL fails to get a correct toolchain.
> >
> > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> >   - Improve git description
> 
> Should we perhaps apply the patch to deprecate MAKEALL?

With the last patch I posted about what binaries to save all my usecases
are covered now.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150401/76ab8dd7/attachment.sig>

[U-Boot] [PATCH v2] MAKEALL: fix get_target_arch() to adjust to '-' in Status field

[Simon Glass]

Hi Tom,

On 1 April 2015 at 05:19, Tom Rini <trini@konsulko.com> wrote:
> On Tue, Mar 31, 2015 at 08:39:57PM -0600, Simon Glass wrote:
>
>> Hi Masahiro,
>>
>> On 30 March 2015 at 05:59, Masahiro Yamada
>> <yamada.masahiro@socionext.com> wrote:
>> > Since the Kconfig conversion, boards.cfg scanned by MAKEALL is
>> > generated by tools/genboardscfg.py.  Every board is supposed to have
>> > its own MAINTAINERS that contains maintainer and status information,
>> > but, in fact, MAINTAINERS is missing from some boards.
>> >
>> > For such boards, the first field, Status, is filled with '-'.
>> > It causes a problem for "set" command, which ignores '-' in its
>> > arguments.  Consequently, get_target_arch() returns a wrong field
>> > and MAKEALL fails to get a correct toolchain.
>> >
>> > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
>> >   - Improve git description
>>
>> Should we perhaps apply the patch to deprecate MAKEALL?
>
> With the last patch I posted about what binaries to save all my usecases
> are covered now.

OK, do you think we can wait until the merge window before doing
these? I don't see any great hurry.

Regards,
Simon

[U-Boot] [PATCH v2] MAKEALL: fix get_target_arch() to adjust to '-' in Status field

[Masahiro Yamada]

Hi Simon, Tom,


2015-04-06 3:31 GMT+09:00 Simon Glass <sjg@chromium.org>:
> Hi Tom,
>
> On 1 April 2015 at 05:19, Tom Rini <trini@konsulko.com> wrote:
>> On Tue, Mar 31, 2015 at 08:39:57PM -0600, Simon Glass wrote:
>>
>>> Hi Masahiro,
>>>
>>> On 30 March 2015 at 05:59, Masahiro Yamada
>>> <yamada.masahiro@socionext.com> wrote:
>>> > Since the Kconfig conversion, boards.cfg scanned by MAKEALL is
>>> > generated by tools/genboardscfg.py.  Every board is supposed to have
>>> > its own MAINTAINERS that contains maintainer and status information,
>>> > but, in fact, MAINTAINERS is missing from some boards.
>>> >
>>> > For such boards, the first field, Status, is filled with '-'.
>>> > It causes a problem for "set" command, which ignores '-' in its
>>> > arguments.  Consequently, get_target_arch() returns a wrong field
>>> > and MAKEALL fails to get a correct toolchain.
>>> >
>>> > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
>>> >   - Improve git description
>>>
>>> Should we perhaps apply the patch to deprecate MAKEALL?
>>
>> With the last patch I posted about what binaries to save all my usecases
>> are covered now.
>
> OK, do you think we can wait until the merge window before doing
> these? I don't see any great hurry.
>

I know MAKEALL is deprecated, but I still use it with a local-hack to
compare MD5SUM.

Buildman is a bigger program than MAKEALL and I have not been able to
find time to
dig into it.

The above is my personal reason, but this patch is an apparent bug-fix
and it is easy enough.
Why don't we fix it?


-- 
Best Regards
Masahiro Yamada

[U-Boot] [PATCH v2] MAKEALL: fix get_target_arch() to adjust to '-' in Status field

[Simon Glass]

Hi Masahiro,

On 5 April 2015 at 21:08, Masahiro Yamada <yamada.masahiro@socionext.com> wrote:
>
> Hi Simon, Tom,
>
>
> 2015-04-06 3:31 GMT+09:00 Simon Glass <sjg@chromium.org>:
> > Hi Tom,
> >
> > On 1 April 2015 at 05:19, Tom Rini <trini@konsulko.com> wrote:
> >> On Tue, Mar 31, 2015 at 08:39:57PM -0600, Simon Glass wrote:
> >>
> >>> Hi Masahiro,
> >>>
> >>> On 30 March 2015 at 05:59, Masahiro Yamada
> >>> <yamada.masahiro@socionext.com> wrote:
> >>> > Since the Kconfig conversion, boards.cfg scanned by MAKEALL is
> >>> > generated by tools/genboardscfg.py.  Every board is supposed to have
> >>> > its own MAINTAINERS that contains maintainer and status information,
> >>> > but, in fact, MAINTAINERS is missing from some boards.
> >>> >
> >>> > For such boards, the first field, Status, is filled with '-'.
> >>> > It causes a problem for "set" command, which ignores '-' in its
> >>> > arguments.  Consequently, get_target_arch() returns a wrong field
> >>> > and MAKEALL fails to get a correct toolchain.
> >>> >
> >>> > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> >>> >   - Improve git description
> >>>
> >>> Should we perhaps apply the patch to deprecate MAKEALL?
> >>
> >> With the last patch I posted about what binaries to save all my usecases
> >> are covered now.
> >
> > OK, do you think we can wait until the merge window before doing
> > these? I don't see any great hurry.
> >
>
> I know MAKEALL is deprecated, but I still use it with a local-hack to
> compare MD5SUM.
>
> Buildman is a bigger program than MAKEALL and I have not been able to
> find time to
> dig into it.
>
> The above is my personal reason, but this patch is an apparent bug-fix
> and it is easy enough.
> Why don't we fix it?

I wan't suggesting we don't fix it! I agree you patch is a good thing
- it just jogged my memory about MAKEALL deprecation.

Regards,
Simon

[U-Boot] [PATCH v2] MAKEALL: fix get_target_arch() to adjust to '-' in Status field

[Tom Rini]

On Sun, Apr 05, 2015 at 12:31:44PM -0600, Simon Glass wrote:
> Hi Tom,
> 
> On 1 April 2015 at 05:19, Tom Rini <trini@konsulko.com> wrote:
> > On Tue, Mar 31, 2015 at 08:39:57PM -0600, Simon Glass wrote:
> >
> >> Hi Masahiro,
> >>
> >> On 30 March 2015 at 05:59, Masahiro Yamada
> >> <yamada.masahiro@socionext.com> wrote:
> >> > Since the Kconfig conversion, boards.cfg scanned by MAKEALL is
> >> > generated by tools/genboardscfg.py.  Every board is supposed to have
> >> > its own MAINTAINERS that contains maintainer and status information,
> >> > but, in fact, MAINTAINERS is missing from some boards.
> >> >
> >> > For such boards, the first field, Status, is filled with '-'.
> >> > It causes a problem for "set" command, which ignores '-' in its
> >> > arguments.  Consequently, get_target_arch() returns a wrong field
> >> > and MAKEALL fails to get a correct toolchain.
> >> >
> >> > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> >> >   - Improve git description
> >>
> >> Should we perhaps apply the patch to deprecate MAKEALL?
> >
> > With the last patch I posted about what binaries to save all my usecases
> > are covered now.
> 
> OK, do you think we can wait until the merge window before doing
> these? I don't see any great hurry.

I can continue cursing and doing make O=temp by hand for a bit longer,
sure ;)

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150406/2d92db53/attachment.sig>