All of lore.kernel.org
 help / color / mirror / Atom feed
From: Phil Oester <kernel@linuxace.com>
To: "Jérôme Poulin" <jeromepoulin@gmail.com>
Cc: netfilter-devel@lists.netfilter.org
Subject: Re: 3896 pom-ng broken
Date: Tue, 10 May 2005 17:51:21 -0700	[thread overview]
Message-ID: <20050511005121.GA31199@linuxace.com> (raw)
In-Reply-To: <debc30fc050510121564dd0703@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 643 bytes --]

On Tue, May 10, 2005 at 03:15:25PM -0400, Jérôme Poulin wrote:
> On 5/10/05, Jérôme Poulin <jeromepoulin@gmail.com> wrote:
> > See my e-mail, posted the same problem / solution and got no answer,
> > only 3 patchs apply with that new snapshot, other patch with no
> > requirements or even with a valid requirement won't apply, only set,
> > h323, goto and another apply, anyway see my other mails.

Indeed, my patch was broken.  But the 'Requires' handling is also broken, but
will take something nontrivial to fix.

In the interim, the below patch reverts my changes, and adds a necessary
2nd Requires to pptp.  Harald, please apply.

Phil



[-- Attachment #2: patch-requiresfix --]
[-- Type: text/plain, Size: 957 bytes --]

diff -ruN pom-orig/Netfilter_POM.pm pom-new/Netfilter_POM.pm
--- pom-orig/Netfilter_POM.pm	2005-05-10 18:15:27.177796688 -0400
+++ pom-new/Netfilter_POM.pm	2005-05-10 18:16:58.249951616 -0400
@@ -196,10 +196,10 @@
 		# if the requirement refers to the tested patchlet,
 		# project version must fulfil the requirement.
 		# Multiple requirements are ANDed.
-		return 1 if $pver eq $prog 
-			    && $self->version_compare($proj, $op, $ver);
+		return 0 if $pver eq $prog 
+			    && !$self->version_compare($proj, $op, $ver);
 	}
-	return 0;
+	return 1;
 }
 
 # are the requirements for a specific patchlet fulfilled?
diff -ruN pom-orig/pptp-conntrack-nat/info pom-new/pptp-conntrack-nat/info
--- pom-orig/pptp-conntrack-nat/info	2005-05-10 20:35:15.584820688 -0400
+++ pom-new/pptp-conntrack-nat/info	2005-05-10 20:34:12.225452776 -0400
@@ -4,3 +4,4 @@
 Repository: extra
 Depends: !nat-reservations
 Requires: linux < 2.6.11
+Requires: linux-2.6 < 2.6.11

  reply	other threads:[~2005-05-11  0:51 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-05-10 11:15 3896 pom-ng broken Amin Azez
2005-05-10 15:43 ` Phil Oester
2005-05-10 16:05   ` Amin Azez
     [not found]     ` <debc30fc0505101214699d85b8@mail.gmail.com>
2005-05-10 19:15       ` Jérôme Poulin
2005-05-11  0:51         ` Phil Oester [this message]
2005-05-17 15:05           ` 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=20050511005121.GA31199@linuxace.com \
    --to=kernel@linuxace.com \
    --cc=jeromepoulin@gmail.com \
    --cc=netfilter-devel@lists.netfilter.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.