Please merge readahead-list 1:1.20060421.1016-3 (main) from Debian unstable (main)

Bug #228226 reported by Julian Andres Klode
2
Affects Status Importance Assigned to Milestone
readahead-list (Ubuntu)
Invalid
Undecided
Unassigned

Bug Description

Binary package hint: readahead-list

Please merge readahead-list from Debian unstable.
I attached a debdiff to the current version in Debian.

This brings us a new release of readahead-list and some other stuff from the Debian package. It also removes the compat package
which is really not needed anymore.

Revision history for this message
Julian Andres Klode (juliank) wrote :
Changed in readahead-list:
status: New → Confirmed
Revision history for this message
Julian Andres Klode (juliank) wrote :

Needs some fixes first

Changed in readahead-list:
assignee: nobody → juliank
status: Confirmed → In Progress
Revision history for this message
Julian Andres Klode (juliank) wrote :

So this should be the final debdiff.

Changed in readahead-list:
assignee: juliank → nobody
status: In Progress → Confirmed
Revision history for this message
Daniel Holbach (dholbach) wrote :

Scott: can you please take a look at it?

Revision history for this message
Scott James Remnant (Canonical) (canonical-scott) wrote :

Our readahead packages aren't based on those in Debian, so this will require some careful consideration since it would be merging two packages with potentially different functionality.

Julian: the debdiff attached is _NOT_ based on the current Ubuntu package. Could you resubmit the debdiff to be from the current Ubuntu package to your merge?

Revision history for this message
Julian Andres Klode (juliank) wrote :

Please note that the Debian package is based on the Ubuntu package. (-0ubuntu11).

The only changes made afterwards in Ubuntu are updates of the list and a bugfix for building with GCC 4.3, which has been integrated in the Debian package in a slightly different way (See Debian #474354).

Please also note that the Debian package is a newer upstream version, found in gentoo portage (the developer of the software is a Gentoo dev, and this new version was not published at the original location.)

I have not kept all Ubuntu changelog entries, as these were only " Regenerate boot.list using an installation of ...." entries.

Revision history for this message
Scott James Remnant (Canonical) (canonical-scott) wrote :

Why the change to the boot ordering?

Revision history for this message
Julian Andres Klode (juliank) wrote :

readahead-list (1:0.20050517.0220-2) unstable; urgency=low
  * Move readahead init.d script after mountkernfs, to make sure /proc/ is mounted when checking /proc/cmdline

- dh_installinit --no-start -- start 01 S .
+ dh_installinit --no-start -- start 02 S .

----
readahead-list (1:0.20050517.0220-1) unstable; urgency=low
  * Make sure to run init.d/stop-readahead in runlevels 3, 4 and 5 too.

- dh_installinit --name=stop-readahead --no-start -- start 99 2 .
+ dh_installinit --name=stop-readahead --no-start -- start 99 2 3 4 5 .

--
The second change is to make readahead stop if we are in a higher runlevel than 2. Other packages do the same (see e.g. diff /etc/rc2.d/ /etc/rc5.d/)

Revision history for this message
Scott James Remnant (Canonical) (canonical-scott) wrote :

"m" < "r"

so S01mountkernfs.sh will always be run before S01readahead.

This change seems unnecessary.

Revision history for this message
Julian Andres Klode (juliank) wrote :

It may be technically unnecessary, but it is the way described in the policy:

"9.3 System run levels and init.d scripts
[...]
The names of the links all have the form Smmscript or Kmmscript where mm is a two-digit number and script is the name of the script (this should be the same as the name of the actual script in /etc/init.d).
[...]
The two-digit number mm is used to determine the order in which to run the scripts: low-numbered links have their scripts run first. For example, the K20 scripts will be executed before the K30 scripts. This is used when a certain service must be started before another."

Revision history for this message
Julian Andres Klode (juliank) wrote :

Updated to keep the current boot order.

Revision history for this message
Julian Andres Klode (juliank) wrote :

Update?

Revision history for this message
Scott James Remnant (Canonical) (canonical-scott) wrote :

After UVF and I haven't been able to review, deferred to Intrepid+1

Revision history for this message
Benjamin Drung (bdrung) wrote :

readahead-list was removed from Debian and Ubuntu.

Changed in readahead-list (Ubuntu):
status: Confirmed → Invalid
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.