public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
From: Jon Turney <jon.turney@dronecode.org.uk>
To: cygwin-apps@cygwin.com
Cc: Jon Turney <jon.turney@dronecode.org.uk>
Subject: [PATCH setup 02/15] Don't bother storing prev version
Date: Tue, 16 May 2017 11:49:00 -0000	[thread overview]
Message-ID: <20170516114900.168120-3-jon.turney@dronecode.org.uk> (raw)
In-Reply-To: <20170516114900.168120-1-jon.turney@dronecode.org.uk>

There's no means to select it since ec9c1d70, so don't bother remembering
which version is the prev version.
---
 IniDBBuilderPackage.cc |  4 ----
 package_meta.cc        | 10 ++--------
 package_meta.h         |  6 ++----
 prereq.cc              |  2 +-
 4 files changed, 5 insertions(+), 17 deletions(-)

diff --git a/IniDBBuilderPackage.cc b/IniDBBuilderPackage.cc
index d506a9f..303ac34 100644
--- a/IniDBBuilderPackage.cc
+++ b/IniDBBuilderPackage.cc
@@ -139,7 +139,6 @@ IniDBBuilderPackage::buildPackageSource (const std::string& path,
       csp->versions.clear();
       csp->desired = packageversion();
       csp->installed = packageversion();
-      csp->prev = packageversion();
       csp->curr = packageversion();
       csp->exp = packageversion();
       db.sourcePackages.insert (packagedb::packagecollection::value_type(csp->name,csp));
@@ -424,9 +423,6 @@ IniDBBuilderPackage::add_correct_version()
     case TRUST_CURR:
       v = &(cp->curr);
     break;
-    case TRUST_PREV:
-      v = &(cp->prev);
-    break;
     case TRUST_TEST:
       v = &(cp->exp);
     break;
diff --git a/package_meta.cc b/package_meta.cc
index 24ed17e..cffb5b7 100644
--- a/package_meta.cc
+++ b/package_meta.cc
@@ -89,7 +89,7 @@ packagemeta::_actions::caption ()
 packagemeta::packagemeta (packagemeta const &rhs) :
   name (rhs.name), key (rhs.name),
   categories (rhs.categories), versions (rhs.versions),
-  installed (rhs.installed), prev (rhs.prev),
+  installed (rhs.installed),
   curr (rhs.curr),
   exp (rhs.exp),
   desired (rhs.desired)
@@ -593,8 +593,6 @@ packagemeta::logAllVersions () const
 std::string 
 packagemeta::trustLabel(packageversion const &aVersion) const
 {
-    if (aVersion == prev)
-	return "Prev";
     if (aVersion == curr)
 	return "Curr";
     if (aVersion == exp)
@@ -606,8 +604,7 @@ void
 packagemeta::logSelectionStatus() const
 {
   packagemeta const & pkg = *this;
-  const char *trust = ((pkg.desired == pkg.prev) ? "prev"
-               : (pkg.desired == pkg.curr) ? "curr"
+  const char *trust = ((pkg.desired == pkg.curr) ? "curr"
                : (pkg.desired == pkg.exp) ? "test" : "unknown");
   std::string action = pkg.action_caption ();
   const std::string installed =
@@ -648,7 +645,6 @@ packagemeta::ScanDownloadedFiles (bool mirror_mode)
 	  /* scan doesn't alter operator == for packageversions */
 	  bool lazy_scan = mirror_mode
 			   && (*i != pkg.installed
-			       || pkg.installed == pkg.prev
 			       || pkg.installed == pkg.curr
 			       || pkg.installed == pkg.exp);
 	  const_cast<packageversion &>(*i).scan (lazy_scan);
@@ -662,8 +658,6 @@ packagemeta::ScanDownloadedFiles (bool mirror_mode)
 	  if (!i->accessible () && !pkgsrcver.accessible ()
 	      && *i != pkg.installed)
 	    {
-	      if (pkg.prev == *i)
-		pkg.prev = packageversion ();
 	      if (pkg.curr == *i)
 		pkg.curr = packageversion ();
 	      if (pkg.exp == *i)
diff --git a/package_meta.h b/package_meta.h
index 5f7842c..5c5e718 100644
--- a/package_meta.h
+++ b/package_meta.h
@@ -126,11 +126,9 @@ public:
   bool user_picked;
   /* which one is installed. */
   packageversion installed;
-  /* which one is listed as "prev" in our available packages db */
-  packageversion prev;
-  /* ditto for current - stable */
+  /* which one is listed as "current" (stable) in our available packages db */
   packageversion curr;
-  /* and finally the experimental version */
+  /* ditto for "test" (experimental) */
   packageversion exp;
   /* Now for the user stuff :] */
   /* What version does the user want ? */
diff --git a/prereq.cc b/prereq.cc
index 4b6cd68..5a94373 100644
--- a/prereq.cc
+++ b/prereq.cc
@@ -210,7 +210,7 @@ PrereqChecker::isMet ()
       Progress.SetBar1(pos, max);
 
       // Fetch the dependencies of the package. This assumes that the
-      // dependencies of the prev, curr, and exp versions are all the same.
+      // dependencies of all versions are all the same.
       const vector <vector <PackageSpecification *> *> *deps = pack->curr.depends ();
 
       // go through the package's dependencies
-- 
2.12.3

  parent reply	other threads:[~2017-05-16 11:49 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-16 11:49 [PATCH setup 00/15] Various setup patches Jon Turney
2017-05-16 11:49 ` [PATCH setup 06/15] Remove unneeded virtual from class packagesource's methods Jon Turney
2017-05-16 11:49 ` [PATCH setup 05/15] Remove unused forward declaration of non-existent class category Jon Turney
2017-05-16 11:49 ` [PATCH setup 03/15] Rename "Internet Explorer Proxy Settings" to "System Proxy Settings" Jon Turney
2017-05-16 11:49 ` [PATCH setup 01/15] Add Makefile rule to rename build products to form used when uploading Jon Turney
2017-05-16 11:49 ` Jon Turney [this message]
2017-05-16 11:49 ` [PATCH setup 04/15] Remove packageversion::sources(), only packageversion::source() is useful Jon Turney
2017-05-16 11:50 ` [PATCH setup 07/15] Simplify class packagesource Jon Turney
2017-05-16 11:51 ` [PATCH setup 08/15] Fix comments and indentation in check_for_cached Jon Turney
2017-05-23 17:34   ` Achim Gratz
2017-05-16 11:51 ` [PATCH setup 09/15] Initial setting of trust should be TRUST_CURR Jon Turney
2017-05-16 11:58 ` [PATCH setup 10/15] Change PackageSpecification::_operator to an enum Jon Turney
2017-05-16 11:58   ` [PATCH setup 12/15] Stop pretending to support complex dependencies Jon Turney
2017-05-16 11:58   ` [PATCH setup 11/15] Remove useless PackageSpecification methods Jon Turney
2017-05-16 11:59   ` [PATCH setup 13/15] All, rather than just the first (usually current) version should get dependencies Jon Turney
2017-05-16 12:00   ` [PATCH setup 14/15] Make PackageDepends a type Jon Turney
2017-05-16 12:04   ` [PATCH setup 15/15] Add some progress reporting during preremove and uninstall Jon Turney

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=20170516114900.168120-3-jon.turney@dronecode.org.uk \
    --to=jon.turney@dronecode.org.uk \
    --cc=cygwin-apps@cygwin.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).