New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 681125 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Jan 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Remove all dependencies on symlink /sbin/udevadm

Project Member Reported by mka@chromium.org, Jan 13 2017

Issue description

Newer udev versions install udevadm in /bin. Some parts of the system expect it in /sbin. To accommodate that the udev-208 ebuild creates a symlink. udev-225 has just been merged, and it currently does not create the symlink. 

We could add the symlink, but it seems more reasonable to remove the few dependencies on it:

src/partner_private/img-ddk/build/linux/scripts/common.m4
src/third_party/bluez/tools/hid2hci.rules

Another dependency in initramfs is removed by https://chromium-review.googlesource.com/#/c/427900/
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 17 2017

Labels: merge-merged-chromeos-5.41
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/bluez/+/9eab451fb18c479b5e0f0667804d8e376906b17b

commit 9eab451fb18c479b5e0f0667804d8e376906b17b
Author: Matthias Kaehlcke <mka@chromium.org>
Date: Fri Jan 13 19:41:51 2017

bluez: Fix udevadm path in hid2hci.rules

Newer udev versions install udevadm in /bin and we don't create the
compatibility symlink anymore.

BUG= chromium:681125 
TEST=none

Change-Id: I2dbb61763f1eb51b709864630b5ae753fcc8f4b6
Reviewed-on: https://chromium-review.googlesource.com/428179
Commit-Ready: Matthias Kaehlcke <mka@chromium.org>
Tested-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Luigi Semenzato <semenzato@chromium.org>
Reviewed-by: Miao-chen Chou <mcchou@chromium.org>

[modify] https://crrev.com/9eab451fb18c479b5e0f0667804d8e376906b17b/tools/hid2hci.rules

Comment 2 by mka@chromium.org, Jan 17 2017

Status: Fixed (was: Started)

Comment 3 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58
Status: Verified (was: Fixed)
Verified.
Project Member

Comment 5 by bugdroid1@chromium.org, Apr 19 2017

Labels: merge-merged-chromeos-5.44
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/bluez/+/7cdf3777e99023448801f9bcc8fea42a737f7eb4

commit 7cdf3777e99023448801f9bcc8fea42a737f7eb4
Author: Matthias Kaehlcke <mka@chromium.org>
Date: Wed Apr 19 07:15:43 2017

CHROMIUM: bluez: Fix udevadm path in hid2hci.rules

Newer udev versions install udevadm in /bin and we don't create the
compatibility symlink anymore.

BUG= chromium:681125 
TEST=none

Old-Change-Id: I2dbb61763f1eb51b709864630b5ae753fcc8f4b6
Change-Id: I3d67a1594163844c73cef9f58c9643eef4c3a6b8
Reviewed-on: https://chromium-review.googlesource.com/469432
Reviewed-by: Rahul Chaturvedi <rkc@chromium.org>
Tested-by: Miao-chen Chou <mcchou@chromium.org>
Commit-Queue: Miao-chen Chou <mcchou@chromium.org>
Trybot-Ready: Miao-chen Chou <mcchou@chromium.org>

[modify] https://crrev.com/7cdf3777e99023448801f9bcc8fea42a737f7eb4/tools/hid2hci.rules

Sign in to add a comment