tags 435577 + patch
thanks

gdebi actually has code to support Pre-Depends (because of how it installs 
dependencies before trying to install the target package, installing them 
along with depends works fine).

However, for some reason gdebi reads a field called PreDepends, not 
Pre-Depends (perhaps due to confusion related to apt internally having 
something called PreDepends).  Pre-Depends is the thing that appears in 
control files and thus the thing that gdebi should read.

Patch attached.

        -Tim Abbott
From fe3f6249a8658cdccdf0d565e6b7f0df73a5c967 Mon Sep 17 00:00:00 2001
From: Tim Abbott <tabb...@ksplice.com>
Date: Thu, 25 Jun 2009 15:50:24 -0400
Subject: [PATCH] Use the correct name for the Pre-Depends field in control files.

Signed-off-by: Tim Abbott <tabb...@ksplice.com>
---
 GDebi/DebPackage.py |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/GDebi/DebPackage.py b/GDebi/DebPackage.py
index 161edf0..c6f94a8 100755
--- a/GDebi/DebPackage.py
+++ b/GDebi/DebPackage.py
@@ -183,7 +183,7 @@ class DebPackage(object):
     def getDepends(self):
         depends = []
         # find depends
-        for key in ["Depends","PreDepends"]:
+        for key in ["Depends","Pre-Depends"]:
             if self._sections.has_key(key):
                 depends.extend(apt_pkg.ParseDepends(self._sections[key]))
         return depends
-- 
1.6.3.1

Reply via email to