From: Jon TURNEY <jon.turney@dronecode.org.uk>
Date: Sun, 29 Aug 2010 10:08:34 +0000 (+0000)
Subject: 2010-08-29  Jon TURNEY <jon.turney@dronecode.org.uk>
X-Git-Tag: release_2.869~153
X-Git-Url: https://cygwin.com/git/?a=commitdiff_plain;h=41d8d0f4b60bcc092fc9f1366ff2d50667c09df6;p=cygwin-apps%2Fsetup.git

2010-08-29  Jon TURNEY <jon.turney@dronecode.org.uk>

	* postinstall.cc (do_postinstall_thread): Don't run postinstall
	scripts belonging to a just installed package twice.
---

diff --git a/postinstall.cc b/postinstall.cc
index aaa9b20b..21fc1295 100644
--- a/postinstall.cc
+++ b/postinstall.cc
@@ -161,13 +161,29 @@ do_postinstall_thread (HINSTANCE h, HWND owner)
       Progress.SetBar2 (k, numpkg);
     }
 
-  // Look for any scripts in /etc/postinstall which haven't been renamed .done,
-  // and try to run them...
+  // Look for any scripts in /etc/postinstall which haven't been renamed .done
   std::string postinst = cygpath ("/etc/postinstall");
   vector<Script> scripts;
   RunFindVisitor myVisitor (&scripts);
   Find (postinst).accept (myVisitor);
 
+  // Remove anything which we just tried to run (so we don't try twice)
+  for (i = packages.begin (); i != packages.end (); ++i)
+    {
+       packagemeta & pkg = **i;
+       for (std::vector<Script>::const_iterator j = pkg.installed.scripts().begin();
+            j != pkg.installed.scripts().end();
+            j++)
+         {
+           std::vector<Script>::iterator p = find(scripts.begin(), scripts.end(), *j);
+           if (p != scripts.end())
+             {
+               scripts.erase(p);
+             }
+         }
+    }
+
+  // and try to run what's left...
   {
     RunScript scriptRunner("No package", scripts);
     scriptRunner.run_all(s);