lp:debian/squeeze/cron

Created by James Westby and last modified
Get this branch:
bzr branch lp:debian/squeeze/cron
Members of Ubuntu branches can upload to this branch. Log in for directions.

Related bugs

Related blueprints

Branch information

Owner:
Ubuntu branches
Status:
Development

Recent revisions

15. By Christian Kastner

* Upload with approval from Release Team to get RC bug fixes in Squeeze
  (see http://lists.debian.org/debian-release/2010/12/msg00719.html)
* do_command.c, popen.c:
  - Use fork() instead of vfork().
* do_command.c:
  - Close an unused stream in the fork()ed child prior to exec'ing the
    user's command, thereby avoiding an fd leak. Closes: #604181, LP: #665912
    Previously to this, in conjunction with LVM, the fd leak may have the
    effect of the user being spammed by warnings every time a cron job was
    executed.
* crontab.5:
  - Fixed the example demonstrating how to run a job on a certain weekday of
    the month (date range was off-by-one). Also, the same example contained
    a superfluous escape, resulting in wrong output. Closes: #606325
* cron.init:
  - Added $named to Should-Start, in case @reboot jobs need DNS resolution.
    Closes: #602903
  - Added nslcd to Should-Start. LP: #27520

14. By Javier Fernández-Sanguino

* cron.c: set LC_COLLATE to 'C' in order to properly work when locales
  are set that change the collation order expected by the regular
  expressions used in load_database(). By setting locale we were
  skipping some crontabs unexpectedly due to [a-z] not matching all the ascii
  characters in some languages (such as Estonian). (Closes: #600310)
  Thanks to Michael Tokarev for spotting this issue and for
  Bastian Blank for pointing for the fix.
* debian/cron.init:
    - Fix typo that prevented it from properly loading LC_CTYPE (Closes:
      #600082)
    - Force LC_COLLATE to 'C' regardless of user locale settings.

13. By Christian Kastner

* Cleanup of the package in preparation of the next Debian release.
   - Notice that only one change introduces new behaviour (debian/cron.pam),
     the other changes are just documentation fixes or cleanups to the
     packaging.
* debian/cron.pam:
  - In addition to the reading /etc/security/pam_env.conf and
    /etc/environment (obsolete config file) previous default,
    cron will now also read /etc/default/locale in order to setup
    the proper localisation environment for tasks.
[ Documentation fixes ]
* debian/cron.{default,init}, cron.8, crontab.5:
  - Updated documentation (comments in scripts and manpageS) regarding the
    reading and setting of locale information for the cron daemon and for
    cron tasks to document the new behaviour due to the above change.
* debian/{preinst,postinst,postrm}:
  - Since /etc/cron.monthly/standard does not provide any worthwhile task
    anymore, remove obsolete /etc/cron.monthly/standard after upgrading
    (local changes are preserved, however). Closes: #585680
* crontab.1
  - Minor typo fixes to the manpage.
* crontab.5:
  - Added an example submitted by jidanni (thanks!), showing how to run a
    command on the second Saturday of the month. Closes: #584514
  - Minor fixes
[ Fixes to the packaging ]
* debian/control:
  - Bumped Standards-Version to 3.9.1 (no changes needed)
  - Added Pre-Depends for dpkg (<= 1.15.7.2) for a dpkg-maintscript-helper
    with support for safely removing conffiles
  - Added Homepage: field
  - Added Vcs-* fields
  - Upgraded debhelper dependency version for dh_bugscripts
* debian/rules:
  - Replace deprecated dh_clean -k with dh_prep
* debian/NEWS:
  - Apply Steve Langasek's patch to remove non-newsworthy content, avoiding
    unnecessary output from apt-listchanges during upgrade. Closes: #591005
* debian/source/format:
  - Make source format explicit (1.0)

12. By Christian Kastner

[ Christian Kastner / Javier Fernandez-Sanguino ]
* debian/postinst:
  - Now that permissions and ownership of crontabs are changed unconditionally,
    do not attempt to chown user crontabs if none are present. Closes: #585636
  - Only change permissions if the crontabs directory exist

11. By Christian Kastner

[ Christian Kastner ]
* do_command.c:
  - Don't send mail when a job exits non-zero, only send mail if the job sent
    output to stderr. This behaviour was introduced erroneously; while it
    does have merit, it is completly against standard cron behaviour.
    Closes: #581612
* debian/compat:
  - Bumped debhelper compatibility to 7
* debian/control:
  - Bumped Standards-Version to 3.8.4 (no change needed)
  - Build-Depend on debhelper (>= 7.0.50~)
  - Added dependency on ${misc:Depends} to package cron
* debian/cron.init:
  - Changed Default-Stop from (1) to (empty). rc0 and rc6 were removed in
    3.0pl1-101 because the stop action -- sending SIGTERM/SIGKILL to cron
    on shutdown/reboot -- was redundant. This, however, also applies to
    rc1, because killprocs will do that for us.
* debian/postinst:
  - Removed obsolete dpkg file backup code, this has been handed over to dpkg
    in 3.0pl1-109
  - Removed last remaining stop action (for rc1) from upate-rc.d (see above)
  - Add dpkg-statoverride for /usr/bin/crontab, and unconditionally change
    permissions of /var/spool/cron/crontabs. Closes: #304036, #460095
* debian/standard.monthly:
  - Removed because it had been empty for years and therefore served no
    purpose
* debian/cron.bug-{control,script}
  - Added to extend information submitted by reportbug
* debian/rules:
  - Applied changes for standard.monthly and cron.bug-{control,script} above
* debian/copyright:
  - Updated to reflect recent contributions
* debian/README:
  - Updated ancient information with the current status (PAM support, SELinux
    support, etc.)
* debian/TODO:
  - Added new entry regarding clarification of the purpose behind /etc/cron.d
* debian/REFS:
  - Summary of links to relevant development threads; re-added for
    informational purposes.
[ Javier Fernandez-Sanguino ]
* debian/cron.init:
  - Fix typo (Closes: #585399)

10. By Javier Fernández-Sanguino

[ Christian Kastner ]
* debian/cron.init:
  - When both /etc/environment and /etc/default/locale contain locale
  information, warn that variables from /etc/environment will be ignored.
  Closes: #543895, #580942
  - Add autofs and NIS+ to Should-Starts, because especially NIS+ must be run
  before cron, if present. Closes: #511757
  - init script should be stopped in runlevel 0 and 6, too
[ Javier Fernandez-Sanguino ]
* do_command.c: Revert change so that mails are not sent if MAILTO
  is null. Closes: #580938

9. By Javier Fernández-Sanguino

* cron.c:
   - Apply patch derived from OpenBSD's cron by Petya Kohts to
     handle properly DST and avoid running cron jobs twice, or
     skip cron jobs when there is a savings time change.
     Closes: #217836, #458123, #474157, LP: #36690
* crontab.c:
   - Revert the behaviour that prevented 'alias crontab="crontab -i"' from
     working. Now the -i flag is allowed regardless of other command line
     switches. (Closes: #513379)
   - Be more verbose in the example provided for first-time users than run
     'crontab -e'
   - Check if the crontab exists before attempting its removal and before
     prompting the user (whe using -i)
   - When prompting, tell the user which chars are expected
* crontab.1:
   - Change usage to match crontab's output
   - Clarify use of the cron.allow and cron.deny files (Closes: #511782)
* crontab.5:
   - Describe a known limitation related to the lack of the cron daemon's
     per-user timezone support (Closes: #497741, 353246, 166533)
   - Describe the use of tilde (~) as a substitute for $HOME (Closes: 477198)
   - Describe the lack of replacements of variables (Closes: #493636)
* popen.c: Check return call of initgroups(), associated with
   CVE-2006-2607 fix. Thanks to Christian Kastner for the heads up.
* cron.8:
   - Describe how the environment is managed in Debian
* debian/cron.init: Also use /etc/timezone to setup the timezone environment
  (TZ) if not set.
* debian/standard.daily:
   - Do not make a backup copy of dpkg files anymore since dpkg (1.15.4) does
     this already (when #541412 was fixed) (Closes: #541415)
   - Fix a bug in lost+found reporting. Files in lost+found were never +
     actually reported. [ Patch from Teemu Kiviniemi ]
   - Added ext4 for lost+found searches. [ Patch from Teemu Kiviniemi ]
   Closes LP: #367383
* debian/control:
    - Add a Breaks dpkg (<< 1.15.4) to ensure that one always has the backup
      functionality either via cron or via dpkg.
    - Provides: cron-daemon, which was reinstated in the virtual package
      list in November 2009 (Closes: #349170)

8. By Javier Fernández-Sanguino

Add a Depends: on libpam-runtime (>= 1.0.1-11) since this
is required for the use of common-session-noninteractive
(Closes: 575342)

7. By Javier Fernández-Sanguino

* debian/cron.pam: Use common-session-noninteractive in pam.d configuration
  instead of common-session so that libpam-ssh is not run. If libpam-ssh was
  installed the previous configuration resulted in cron spawning
  many ssh-agent processes (Closes: #572292)
* debian/cron.init: Add Should-Start/Stop dependency to slapd so that cron
* can start up tasks for users define in an LDAP (Closes: #557659, #546367)
* Fix the use of perror so that the error message is correct when
  printing the reason why an operation failed. Thanks to
  Justin Pryzby for the patch. (Closes: 470587)
* pathnames.h: Use /usr/bin/sensible-editor instead of /usr/bin/editor when
  calling crontab -e (Closes: #482284)
* Fix debian/control description, thanks to Justin B Rye for the patch
  (Closes: #535227)
* crontab.c: Produce a different warning message when root tries to setup a
  crontab for a user that is not allowed to do so, and do not log the
  action (Closes: #505288)
* Only allow -i if -r has been specified previously (Closes: #513379)
* Update debian/README.Debian to indicate that the package is
  being maintained at Alioth, thanks Tomas Pospisek for the patch
  (Closes: #511740)
* crontab.c: Update the usage() call so that it refers to the use
  of -i when removing (-r)
* crontab.1: Update the manpage:
   - Use the same option listing as in the crontab usage() call
   - Remove 'tweaked', explain what -u does in each context.
   - Remove V3, use 'Vixie Cron' instead
  (Closes: #572249)
   - Update the manpage years (Closes: #563296)
   - Describe the usage of the crontab group wrt /var/spool/cron/crontabs
  (Closes: #539182)
* Use debhelper compatibility version 5
* Include cross build support with patch to debian/rules provided by Neil
  Williams (Closes: #465077)
* Change maintainer's email address

6. By Javier Fernández-Sanguino

 * SECURITY UPDATE: cron does not check the return code of setgid() and
 initgroups(), which under certain circumstances could cause
 applications to run with elevated group privileges. Note that the more
 serious issue of not checking the return code of setuid() was fixed already
 in 3.0pl1-64. (Closes: #528434)
  - do_command.c: check return code of setgid() and initgroups()
  - This fixes (hopefully completely) CVE-2006-2607
 * crontab.c:
    - close the temporary file after it is edited and
      before calling cleanup_tmp_crontab() to behave properly on NFS
      mounted / (Closes: #413962)
    - if crontab is run without argument then it will read stdin to replace
      the users crontab. This way it is POSIXLY_CORRECT. More information at
      http://www.opengroup.org/onlinepubs/9699919799/utilities/crontab.html
      (Closes: #514062)
 * crontab.5 :
    - Add details about multiple recipients in MAILTO (LP: #235464)
      (Closes: #502650)
    - Indicate that it also reads environment from /etc/environment
    - Substitute ATT for AT&T (Closes: #405474)
 * Proper fix for PAM configuration to make cron read the system
   environment (Closes: #511684)
 * debian/cron.init:
     - Add support for 'status' in the init.d (Closes: #514721)
     - Use 'cron' instead of 'crond' (Closes: #497699)
 * Change lockfile-progs from Suggests: to Recommends: and remove wording
   related to dselect, which is no longer relevant (Closes: #452460, #468262)
 * Change the (outdated) wording of the description based on an example
   provided by Justin B Rye (Closes: 485452)
 * Change the postinst so that update-rc.d is only run if /etc/init.d/cron is
   executable (Closes: #500610)

Branch metadata

Branch format:
Branch format 7
Repository format:
Bazaar repository format 2a (needs bzr 1.16 or later)
This branch contains Public information 
Everyone can see this information.

Subscribers