All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick McHardy <kaber@trash.net>
To: Torsten Luettgert <t.luettgert@pressestimmen.de>
Cc: netfilter-devel@lists.netfilter.org
Subject: Re: nth match
Date: Sat, 27 May 2006 19:45:30 +0200	[thread overview]
Message-ID: <4478903A.3080402@trash.net> (raw)
In-Reply-To: <1148750003.2259.32.camel@murdegern.hindenburgdamm.example>

Torsten Luettgert wrote:
> On Sat, 2006-05-27 at 17:48 +0200, Patrick McHardy wrote:
> 
>> nor is the
>>time undetermined, it will go in 2.6.18. In fact it is queued
>>for upstream submission sometime this weekend.
> 
> 
> That's good to hear. How about putting a patchlet dir into the
> repository? I could update my scripts that use nth to statistics
> once I see how it's used.

The removal was part of a large patch-o-matic cleanup to reduce
the overhead for us. Development today happens a lot nearer to
the kernel than it used to (which is good), so I don't think that
would do any good. I'll happily publish my git tree if anyone is
interested, but for 2.6.18 it doesn't really matter anymore since
I'm going to push out most of my patches this weekend.

>>I certainly won't let trivial patches bitrot in patch-o-matic.
>>Unlike the kernel, patch-o-matic never made any promises about backwards
>>compatibility, support or anything else.
> 
> 
> Oh, I never said you "must" do anything because you promised it
> or something. It's just that something I used and that works
> well just disappeared. Would it hurt so much to leave it in
> while it's still working until an alternative is available?

No, it wouldn't, but now its gone and we didn't even release the
stripped down pom yet, so I think people can just as well use the
last release.

> I always thought pom was an effort to bring useful functionality
> which didn't make the long, painful way into the official kernel
> (and probably never will) to simple users like me who can benefit
> from it, folks that don't have the time (or ability) to write it
> for themselves.
> 
> Apparently you're seeing it more as a step stone on the way to
> kernel inclusion. Now, it's you who are part of the team
> and you're investing a lot of your own time for pom, not me,
> so I'll just have to make do with what you're offering.

Actually I'm seeing it mostly as obsolete. Most of the patches
still in there will get merged at some point. As for step stone,
I think it many cases its counterproductive to put patches in
there because they will bitrot, stop to apply and not get nearly
as much testing as in the mainline kernel. Its usually even less
work just doing the last bits I see necessary for kernel inclusion
myself than apply to pom, wait, apply update, fix compilation,
wait some more, ...

>>You may also have noticed that it didn't even compile with current
>>kernels.
> 
> 
> Interesting. I just made a 2.6.16.18 with it, and it compiled
> just fine. Does it break on 2.6.17?

Yes. It also broke for 2.6.16, but was fixed.

> Sorry again, didn't mean to personally attack anyone.

Don't worry, no harm done.

  reply	other threads:[~2006-05-27 17:45 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-05-27 15:19 nth match Torsten Luettgert
2006-05-27 15:48 ` Patrick McHardy
2006-05-27 17:13   ` Torsten Luettgert
2006-05-27 17:45     ` Patrick McHardy [this message]
2006-05-28  0:13       ` Torsten Luettgert
2006-05-28 12:00         ` Patrick McHardy

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=4478903A.3080402@trash.net \
    --to=kaber@trash.net \
    --cc=netfilter-devel@lists.netfilter.org \
    --cc=t.luettgert@pressestimmen.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 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.