From: Ulf Samuelsson <ulf.samuelsson@atmel.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] RFC: "make DESTDIR=xxx install" ?
Date: Sat, 15 Aug 2009 11:35:11 +0200 [thread overview]
Message-ID: <4A86814F.1070805@atmel.com> (raw)
In-Reply-To: <20090815063258.6D0BC833DBD2@gemini.denx.de>
Wolfgang Denk skrev:
> Dear Ulf Samuelsson,
>
> In message <4A864121.908@atmel.com> you wrote:
>> I think the open source community has converged on the
>> "make DESTDIR=<dir> install" method
>
> $ cd linux
> $ make at91rm9200dk_defconfig
> $ make uImage
> $ mkdir /tmp/foo
> $ mkdir DESTDIR=/tmp/foo install
> ...
> CHK include/linux/compile.h
> Kernel: arch/arm/boot/Image is ready
> /bin/sh /home/wd/git/linux/work/arch/arm/boot/install.sh 2.6.30-rc8-01295-g06b727a \
> arch/arm/boot/Image System.map "/boot"
> Installing normal kernel
> /home/wd/git/linux/work/arch/arm/boot/install.sh: line 40: /boot/vmlinux-2.6.30-rc8-01295-g06b727a: Permission denied
> cp: cannot create regular file `/boot/System.map-2.6.30-rc8-01295-g06b727a': Permission denied
> You have to install it yourself
>
> Hmmm... doesn't seem to ork for me.
There are plenty examples of where it will work, as you know.
>
>> The important thing is however that the solution is
>
> Important for what?
To avoid that external build systems break if anything changes in u-boot.
>
>> 1) INSIDE the U-Boot tree
>> 2) Designed to be stable, even if U-Boot evolves.
>> 3) Documented so it is easy to use.
>
> So far you seem to be the only person who needs this.
>
>> If your proposal is that the "wrapper" script is outside u-boot,
>> then this is nothing new. This is how it is done today,
>> Having wrapper scripts to an unstable interface is an
>> accident waiting to happen.
>
> Could you please explain which part of this has been an unstable
> interface? As far as I can tell PPCBoot / ARMBoot / U-Boot have
> always created the "final binary image" as you called it in the top
> level directory, and also it's name has never changed. So what
> exactly is unstable here?
You mentioned yourself that Marvell want to have something different
than u-boot.bin
>
>> You have a unique position as the maintainer of U-Boot.
>
> I don't. The U-Boot project is driven by a community. If a clear
> majority of voices requests something I would have hard times to make
> my way.
I am not talking about decisions, I am talking about you not running
into problems, if anything changes, because you know it by heart.
>
>> You know immediately when your scripts needs to be updated.
>
> Fact is that I am using some scripts that are 10 years old now, and
> there has never been need to change them because of changes in the
> PPCBoot/ARMBoot/U-Boot "interface" - not even when ARMBoot was forked
> from PPCBoot, nor when PPCBoot and ARMBoot were merged back into
> U-Boot.
>
>> People working on a build environment does not neccessarily
>> have that knowledge, and if not, will run into trouble,
>> or rather their customers might.
>>
>> Maybe I misunderstand you and you propose that the build-script
>> should be inside the tree.
>> Please clarify!
>
> I still fail to understand which sort of trouble you are talking about.
>
next prev parent reply other threads:[~2009-08-15 9:35 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-13 15:38 [U-Boot] RFC: "make DESTDIR=xxx install" ? Ulf Samuelsson
2009-08-13 18:06 ` Wolfgang Denk
2009-08-13 20:29 ` Ulf Samuelsson
2009-08-13 21:03 ` Wolfgang Denk
2009-08-15 5:01 ` Ulf Samuelsson
2009-08-15 6:32 ` Wolfgang Denk
2009-08-15 9:35 ` Ulf Samuelsson [this message]
2009-08-15 10:42 ` Wolfgang Denk
2009-08-13 18:06 ` ksi at koi8.net
2009-08-13 20:29 ` Ulf Samuelsson
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=4A86814F.1070805@atmel.com \
--to=ulf.samuelsson@atmel.com \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.