From a8ca8e6ee8c2fbd5a7bd15e54ec794c2ac2a3c3f Mon Sep 17 00:00:00 2001
From: doktornotor <notordoktor@gmail.com>
Date: Mon, 16 Nov 2015 22:13:21 +0100
Subject: Package install/deinstall cleanup

---
 config/cron/cron.inc | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/config/cron/cron.inc b/config/cron/cron.inc
index 645575d9..87591e08 100644
--- a/config/cron/cron.inc
+++ b/config/cron/cron.inc
@@ -27,22 +27,30 @@
 	ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
 	POSSIBILITY OF SUCH DAMAGE.
 */
+require_once("pfsense-utils.inc");
 require_once("services.inc");
+require_once("util.inc");
 
 function cron_sync_package() {
 	configure_cron();
 	// Previous package versions were "helpfully" killing cron on uninstall.
 	// Also, need to make sure cron is running, otherwise the package is useless.
-	// TODO: Something like this needs to be eventually done in configure_cron() in services.inc.
-	if (!is_process_running("cron")) {
-		exec("cd /tmp && /usr/sbin/cron -s 2>/dev/null");
+	// configure_cron() function in services.inc already does this check on pfSense >=2.2.5
+	$pfs_version = str_replace(".", "", substr(trim(file_get_contents("/etc/version")), 0, 5));
+	if ($pfs_version < 225) {
+		if (!is_process_running("cron")) {
+			exec("cd /tmp && /usr/sbin/cron -s 2>/dev/null");
+		}
 	}
 }
 
 function cron_install_command() {
 	// Clean up possible lingering garbage after previous package versions
 	unlink_if_exists("/usr/local/etc/rc.d/cron.sh");
-	cron_sync_package();
+}
+
+function cron_deinstall_command() {
+	rmdir_recursive("/usr/local/www/packages/cron");
 }
 
 ?>
-- 
cgit v1.2.3


From 55e442814bdd12d1de4f2dc636190ba564c6419f Mon Sep 17 00:00:00 2001
From: doktornotor <notordoktor@gmail.com>
Date: Mon, 16 Nov 2015 22:15:45 +0100
Subject: Package install/deinstall cleanup, nuke useless tags

---
 config/cron/cron.xml | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/config/cron/cron.xml b/config/cron/cron.xml
index f777faff..181a4506 100644
--- a/config/cron/cron.xml
+++ b/config/cron/cron.xml
@@ -41,19 +41,16 @@
 /* ====================================================================================== */
 	]]>
 	</copyright>
-	<description>Cron</description>
-	<name>Cron Settings</name>
-	<version>0.3.1</version>
-	<title>Settings</title>
+	<name>cronsettings</name>
+	<version>0.3.3</version>
+	<title>Cron Settings</title>
 	<include_file>/usr/local/pkg/cron.inc</include_file>
 	<menu>
 		<name>Cron</name>
-		<tooltiptext>Cron settings.</tooltiptext>
 		<section>Services</section>
 		<configfile>cron.xml</configfile>
 		<url>/packages/cron/cron.php</url>
 	</menu>
-	<configpath>installedpackages->package->$packagename->configuration->cron</configpath>
 	<additional_files_needed>
 		<prefix>/usr/local/pkg/</prefix>
 		<item>https://packages.pfsense.org/packages/config/cron/cron.xml</item>
@@ -87,4 +84,7 @@
 	<custom_php_install_command>
 		cron_install_command();
 	</custom_php_install_command>
+	<custom_php_deinstall_command>
+		cron_deinstall_command();
+	</custom_php_deinstall_command>
 </packagegui>
-- 
cgit v1.2.3


From af7b70bf79f7ea4406051708af9455886713c0d0 Mon Sep 17 00:00:00 2001
From: doktornotor <notordoktor@gmail.com>
Date: Mon, 16 Nov 2015 22:16:58 +0100
Subject: Bump cron package version

---
 pkg_config.10.xml | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/pkg_config.10.xml b/pkg_config.10.xml
index 78b56b5b..7472cc36 100644
--- a/pkg_config.10.xml
+++ b/pkg_config.10.xml
@@ -369,10 +369,9 @@
 		<descr>The cron utility is used to manage commands on a schedule.</descr>
 		<category>System</category>
 		<config_file>https://packages.pfsense.org/packages/config/cron/cron.xml</config_file>
-		<version>0.3.2</version>
-		<status>BETA</status>
+		<version>0.3.3</version>
+		<status>RC</status>
 		<required_version>2.2</required_version>
-		<maintainer>markjcrane@gmail.com</maintainer>
 		<configurationfile>cron.xml</configurationfile>
 	</package>
 	<package>
-- 
cgit v1.2.3


From 6b3441095e61dd99fd76a13e530db91454813f3c Mon Sep 17 00:00:00 2001
From: doktornotor <notordoktor@gmail.com>
Date: Mon, 16 Nov 2015 22:17:45 +0100
Subject: Bump cron package version

---
 pkg_config.8.xml | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/pkg_config.8.xml b/pkg_config.8.xml
index 520fb3b0..75082762 100644
--- a/pkg_config.8.xml
+++ b/pkg_config.8.xml
@@ -436,10 +436,9 @@
 		<pkginfolink></pkginfolink>
 		<config_file>https://packages.pfsense.org/packages/config/cron/cron.xml</config_file>
 		<depends_on_package_base_url>https://files.pfsense.org/packages/8/All/</depends_on_package_base_url>
-		<version>0.3.2</version>
-		<status>Beta</status>
-		<required_version>1.2</required_version>
-		<maintainer>markjcrane@gmail.com</maintainer>
+		<version>0.3.3</version>
+		<status>RC</status>
+		<required_version>2.1</required_version>
 		<configurationfile>cron.xml</configurationfile>
 	</package>
 	<package>
-- 
cgit v1.2.3


From 861f997ed50d0c0f2e81f61a36c5c3a5d2b7277e Mon Sep 17 00:00:00 2001
From: doktornotor <notordoktor@gmail.com>
Date: Mon, 16 Nov 2015 22:18:44 +0100
Subject: Bump cron package version

---
 pkg_config.8.xml.amd64 | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/pkg_config.8.xml.amd64 b/pkg_config.8.xml.amd64
index a452835b..18dc6d01 100644
--- a/pkg_config.8.xml.amd64
+++ b/pkg_config.8.xml.amd64
@@ -423,10 +423,9 @@
 		<pkginfolink></pkginfolink>
 		<config_file>https://packages.pfsense.org/packages/config/cron/cron.xml</config_file>
 		<depends_on_package_base_url>https://files.pfsense.org/packages/amd64/8/All/</depends_on_package_base_url>
-		<version>0.3.2</version>
-		<status>Beta</status>
-		<required_version>1.2</required_version>
-		<maintainer>markjcrane@gmail.com</maintainer>
+		<version>0.3.3</version>
+		<status>RC</status>
+		<required_version>2.1</required_version>
 		<configurationfile>cron.xml</configurationfile>
 	</package>
 	<package>
-- 
cgit v1.2.3