[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#325885: ITP: readahead-list -- read files into the page cache



[Petter Reinholdtsen]
> Hi.  Any progress on your plan to package readahead-list?

I got a draft package based on the Ubuntu version
1:0.20050517.0220-0ubuntu11, and plan to upload it today unless I hear
from you.  I added LSB headers to the init.d scripts to get it working
with insserv.

One thing that occur to me is that the package names in the ubuntu
package contain some historical cruft, and we might want to reconsider
the package names in Debian.  The source package is called
readahead-list, and it generates two binary packages readahead and
readahead-list.  The latter is a transition package only pulling in
readahead.  There should be no need for a transition package in
Debian.

On the other hand, it does not make sense to let the readahead-list
source package only generate the readahead package, so it might make
more sense to drop readahead and only make a readahead-list binary
package.  It all depends on the scopes defined for the packages, I
guess.

As Ubuntu might be seen as upstream for this package, it make sense to
avoid needless divergence from the Ubuntu package.  This is an
argument for keeping readahead as the binary package with content.
One could also generate one binary package providing both the names,
to ease upgrades.

One thing to consider is that some people compiled readahead on their
own and are already using it.  Popularity-contest report 8
installations of readahead, and none of readahead-list.  To ease their
upgrade path, it is best to let the binary be called readahead.

Here is the diff with the LSB headers.

diff -u readahead-list-0.20050517.0220/debian/readahead.init readahead-list-0.20050517.0220/debian/readahead.init
--- readahead-list-0.20050517.0220/debian/readahead.init
+++ readahead-list-0.20050517.0220/debian/readahead.init
@@ -1,5 +1,13 @@
 #!/bin/sh -e
-# init script for readahead
+### BEGIN INIT INFO
+# Provides:          readahead
+# Required-Start:
+# Required-Stop:
+# X-Start-Before:    mountkernfs hostname
+# Default-Start:     S
+# Default-Stop:
+# Short-Description: init script for readahead
+### END INIT INFO

 # Check the package is still installed
 [ -x /sbin/readahead-list ] || exit 0

diff -u readahead-list-0.20050517.0220/debian/readahead-desktop.init readahead-list-0.20050517.0220/debian/readahead-desktop.init
--- readahead-list-0.20050517.0220/debian/readahead-desktop.init
+++ readahead-list-0.20050517.0220/debian/readahead-desktop.init
@@ -1,5 +1,13 @@
 #!/bin/sh -e
-# init script for readahead (second stage)
+### BEGIN INIT INFO
+# Provides:          readahead-desktop
+# Required-Start:    $local_fs readahead
+# Required-Stop:
+# X-Start-Before:    $network
+# Default-Start:     S
+# Default-Stop:
+# Short-Description: init script for readahead (second stage)
+### END INIT INFO

 # Check the package is still installed
 [ -x /sbin/readahead-list ] || exit 0
diff -u readahead-list-0.20050517.0220/debian/stop-readahead.init readahead-list-0.20050517.0220/debian/stop-readahead.init
--- readahead-list-0.20050517.0220/debian/stop-readahead.init
+++ readahead-list-0.20050517.0220/debian/stop-readahead.init
@@ -1,5 +1,12 @@
 #!/bin/sh -e
-# init script for stopping readahead profiling
+### BEGIN INIT INFO
+# Provides:          stop-readahead
+# Required-Start:    $all readahead-desktop
+# Required-Stop:
+# Default-Start:     2 3 4 5
+# Default-Stop:
+# Short-Description: init script for stopping readahead profiling
+### END INIT INFO

 # Check the package is still installed
 [ -x /sbin/readahead-list ] || exit 0

Happy hacking,
-- 
Petter Reinholdtsen



Reply to: