From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Mon, 24 Sep 2012 19:25:30 +0200 Subject: [U-Boot] [PATCH 3/3] common: Add .u_boot_list into all linker files In-Reply-To: <20120924171813.GE1797@bill-the-cat> References: <1348449718-16463-1-git-send-email-marex@denx.de> <201209241909.35433.marex@denx.de> <20120924171813.GE1797@bill-the-cat> Message-ID: <201209241925.30930.marex@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Dear Tom Rini, > On Mon, Sep 24, 2012 at 07:09:35PM +0200, Marek Vasut wrote: > > Dear Tom Rini, > > > > > On Mon, Sep 24, 2012 at 06:53:19PM +0200, Marek Vasut wrote: > > > > Dear Joe Hershberger, > > > > > > > > [...] > > > > > > > > > While you're touching all these files, why not replace the > > > > > __u_boot_cmd* implementation with this as well. There's no need to > > > > > leave a special case in there as well is there? > > > > > > > > Not in the first stab, it can be indeed done later, but now I'd like > > > > to avoid breakage. Besides, replacing u_boot_cmd would break > > > > bisectability, I want to preserve it and rather apply more patches > > > > slowly than less patches recklessly. > > > > > > Agreed. We need to work hard to make sure we do lots of incremental > > > steps here to make sure breakage can be bisected. > > > > +1 (actually +inf.) > > > > > Making use of the > > > 'exec' step in git rebase is very helpful for making sure this is the > > > case too, btw. > > > > exec step ? > > Yes, in 'git rebase -i' you can insert exec lines after each commit such > as: > pick 12345 Commit 1 > exec script-that-runs-MAKEALL.sh > pick 6789a Commit 2 > exec script-that-runs-MAKEALL.sh > > I use my MAKEALL wrapper (http://pastebin.com/fNhG4iCd but I've updated > a bit more since I uploaded that) and --log `git rev-parse --short HEAD` > so I can see what's gone on for every step in a series. I guess you have one hell of a badass computer cluster then ;-) Best regards, Marek Vasut