James Hunt has proposed merging 
lp:~jamesodhunt/upstart/bug-980917-the-bug-that-would-not-die into lp:upstart.

Requested reviews:
  Upstart Reviewers (upstart-reviewers)

For more details, see:
https://code.launchpad.net/~jamesodhunt/upstart/bug-980917-the-bug-that-would-not-die/+merge/118579

* init/main.c: main(): Remove checks for /dev/kmsg, /dev/null,
  /dev/console and /dev/tty since some environments use different
  major/minor numbers to the norm (for example on LXC, /dev/console is
  not (5,1), but (136,*)). Do not attempt to create /dev/console again,
  due to LXC numbering difference.
-- 
https://code.launchpad.net/~jamesodhunt/upstart/bug-980917-the-bug-that-would-not-die/+merge/118579
Your team Upstart Reviewers is requested to review the proposed merge of 
lp:~jamesodhunt/upstart/bug-980917-the-bug-that-would-not-die into lp:upstart.
=== modified file 'ChangeLog'
--- ChangeLog	2012-08-03 15:47:59 +0000
+++ ChangeLog	2012-08-07 15:24:24 +0000
@@ -1,3 +1,11 @@
+2012-08-07  James Hunt  <[email protected]>
+
+	* init/main.c: main(): Remove checks for /dev/kmsg, /dev/null,
+	  /dev/console and /dev/tty since some environments use different
+	  major/minor numbers to the norm (for example on LXC, /dev/console is
+	  not (5,1), but (136,*)). Do not attempt to create /dev/console again,
+	  due to LXC numbering difference.
+
 2012-08-03  James Hunt  <[email protected]>
 
 	* init/main.c:main(): Handle hostile initramfs-less environments by

=== modified file 'init/main.c'
--- init/main.c	2012-08-07 14:33:12 +0000
+++ init/main.c	2012-08-07 15:24:24 +0000
@@ -244,15 +244,11 @@
 		 */
 		(void)umask (0);
 
-		/* Check if key devices already exist; if they do,
+		/* Check if key /dev entries already exist; if they do,
 		 * we should assume we don't need to mount /dev.
 		 */
 		if (system_check_file ("/dev/ptmx", S_IFCHR, makedev (5, 2)) < 0
-			|| system_check_file ("/dev/pts", S_IFDIR, 0) < 0
-			|| system_check_file ("/dev/kmsg", S_IFCHR, makedev (1, 11)) < 0
-			|| system_check_file ("/dev/null", S_IFCHR, makedev (1, 3)) < 0
-			|| system_check_file ("/dev/console", S_IFCHR, makedev (5, 1)) < 0
-			|| system_check_file ("/dev/tty", S_IFCHR, makedev (5, 0)) < 0)
+			|| system_check_file ("/dev/pts", S_IFDIR, 0) < 0)
 			needs_devtmpfs = 1;
 
 		if (needs_devtmpfs) {
@@ -290,7 +286,6 @@
 		 */
 		system_mknod ("/dev/null", (S_IFCHR | 0666), makedev (1, 3));
 		system_mknod ("/dev/tty", (S_IFCHR | 0666), makedev (5, 0));
-		system_mknod ("/dev/console", (S_IFCHR | 0600), makedev (5, 1));
 		system_mknod ("/dev/kmsg", (S_IFCHR | 0600), makedev (1, 11));
 
 		/* Set the standard file descriptors to the ordinary console device,

-- 
upstart-devel mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/upstart-devel

Reply via email to