public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Krzysztof Halasa <khc@pm.waw.pl>
To: Paul Fulghum <paulkf@microgate.com>
Cc: "Jeff Garzik" <jeff@garzik.org>,
	"Toralf Förster" <toralf.foerster@gmx.de>,
	linux-kernel@vger.kernel.org, "Andrew Morton" <akpm@osdl.org>
Subject: Re: [PATCH] Re: linux-2.6.19-rc5-g088406bc build #120 failed
Date: Mon, 13 Nov 2006 19:33:18 +0100	[thread overview]
Message-ID: <m3ejs78adt.fsf@defiant.localdomain> (raw)
In-Reply-To: <45588895.7010501@microgate.com> (Paul Fulghum's message of "Mon, 13 Nov 2006 09:00:37 -0600")

Paul Fulghum <paulkf@microgate.com> writes:

> synclink drivers can *optionally* support generic hdlc.
> You *must* be able to build synclink driver without generic hdlc.
> Because of this you *can't* just put in the generic hdlc dependency.
>
> Several alternative patches were posted (3 or 4 months) ago.
> No particular patch won the approval of all kernel developers,
> so nothing was done.

I remember something actually did "won the approval". Don't remember
the details but I think it included conditional compilation, not
changing CONFIG_* macros (using different name(s) instead, just not
starting with "CONFIG_"), and IIRC no changes to Kconfigs.

At least I don't remember any objections to that last version so
I assumed it's closed.

Perhaps it was just a dream, who knows :-)

Nevertheless a fix outlined above would be acceptable, wouldn't it?
-- 
Krzysztof Halasa

  parent reply	other threads:[~2006-11-13 18:33 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-13  8:43 linux-2.6.19-rc5-g088406bc build #120 failed Toralf Förster
2006-11-13 14:49 ` [PATCH] " Jeff Garzik
2006-11-13 15:00   ` Paul Fulghum
2006-11-13 15:14     ` Jeff Garzik
2006-11-13 15:37       ` Paul Fulghum
2006-11-13 18:33     ` Krzysztof Halasa [this message]
2006-11-13 18:54       ` Paul Fulghum
2006-11-13 20:49         ` Krzysztof Halasa
2006-11-13 21:40           ` Paul Fulghum
2006-11-13 23:05             ` Krzysztof Halasa
2006-11-13 23:10               ` Paul Fulghum
2006-11-14  0:07                 ` Krzysztof Halasa
2006-11-14  1:08                   ` Paul Fulghum
2006-11-14 14:25                     ` Krzysztof Halasa
2006-11-13 23:13               ` Randy Dunlap
2006-11-13 23:06             ` Andrew Morton
2006-11-13 23:15               ` Paul Fulghum
2006-11-14  0:00                 ` Daniel Barkalow
2006-11-13 21:36   ` Toralf Förster

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=m3ejs78adt.fsf@defiant.localdomain \
    --to=khc@pm.waw.pl \
    --cc=akpm@osdl.org \
    --cc=jeff@garzik.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=paulkf@microgate.com \
    --cc=toralf.foerster@gmx.de \
    /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