Ticket #923 (closed defect: fixed)

Opened 10 years ago

Last modified 9 years ago

debathena-bash-config interferes with base-files’s upgrading of /etc/profile

Reported by: andersk Owned by:
Priority: normal Milestone: The Distant Future
Component: -- Keywords:
Cc: Fixed in version: debathena-bash-config 1.18
Upstream bug:

Description

base-files.postinst on Ubuntu upgrades /etc/profile as follows:

  if [ -f /etc/profile ]; then
    md5=`md5sum /etc/profile | cut -f 1 -d\ `
    if grep -qw "$md5" /usr/share/base-files/profile.md5sums; then
        cp /usr/share/base-files/profile /etc/profile
    fi
  fi

Obviously, this breaks when /etc/profile is diverted. We could go ask Ubuntu to use ‘dpkg-divert --truename /etc/profile’. However, Ubuntu since hardy (base-files 4.0.1ubuntu2) and even Debian since squeeze (base-files 5.3) support /etc/profile.d/*.sh, so we should really just go use that instead.

(Sadly there is no bashrc.d yet, but that’s okay because /etc/bash.bashrc is a conffile in the bash package, not generated in a postinst.)

Change History

comment:1 Changed 9 years ago by jdreed

  • Status changed from new to committed
  • Fixed in version set to debathena-bash-config 1.16

comment:2 Changed 9 years ago by jdreed

  • Status changed from committed to development

comment:3 Changed 9 years ago by jdreed

  • Fixed in version changed from debathena-bash-config 1.16 to debathena-bash-config 1.17

1.16 had a typo in the rules file that broke the postinst. Fixed in 1.17

comment:4 Changed 9 years ago by jdreed

  • Fixed in version changed from debathena-bash-config 1.17 to debathena-bash-config 1.18

1.17 broke the world by attempting source the nonexistent pre-diverted profile.

comment:5 Changed 9 years ago by jdreed

  • Status changed from development to proposed

comment:6 Changed 9 years ago by jdreed

  • Status changed from proposed to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.