From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [RFC] Review request for branch for-2011.08/bootloaders
Date: Mon, 30 May 2011 22:34:03 +0200 [thread overview]
Message-ID: <20110530223403.0f8fcbd4@surf> (raw)
In-Reply-To: <BANLkTim_Bb3fRZcC4rT2ofpF7GhuroUULA@mail.gmail.com>
G?nayd?n Alper!
On Mon, 30 May 2011 22:53:11 +0300
Alper Y?ld?r?m <yildirimalper@gmail.com> wrote:
> Thanks for the comprehensive work. I have a suggestion, maybe you can also
> implement this within the context of your work.
> In our project, we have some local applications, hosted under local git
> repositories. We are currently using the package infrastructure with git. If
> we give a commit id or tag name everything is fine but in some use cases we
> want to build the latest commit (HEAD) with buildroot. In that case, giving
> HEAD as the version is not sufficient, since buildroot checks the dl dir for
> pkgname-HEAD.tar.gz and finds it there after first build. Currently we
> manually delete this file and force buildroot to clone the repo again. In my
> opinion this is not the proper way.
>
> My suggestion is:
> For the first build, clone the git repo to the build directory and for the
> next builds make a git pull in the build directory.
>
> It would be nice to have such an option in buildroot.
I think I understand your workflow and your need. However, I
unfortunately don't think your proposal works properly: the next time
you will do "make clean", the build directory of your package will have
disappeared, so Buildroot won't know that it should do a "git pull",
and will use again the pkgname-HEAD.tar.gz file from the dl directory.
Or maybe I misunderstood your proposal ?
In your case, wouldn't it be better to not let Buildroot download your
custom applications from Git, but rather give Buildroot the path to a
directory where you have the source of code of the applications ? This
way, you can handle this directory however you want in terms of
revision control: Buildroot will never remove it.
What do you think of the proposal I've sent on May, 18th ? See
http://lists.busybox.net/pipermail/buildroot/2011-May/043320.html.
Thanks for your suggestion!
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2011-05-30 20:34 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-16 20:21 [Buildroot] [RFC] Review request for branch for-2011.08/bootloaders Thomas Petazzoni
2011-05-16 20:21 ` [Buildroot] [PATCH 01/21] Makefile.package.in: Makes it possible to override the default extract commands Thomas Petazzoni
2011-05-16 20:21 ` [Buildroot] [PATCH 02/21] Fix quoting in default extract command Thomas Petazzoni
2011-05-16 20:21 ` [Buildroot] [PATCH 03/21] linux: make it possible to install the kernel in /boot Thomas Petazzoni
2011-05-16 20:21 ` [Buildroot] [PATCH 04/21] linux: use the defconfig mechanism after tuning the config Thomas Petazzoni
2011-05-16 20:21 ` [Buildroot] [PATCH 05/21] package: add infrastructure to install things in images/ Thomas Petazzoni
2011-05-16 20:21 ` [Buildroot] [PATCH 06/21] package: fix patching procedure for host packages Thomas Petazzoni
2011-05-16 20:21 ` [Buildroot] [PATCH 07/21] barebox: use = instead of := and re-indent Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 08/21] barebox: convert to GENTARGETS Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 09/21] at91bootstrap: use = instead of := and indent properly Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 10/21] at91bootstrap: convert to GENTARGETS Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 11/21] at91dataflashboot: use = instead of := and indent Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 12/21] at91dataflashboot: convert to GENTARGETS Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 13/21] syslinux: " Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 14/21] grub: convert to AUTOTARGETS Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 15/21] grub2: remove package Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 16/21] u-boot: convert to GENTARGETS Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 17/21] uboot-mkimage: new package Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 18/21] linux: rename LINUX26 to LINUX Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 19/21] linux: indent and align some definitions Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 20/21] package: add support for post-download hooks Thomas Petazzoni
2011-05-16 20:22 ` [Buildroot] [PATCH 21/21] linux: convert to the GENTARGETS infrastructure Thomas Petazzoni
2011-05-27 22:16 ` [Buildroot] [RFC] Review request for branch for-2011.08/bootloaders Luca Ceresoli
2011-05-30 20:27 ` Thomas Petazzoni
2011-05-30 19:53 ` Alper Yıldırım
2011-05-30 20:34 ` Thomas Petazzoni [this message]
2011-06-08 20:11 ` Alper Yıldırım
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=20110530223403.0f8fcbd4@surf \
--to=thomas.petazzoni@free-electrons.com \
--cc=buildroot@busybox.net \
/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