summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKay Sievers <kay.sievers@vrfy.org>2009-09-06 17:24:26 +0200
committerKay Sievers <kay.sievers@vrfy.org>2009-09-07 12:13:49 +0200
commit6d837a5348e7b38cb122405290b7bc3c17d069aa (patch)
treeea951e5197ef3491cf18c8de9e91badcffa394eb
parent9dae0e89b9d2138e028c2a69170ba2a96503b766 (diff)
udevadm: remove symlink support for old commands
-rw-r--r--NEWS3
-rw-r--r--TODO1
-rw-r--r--udev/udevadm.c31
3 files changed, 3 insertions, 32 deletions
diff --git a/NEWS b/NEWS
index daefed8812..dcbf3a66b2 100644
--- 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 84bfbf372e..05b763efa0 100644
--- 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
diff --git a/udev/udevadm.c b/udev/udevadm.c
index b7295f3296..452489a0c7 100644
--- a/udev/udevadm.c
+++ b/udev/udevadm.c
@@ -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;