public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Jeroen Hofstee <jeroen@myspectrum.nl>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Makefile: restore *_config target
Date: Mon, 04 Aug 2014 19:35:19 +0200	[thread overview]
Message-ID: <53DFC457.2010907@myspectrum.nl> (raw)
In-Reply-To: <CAPnjgZ22sLg8e-cEioA_LgqL6vE2OCWS+razCXMcRqEuZuSwxA@mail.gmail.com>

Hello Simon,

On 04-08-14 12:09, Simon Glass wrote:
> Hi Jeroen,
>
> On 1 August 2014 13:28, Jeroen Hofstee <jeroen@myspectrum.nl> wrote:
>> The switch to KConfig break the [board]_config target. This breaks
>> backwards compatibility and hence all documentation about how to
>> build u-boot is no longer valid. It is also annoying for general
>> recipes to build u-boot as contained in buildroot, crochet, oe,
>> poky, eldk etc. Hence restore the *_config target.
>>
>> cc: Tom Rini <trini@ti.com>
>> cc: Masahiro Yamada <yamada.m@jp.panasonic.com>
>> cc: Simon Glass <sjg@chromium.org>
>> Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl>
> Is there any way we can emit a deprecation warning without breaking these tools?
>

I guess the number of ways to do that are just limited by our imagination:
- abusing the bell character can be quite annoying, but is broken / 
fixed in the
   linux distro's I know.
- echo "Hi there, U-boot folks over here, please STOP using _config, use 
_defconfig instead!" | write $USER
   will scare quite a few users I guess.
- sendmail + git config user.email of the OE tree should be annoying enough
- lpr ;)
- play ?
- figure out which oe recipe invoked it and patch it from u-boot ;)

And as a serious answer, no I don't know a decent method
to do it. As far as I know OE e.g. just logs to a file and only
displays its contents if an error occurred.

Afaiac, the target is just kept. Just to prevent wasting time
updating all wikis, docs etc, recipe etc. (it is not that it is a
big pile of code or something). I can add an echo of course,
but it won't be noticed by such build tools.

Regards,
Jeroen

      reply	other threads:[~2014-08-04 17:35 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-01 19:28 [U-Boot] [PATCH] Makefile: restore *_config target Jeroen Hofstee
2014-08-04  1:51 ` Masahiro Yamada
2014-08-04 17:49   ` Jeroen Hofstee
2014-08-05  1:50     ` Masahiro Yamada
2014-08-05 19:35       ` Jeroen Hofstee
2014-08-06  2:11         ` Masahiro Yamada
2014-08-07 20:40           ` Jeroen Hofstee
2014-08-04 10:09 ` Simon Glass
2014-08-04 17:35   ` Jeroen Hofstee [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53DFC457.2010907@myspectrum.nl \
    --to=jeroen@myspectrum.nl \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox