All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: Richard Henderson <richard.henderson@linaro.org>
Cc: qemu-devel@nongnu.org
Subject: Re: [PATCH] .gitignore: include common build sub-directories
Date: Tue, 14 Apr 2020 17:31:55 +0100	[thread overview]
Message-ID: <87v9m281vo.fsf@linaro.org> (raw)
In-Reply-To: <c137f5b3-3e93-e497-e2d2-16ced7d2fb42@linaro.org>


Richard Henderson <richard.henderson@linaro.org> writes:

> On 4/13/20 9:29 AM, Alex Bennée wrote:
>> As out-of-tree builds become more common (or rather building in a
>> subdir) we can add a lot of load to "git ls-files" as it hunts down
>> sub-directories that are irrelevant to the source tree. This is
>> especially annoying if you have a prompt that attempts to summarise
>> the current git status on command completion.
>
> Not a fan.  Why do we encourage putting the build directory in tree anyway?
> IMO any out-of-tree build really should be OUT of tree.
>
> My build directories are always siblings of the source directory in the parent
> directory.


It's not exactly an un-common pattern - especially if you have a lot of
build variants. Keeping them under builds makes it easier to blow the
whole lot away rather than having a messy parent directory which you
have to clean manually.

Personally it also works better with my tooling which can slurp up all
the build directories and present me with a menu of make commands to
build from. That's not a good argument for accepting the patch though.

>
>
> r~


-- 
Alex Bennée


  reply	other threads:[~2020-04-14 16:37 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-13 16:29 [PATCH] .gitignore: include common build sub-directories Alex Bennée
2020-04-13 18:26 ` Eric Blake
2020-04-13 21:32   ` Alex Bennée
2020-04-13 21:42     ` Eric Blake
2020-04-13 21:45       ` Philippe Mathieu-Daudé
2020-04-14  8:00       ` Markus Armbruster
2020-04-14  8:47         ` Alex Bennée
2020-04-14  9:53           ` Philippe Mathieu-Daudé
2020-04-14 15:06 ` Richard Henderson
2020-04-14 16:31   ` Alex Bennée [this message]
2020-04-14 16:39     ` Richard Henderson

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=87v9m281vo.fsf@linaro.org \
    --to=alex.bennee@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    /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.