Code review comment for lp:~fougner/ubuntu/oneiric/base-files/fix-for-790050

Revision history for this message
Anders Kaseorg (andersk) wrote :

In the interest of reducing the delta from Debian <http://patches.ubuntu.com/b/base-files/base-files_6.3ubuntu2.patch>:
• There also are two copies of ‘etc/profile.d’ in debian/directory-list. The second was added by Ubuntu and should be removed.
• The comment about the default umask isn’t in Debian at all and might as well be removed now too.

When modifying share/profile, you also need to update share/profile.md5sums (by running ‘debian/rules clean’), so that old copies of /etc/profile will be automatically updated by the postinst.

« Back to merge proposal