[Piuparts-commits] rev 82 - in trunk: . debian

[Ana Beatriz Guerrero López]

Author: ana
Date: 2007-08-31 14:29:08 +0000 (Fri, 31 Aug 2007)
New Revision: 82

Modified:
   trunk/debian/changelog
   trunk/piuparts.py
Log:
In the cron test, not executable files are not run. (Closes: #440141)



Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog	2007-08-29 12:17:03 UTC (rev 81)
+++ trunk/debian/changelog	2007-08-31 14:29:08 UTC (rev 82)
@@ -1,8 +1,9 @@
 piuparts (0.26) unstable; urgency=low
 
   * Update list of ignored files. (Closes: #439592)
+  * In the cron test, not executable files are not run. (Closes: #440141)
 
- -- Ana Beatriz Guerrero Lopez <ana at debian.org>  Tue, 28 Aug 2007 22:30:43 +0200
+ -- Ana Beatriz Guerrero Lopez <ana at debian.org>  Fri, 31 Aug 2007 15:44:36 +0200
 
 piuparts (0.25) unstable; urgency=low
 

Modified: trunk/piuparts.py
===================================================================
--- trunk/piuparts.py	2007-08-29 12:17:03 UTC (rev 81)
+++ trunk/piuparts.py	2007-08-31 14:29:08 UTC (rev 82)
@@ -795,10 +795,13 @@
                 pathname = line.strip()
                 if pathname.startswith("/etc/cron."):
                     if os.path.isfile(self.relative(pathname.strip("/"))):
-                        if not has_cronfiles:
-                            has_cronfiles = True
-                        list.append(pathname)
-                        logging.info("Package " + p + " contains cron file: " + pathname)
+                        st = os.lstat(self.relative(pathname.strip("/")))
+                        mode = st[stat.ST_MODE]
+                        if (mode & stat.S_IEXEC): 
+                            if not has_cronfiles:
+                                has_cronfiles = True
+                            list.append(pathname)
+                            logging.info("Package " + p + " contains cron file: " + pathname)
             f.close()
 
         return has_cronfiles, list