linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jeff Garzik <jgarzik@pobox.com>
To: Andries Brouwer <aebr@win.tue.nl>
Cc: "J.A. Magallon" <jamagallon@able.es>,
	SCSI Mailing List <linux-scsi@vger.kernel.org>,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: libata update posted
Date: Sun, 14 Sep 2003 16:09:44 -0400	[thread overview]
Message-ID: <3F64CB08.9010506@pobox.com> (raw)
In-Reply-To: <20030914213821.A11134@pclin040.win.tue.nl>

Andries Brouwer wrote:
> On Sun, Sep 14, 2003 at 01:26:21PM -0400, Jeff Garzik wrote:
> 
> 
>>>This shows why defkeymap.c is not generated in the kernel build
>>>but distributed.
>>
>>There is a difference between distributing generated files, and checking 
>>generated files into a repository...  I do not advocate changing the 
>>tarball, just the BK repo behind it.
> 
> 
> So you would like to remove defkeymap.c from the bitkeeper repository.
> Can you briefly explain why?
> I am not a bk user so have no feeling for what one would like bk to do.
> 
> But it seems to me that if defkeymap.c is only a generated file when
> no kbd headers have changed, while in the opposite case one needs a
> private version of loadkeys until the next version of kbd has been
> distributed, it is easier to regard it as part of the kernel source.

defkeymap.c is _always_ a generated file.  However, some environments 
lack the capability to generate it (properly).

My motivation is not bitkeeper-specific:  I dislike the practice of 
checking build-generated files into an SCM of any sort.  That sort of 
stuff should be handled by the maintainer, when generating the release 
patches and tarballs.  aic7xxx is another example

Package developers (read: kernel hackers) are expected to have a correct 
environment to rebuild generated files properly.  Package builders are 
only expected to have a minimal build environment, with stuff like 
configure scripts, yacc/lex output, defkeymap, and similar things 
pre-generated from a canonical source (the maintainer).

The GNOME guys recognized this distinction between hackers and builders 
a long time ago.  When you check things out of GNOME cvs, you get _just_ 
the sources.  One must run ./autogen.sh to generate the 
autoconf/automake/libtool junk needed to actually build successfully.

	Jeff




  reply	other threads:[~2003-09-14 20:10 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-09-13  3:23 libata update posted Jeff Garzik
2003-09-13 20:56 ` J.A. Magallon
2003-09-13 21:28   ` Jeff Garzik
2003-09-14  9:12     ` Andries Brouwer
2003-09-14 17:26       ` Jeff Garzik
2003-09-14 19:38         ` Andries Brouwer
2003-09-14 20:09           ` Jeff Garzik [this message]
2003-09-14 21:09             ` Andries Brouwer
2003-09-14 21:36               ` Jeff Garzik
2003-09-14 21:38                 ` James Bottomley
2003-09-14 22:18                   ` Andries Brouwer
2003-09-16 19:34                 ` Pavel Machek
2003-09-13 21:13 ` J.A. Magallon
2003-09-13 21:38   ` Jeff Garzik
2003-09-14 22:27     ` Hugo Mills
2003-09-14 22:31       ` Jeff Garzik
  -- strict thread matches above, loose matches on Subject: below --
2003-10-05 23:15 Jeff Garzik
2003-10-05 23:41 ` Bartlomiej Zolnierkiewicz
2003-08-13  3:40 Jeff Garzik

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=3F64CB08.9010506@pobox.com \
    --to=jgarzik@pobox.com \
    --cc=aebr@win.tue.nl \
    --cc=jamagallon@able.es \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).