* apt-pkg/deb/debversion.cc:

  - applied patch from Lionel Elie Mamane to fix epoch handling
    (debian: #363358)

diff --git a/apt-pkg/deb/debversion.cc b/apt-pkg/deb/debversion.cc
index aeee619296d19ee0df96eae0fd6b2b8d448ff077..064d8fa5be69371595bceba4058964e6adf36046 100644 (file)
--- a/apt-pkg/deb/debversion.cc
+++ b/apt-pkg/deb/debversion.cc
@@ -59,7 +59,7 @@ int debVersioningSystem::CmpFragment(const char *A,const char *AEnd,
    }
 
    /* Iterate over the whole string
    }
 
    /* Iterate over the whole string

-      What this does is to spilt the whole string into groups of
+      What this does is to split the whole string into groups of

       numeric and non numeric portions. For instance:
          a67bhgs89
       Has 4 portions 'a', '67', 'bhgs', '89'. A more normal:
       numeric and non numeric portions. For instance:
          a67bhgs89
       Has 4 portions 'a', '67', 'bhgs', '89'. A more normal:


@@ -140,6 +140,27 @@ int debVersioningSystem::DoCmpVersion(const char *A,const char *AEnd,
    if (rhs == BEnd)
       rhs = B;
    
    if (rhs == BEnd)
       rhs = B;
    

+   // Special case: a zero epoch is the same as no epoch,
+   // so remove it.
+   if (lhs != A)
+   {
+      for (; *A == '0'; ++A);
+      if (A == lhs)
+      {
+        ++A;
+        ++lhs;
+      }
+   }
+   if (rhs != B)
+   {
+      for (; *B == '0'; ++B);
+      if (B == rhs)
+      {
+        ++B;
+        ++rhs;
+      }
+   }
+

    // Compare the epoch
    int Res = CmpFragment(A,lhs,B,rhs);
    if (Res != 0)
    // Compare the epoch
    int Res = CmpFragment(A,lhs,B,rhs);
    if (Res != 0)

diff --git a/debian/changelog b/debian/changelog
index 0347020909baab50a12ff0f71613861d308c1716..1412fe74e9c377e1c257ad711241bb451adfe4eb 100644 (file)
--- a/debian/changelog
+++ b/debian/changelog
@@ -22,8 +22,12 @@ apt (0.6.44) unstable; urgency=low
   * methods/ftp.cc:
     - unlink empty file in partial if the download failed because
       the file is missing on the server (closes: #316337)
   * methods/ftp.cc:
     - unlink empty file in partial if the download failed because
       the file is missing on the server (closes: #316337)

+  * apt-pkg/deb/debversion.cc:
+    - treats a version string with explicit zero epoch equal
+      than the same without epoch (Policy 5.6.12, closes: #363358)
+      Thanks to Lionel Elie Mamane for the patch

   
   

- --
+ -- Michael Vogt <michael.vogt@ubuntu.com>  Mon,  8 May 2006 22:04:43 +0200

 
 apt (0.6.43.3) unstable; urgency=low
 
 
 apt (0.6.43.3) unstable; urgency=low
 

diff --git a/test/testdeb.cc b/test/testdeb.cc
index 5986621bb46a092e21e977fd6e013e558d04ec66..d28f20114fa1516170a352218ca2c74e31ddbe96 100644 (file)
--- a/test/testdeb.cc
+++ b/test/testdeb.cc
@@ -23,7 +23,7 @@ bool Test(const char *File)
       return false;
       
    // Extract it.
       return false;
       
    // Extract it.

-   ExtractTar Tar(Deb.GetFile(),Member->Size);
+   ExtractTar Tar(Deb.GetFile(),Member->Size, "gzip");

    NullStream Dir;
    if (Tar.Go(Dir) == false)
       return false;   
    NullStream Dir;
    if (Tar.Go(Dir) == false)
       return false;   

diff --git a/test/versions.lst b/test/versions.lst
index 008a0f2d797ff23422e3a9344978f8986860ad2a..efc19c4f0e15b4f76581579c2d2a4151cebae90b 100644 (file)
--- a/test/versions.lst
+++ b/test/versions.lst
@@ -20,6 +20,13 @@ z . -1
 # Epochs
 1:0.4 10.3 1
 1:1.25-4 1:1.25-8 -1
 # Epochs
 1:0.4 10.3 1
 1:1.25-4 1:1.25-8 -1

+0:1.18.36 1.18.36 0
+
+# Funky, but allowed, characters in upstream version
+9:1.18.36:5.4-20 10:0.5.1-22 -1
+9:1.18.36:5.4-20 9:1.18.36:5.5-1 -1
+9:1.18.36:5.4-20 9:1.18.37:4.3-22 -1
+1.18.36-0.17.35-18 1.18.36-19 1

 
 # Junk
 1:1.2.13-3 1:1.2.13-3.1 -1
 
 # Junk
 1:1.2.13-3 1:1.2.13-3.1 -1