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 17:48:35 +0200	[thread overview]
Message-ID: <447874D3.3030901@trash.net> (raw)
In-Reply-To: <1148743183.2259.8.camel@murdegern.hindenburgdamm.example>

Torsten Luettgert wrote:
> Hello,
> 
> I understand Patrick wanted to integrate nth and random
> into a new statistics match. That's fine with me, but why
> was nth and random removed from pom without any replacement,
> i.e. the new statistics match?
> 
> You're just ripping out functionality based on something which
> will probably appear at some undetermined time in the future.

I did neither "rip something out", as the match was never part
of anything but a mostly random crap collection, 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 no good style, sorry. Good style would be to first write
> the new match, adding it, giving it time to mature and marking
> the old stuff as obsolescent, then at some time removing it.

You're free to grab one of the many releases of patch-o-matic
that still contain the nth match and use that. 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. You may also have
noticed that it didn't even compile with current kernels.

> (P.S.: sorry if this appears twice. I was so pissed off at the
> time that I forgot to use the correct sender address)

If you're so pissed off, how about you maintain a version until
2.6.18 is released? Feel free to set up a repository and send
me the URL to be included in the distibuted sources.list.

  reply	other threads:[~2006-05-27 15:48 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 [this message]
2006-05-27 17:13   ` Torsten Luettgert
2006-05-27 17:45     ` Patrick McHardy
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=447874D3.3030901@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.