public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Ulf Samuelsson <ulfs@dof.se>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] Makefile suggestion
Date: Wed, 23 Aug 2006 22:21:54 +0200	[thread overview]
Message-ID: <008f01c6c6f2$1b58fd80$8f4765d5@atmel.com> (raw)
In-Reply-To: 20060822202630.9EAAA353A61@atlas.denx.de

Have been thinking about how to add patches in a non-intrusive way.

The U-boot Makefile is quite large and growing.
I have seen problem where a new patch breaks because another patch has been applied 
between the checkout of the source dir and the acceptance of the patch.
("avr32" board support breaks on "blackfin")

Even if you make sure that the patch is OK vs todays git, 
if another guy sends in a board patch at the same time,
and they are close to each other, then the first one
to get acceptance first "wins".

The main function of the proposed patch is to add:
"include    board/*/board.mk"
to the toplevel Makefile right before the mkconfig's start

Each provider of a new board package can put his/her own
./mkconfig xxx xxx xxx xxx xxx xxx
 inside the board directory board.mk file.

There are no board/board.mk files in the current tree, so I do not think there is any conflict with existing boards.
If there aren't any "board/*/board.mk" files present, "make" fails, 
so the patch also creates a dummy file  : "board/template/board.mk".

The patch does not do *anything* at the moment except including empty files in the make process.
It does create a platform to allow easier patch management for both developers and Wolfgang
 (if he thinks it is a good idea and accepts the patch)

This is the first patch I send, if I have made any glaring mistakes, 
I will try to improve based on feedback.

Best Regards
Ulf Samuelsson

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0.Makefile.patch
Type: application/octet-stream
Size: 821 bytes
Desc: not available
Url : http://lists.denx.de/pipermail/u-boot/attachments/20060823/74a25a64/attachment.obj 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0.Makefile.CHANGELOG
Type: application/octet-stream
Size: 272 bytes
Desc: not available
Url : http://lists.denx.de/pipermail/u-boot/attachments/20060823/74a25a64/attachment-0001.obj 

  parent reply	other threads:[~2006-08-23 20:21 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-22  9:45 [U-Boot-Users] Submitting Patch at sourceforge fails Ulf Samuelsson
2006-08-22 10:25 ` Wolfgang Denk
2006-08-22 13:52   ` Ulf Samuelsson
2006-08-22 15:38     ` Wolfgang Denk
2006-08-22 19:02       ` Ulf Samuelsson
2006-08-22 19:02       ` Ulf Samuelsson
2006-08-22 20:26         ` Wolfgang Denk
2006-08-23 15:03           ` Andreas Schweigstill
2006-08-23 15:23             ` Jeff Mock
2006-08-23 16:44               ` [U-Boot-Users] Off-topic: Serial port redirection Andreas Schweigstill
2006-08-23 19:35                 ` Wolfgang Denk
2006-08-23 19:48                   ` Jerry Van Baren
2006-08-23 20:45             ` [U-Boot-Users] Submitting Patch at sourceforge fails Ulf Samuelsson
2006-08-23 22:25               ` Wolfgang Denk
2006-08-23 20:21           ` Ulf Samuelsson [this message]
2006-08-23 22:07             ` [U-Boot-Users] Makefile suggestion Wolfgang Denk
2006-08-24  7:53               ` Ulf Samuelsson
2006-08-24  9:29                 ` Wolfgang Denk
2006-08-23  8:06         ` [U-Boot-Users] Submitting Patch at sourceforge fails Stefan Roese
2006-08-23  8:13           ` Ulf Samuelsson
2006-08-23  9:46             ` Stefan Roese

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='008f01c6c6f2$1b58fd80$8f4765d5@atmel.com' \
    --to=ulfs@dof.se \
    --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