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 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.