duplicate 00-header file shipped

Bug #516293 reported by James Troup
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
base-files (Ubuntu)
Fix Released
Medium
Dustin Kirkland 
mountall (Ubuntu)
Fix Released
Medium
Dustin Kirkland 

Bug Description

Binary package hint: base-files

I don't believe /etc/00-header should be in the package.

| root@adare:/etc# dpkg -S 00-header
| base-files: /etc/update-motd.d/00-header
| base-files: /etc/00-header
| root@adare:/etc# cat 00-header
| #!/bin/sh
|
| # Header for /etc/motd
| # These two lines were taken from /etc/init.d/bootlogs.sh
| uname -snrvm
| [ -f /etc/motd.tail ] && cat /etc/motd.tail
| root@adare:/etc# cmp /etc/update-motd.d/00-header /etc/00-header
| root@adare:/etc# ls -l /etc/update-motd.d/00-header /etc/00-header
| -rw-r--r-- 1 root root 149 2009-07-14 02:25 /etc/00-header
| -rwxr-xr-x 1 root root 149 2009-07-14 02:25 /etc/update-motd.d/00-header
| root@adare:/etc#

This is a machine freshly upgraded to karmic, so is
base-files_5.0.0ubuntu7.

Revision history for this message
James Troup (elmo) wrote :

This is still present in lucid along with the 10-help-text file.

Changed in base-files (Ubuntu):
status: New → Confirmed
Revision history for this message
Daniel Richard G. (skunk) wrote :

I believe the problem is line 53 in debian/rules:

    install -p -m 644 etc/* debian/tmp/etc

What's in the etc/ subdirectory of the deb source tree?

root@test-ubuntu:/tmp/base-files-5.0.0ubuntu13# ls etc/
00-header debian_version issue legal motd
10-help-text host.conf issue.net lsb-release

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

mountall's /etc/init/mounted-varrun.conf is creating motd.tail. It doesn't need to do so anymore. I'll get this fixed.

Changed in base-files (Ubuntu):
importance: Undecided → Medium
status: Confirmed → In Progress
assignee: nobody → Dustin Kirkland (kirkland)
Changed in mountall (Ubuntu):
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Dustin Kirkland (kirkland)
Changed in mountall (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package mountall - 2.11

---------------
mountall (2.11) lucid; urgency=low

  [ Scott James Remnant ]
  * conf/mounted-*.conf: Add defaults for $MOUNTPOINT just in case somebody
    tries to run this by hand. LP: #557177.

  [ Dustin Kirkland ]
  * conf/mounted-varrun.conf: drop initial motd creation, as this is now
    handled entirely dynamically by /etc/update-motd.d/* scripts, with
    this part specifically handled by base-files's 00-header,
    LP: #516293
 -- Dustin Kirkland <email address hidden> Fri, 09 Apr 2010 13:50:29 -0500

Changed in mountall (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package base-files - 5.0.0ubuntu15

---------------
base-files (5.0.0ubuntu15) lucid; urgency=low

  * debian/links, debian/postinst.in, debian/rules:
    remove artifacts of old base motd creation, keep duplicated data
    from popping up in motd, LP: #516293
 -- Dustin Kirkland <email address hidden> Fri, 09 Apr 2010 14:23:01 -0500

Changed in base-files (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.