chiark / gitweb /
udevadm: remove symlink support for old commands
authorKay Sievers <kay.sievers@vrfy.org>
Sun, 6 Sep 2009 15:24:26 +0000 (17:24 +0200)
committerKay Sievers <kay.sievers@vrfy.org>
Mon, 7 Sep 2009 10:13:49 +0000 (12:13 +0200)
NEWS
TODO
udev/udevadm.c

diff --git a/NEWS b/NEWS
index daefed8..dcbf3a6 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -13,6 +13,9 @@ NAME="%k" causes a warning now. It's is and always was completely superfluous.
 It will break kernel supplied DEVNAMEs and therefore it needs to be removed
 from all rules.
 
+Symlinks to udevadm with the old command names are no longer resolved to
+the udevadm commands.
+
 udev 146
 ========
 Bugfixes.
diff --git a/TODO b/TODO
index 84bfbf3..05b763e 100644 (file)
--- a/TODO
+++ b/TODO
@@ -8,7 +8,6 @@
      from the name stack, they can not conflict.
   o remove most NAME= rules (they are provided by the 2.6.31 kernel)
   o convert firmware.sh to C
-  o symlink names to udevadm will no longer be resolved to old command names
   o "udevadm control" commands will only accept the --<command> syntax
   o get rid of braindead "scan all devices to find myself" libusb interface
       if it can not be fixed, drop libusb entirely
index b7295f3..452489a 100644 (file)
@@ -131,7 +131,6 @@ int main(int argc, char *argv[])
        };
        const char *command;
        int i;
-       const char *pos;
        int rc = 1;
 
        udev = udev_new();
@@ -142,36 +141,6 @@ int main(int argc, char *argv[])
        udev_set_log_fn(udev, log_fn);
        udev_selinux_init(udev);
 
-       /* see if we are a compat link, this will be removed in a future release */
-       command = argv[0];
-       pos = strrchr(command, '/');
-       if (pos != NULL)
-               command = &pos[1];
-
-       /* the trailing part of the binary or link name is the command */
-       if (strncmp(command, "udev", 4) == 0)
-               command = &command[4];
-
-       for (i = 0; cmds[i].cmd != NULL; i++) {
-               if (strcmp(cmds[i].name, command) == 0) {
-                       char path[128];
-                       char prog[512];
-                       ssize_t len;
-
-                       snprintf(path, sizeof(path), "/proc/%lu/exe", (unsigned long) getppid());
-                       len = readlink(path, prog, sizeof(prog));
-                       if (len > 0) {
-                               prog[len] = '\0';
-                               fprintf(stderr, "the program '%s' called '%s', it should use 'udevadm %s <options>', "
-                                      "this will stop working in a future release\n", prog, argv[0], command);
-                               err(udev, "the program '%s' called '%s', it should use 'udevadm %s <options>', "
-                                   "this will stop working in a future release\n", prog, argv[0], command);
-                       }
-                       rc = run_command(udev, &cmds[i], argc, argv);
-                       goto out;
-               }
-       }
-
        while (1) {
                int option;