merged from david

[apt.git] / apt-pkg / versionmatch.cc

diff --git a/apt-pkg/versionmatch.cc b/apt-pkg/versionmatch.cc
index 093180f9b78e2f9d9397eed1e39b3e191fa52626..e4fa0ea65f10fd2be0a3d4c737db3e5eefc89673 100644 (file)
--- a/apt-pkg/versionmatch.cc
+++ b/apt-pkg/versionmatch.cc
@@ -11,8 +11,9 @@
    ##################################################################### */
                                                                        /*}}}*/
 // Include Files                                                       /*{{{*/
    ##################################################################### */
                                                                        /*}}}*/
 // Include Files                                                       /*{{{*/

-#include <apt-pkg/versionmatch.h>
+#include<config.h>

 
 

+#include <apt-pkg/versionmatch.h>

 #include <apt-pkg/strutl.h>
 #include <apt-pkg/error.h>
 
 #include <apt-pkg/strutl.h>
 #include <apt-pkg/error.h>
 


@@ -21,9 +22,10 @@
 #include <fnmatch.h>
 #include <sys/types.h>
 #include <regex.h>
 #include <fnmatch.h>
 #include <sys/types.h>
 #include <regex.h>

-

                                                                        /*}}}*/
 
                                                                        /*}}}*/
 

+using std::string;
+

 // VersionMatch::pkgVersionMatch - Constructor                         /*{{{*/
 // ---------------------------------------------------------------------
 /* Break up the data string according to the selected type */
 // VersionMatch::pkgVersionMatch - Constructor                         /*{{{*/
 // ---------------------------------------------------------------------
 /* Break up the data string according to the selected type */


@@ -60,7 +62,7 @@ pkgVersionMatch::pkgVersionMatch(string Data,MatchType Type) : Type(Type)
       
       // Are we a simple specification?
       string::const_iterator I = Data.begin();
       
       // Are we a simple specification?
       string::const_iterator I = Data.begin();

-      for (; I != Data.end() && *I != '='; I++);
+      for (; I != Data.end() && *I != '='; ++I);

       if (I == Data.end())
       {
         // Temporary
       if (I == Data.end())
       {
         // Temporary


@@ -118,7 +120,10 @@ pkgVersionMatch::pkgVersionMatch(string Data,MatchType Type) : Type(Type)
    
    if (Type == Origin)
    {
    
    if (Type == Origin)
    {

-      OrSite = Data;
+      if (Data[0] == '"' && Data.length() >= 2 && Data.end()[-1] == '"')
+        OrSite = Data.substr(1, Data.length() - 2);
+      else
+        OrSite = Data;

       return;
    }   
 }
       return;
    }   
 }


@@ -149,7 +154,7 @@ bool pkgVersionMatch::MatchVer(const char *A,string B,bool Prefix)
 pkgCache::VerIterator pkgVersionMatch::Find(pkgCache::PkgIterator Pkg)
 {
    pkgCache::VerIterator Ver = Pkg.VersionList();
 pkgCache::VerIterator pkgVersionMatch::Find(pkgCache::PkgIterator Pkg)
 {
    pkgCache::VerIterator Ver = Pkg.VersionList();

-   for (; Ver.end() == false; Ver++)
+   for (; Ver.end() == false; ++Ver)

    {
       if (Type == Version)
       {
    {
       if (Type == Version)
       {


@@ -160,7 +165,7 @@ pkgCache::VerIterator pkgVersionMatch::Find(pkgCache::PkgIterator Pkg)
         continue;
       }
       
         continue;
       }
       

-      for (pkgCache::VerFileIterator VF = Ver.FileList(); VF.end() == false; VF++)
+      for (pkgCache::VerFileIterator VF = Ver.FileList(); VF.end() == false; ++VF)

         if (FileMatch(VF.File()) == true)
            return Ver;
    }
         if (FileMatch(VF.File()) == true)
            return Ver;
    }


@@ -259,10 +264,10 @@ bool pkgVersionMatch::FileMatch(pkgCache::PkgFileIterator File)
    if (Type == Origin)
    {
       if (OrSite.empty() == false) {
    if (Type == Origin)
    {
       if (OrSite.empty() == false) {

-        if (File->Site == 0 || !ExpressionMatches(OrSite, File.Site()))
+        if (File->Site == 0)

            return false;
       } else // so we are talking about file:// or status file
            return false;
       } else // so we are talking about file:// or status file

-        if (strcmp(File.Site(),"") == 0 && File->Archive != 0) // skip the status file
+        if (strcmp(File.Site(),"") == 0 && File->Archive != 0 && strcmp(File.Archive(),"now") == 0) // skip the status file

            return false;
       return (ExpressionMatches(OrSite, File.Site())); /* both strings match */
    }
            return false;
       return (ExpressionMatches(OrSite, File.Site())); /* both strings match */
    }