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

Issue 685738 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: May 2017
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Split up media-rules package into multiple chipset specific ebuilds

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

Issue description

The media-rules ebuild (https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/release-R57-9202.B/media-libs/media-rules/) installs platform/chipset specific udev rules and trigger commands on all platforms. Split it up into multiple chipset specific ebuilds.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 10 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/ddc91c05383d9c46887f85f548895d8ca1e1ed6c

commit ddc91c05383d9c46887f85f548895d8ca1e1ed6c
Author: Matthias Kaehlcke <mka@chromium.org>
Date: Fri Feb 10 20:11:46 2017

media-libs: Add package go2001-rules

CL:436524 moves most of the media-rules to the BSP overlays. The only
remainder are the rules and triggers for the go2001. These are provided
by the new package, the package media-rules will be removed.

BUG= chromium:685738 
TEST=precq passes

Change-Id: I9f43a5e33f99cfeaac1ee3c84fbf9621079a8191
Reviewed-on: https://chromium-review.googlesource.com/435432
Commit-Ready: Matthias Kaehlcke <mka@chromium.org>
Tested-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Owen Lin <owenlin@chromium.org>

[add] https://crrev.com/ddc91c05383d9c46887f85f548895d8ca1e1ed6c/media-libs/go2001-rules/go2001-rules-0.0.1.ebuild
[add] https://crrev.com/ddc91c05383d9c46887f85f548895d8ca1e1ed6c/media-libs/go2001-rules/files/udev-trigger-codec.conf
[add] https://crrev.com/ddc91c05383d9c46887f85f548895d8ca1e1ed6c/media-libs/go2001-rules/go2001-rules-0.0.1-r1.ebuild
[add] https://crrev.com/ddc91c05383d9c46887f85f548895d8ca1e1ed6c/media-libs/go2001-rules/files/50-go2001.rules

Project Member

Comment 3 by bugdroid1@chromium.org, Mar 7 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/b9cf9a27514a2117903dc3a12cd58f7616d88a71

commit b9cf9a27514a2117903dc3a12cd58f7616d88a71
Author: Matthias Kaehlcke <mka@chromium.org>
Date: Tue Mar 07 19:56:58 2017

chromeos-bsp-baseboard-oak: Install udev rules and triggers for codecs

media-libs/media-rules contains rules for different platforms, use only
applicable rules instead.

Also remove media-rules dependency from chromeos-bsp-elm and
chromeos-bsp-elm which depend on the baseboard.

BUG= chromium:685738 
TEST=precq passes

Change-Id: I7bd11ecf560a2c70155f63e8f88eeaf1f6692561
Reviewed-on: https://chromium-review.googlesource.com/448631
Commit-Ready: Matthias Kaehlcke <mka@chromium.org>
Tested-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Owen Lin <owenlin@chromium.org>

[add] https://crrev.com/b9cf9a27514a2117903dc3a12cd58f7616d88a71/baseboard-oak/chromeos-base/chromeos-bsp-baseboard-oak/files/udev-trigger-codec.conf
[rename] https://crrev.com/b9cf9a27514a2117903dc3a12cd58f7616d88a71/overlay-elm/chromeos-base/chromeos-bsp-elm/chromeos-bsp-elm-0.0.1-r14.ebuild
[modify] https://crrev.com/b9cf9a27514a2117903dc3a12cd58f7616d88a71/baseboard-oak/chromeos-base/chromeos-bsp-baseboard-oak/chromeos-bsp-baseboard-oak-0.0.1.ebuild
[modify] https://crrev.com/b9cf9a27514a2117903dc3a12cd58f7616d88a71/overlay-oak/chromeos-base/chromeos-bsp-oak/chromeos-bsp-oak-0.0.1.ebuild
[rename] https://crrev.com/b9cf9a27514a2117903dc3a12cd58f7616d88a71/overlay-oak/chromeos-base/chromeos-bsp-oak/chromeos-bsp-oak-0.0.1-r19.ebuild
[modify] https://crrev.com/b9cf9a27514a2117903dc3a12cd58f7616d88a71/overlay-elm/chromeos-base/chromeos-bsp-elm/chromeos-bsp-elm-0.0.1.ebuild
[add] https://crrev.com/b9cf9a27514a2117903dc3a12cd58f7616d88a71/baseboard-oak/chromeos-base/chromeos-bsp-baseboard-oak/files/50-media.rules
[rename] https://crrev.com/b9cf9a27514a2117903dc3a12cd58f7616d88a71/baseboard-oak/chromeos-base/chromeos-bsp-baseboard-oak/chromeos-bsp-baseboard-oak-0.0.1-r2.ebuild

Project Member

Comment 4 by bugdroid1@chromium.org, Mar 7 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/c07868d62223a2b8b9d83969d118060f3d449b96

commit c07868d62223a2b8b9d83969d118060f3d449b96
Author: Matthias Kaehlcke <mka@chromium.org>
Date: Tue Mar 07 19:56:59 2017

chromeos-bsp-buddy: Only install codec rules and triggers for kepler

The other rules in media-lib/media-rules are not applicable and the
package will be removed.

BUG= chromium:685738 
TEST=precq passes

Change-Id: Id8a90a262a2744a88dc9596a3aadba47c496094e
Reviewed-on: https://chromium-review.googlesource.com/448632
Commit-Ready: Matthias Kaehlcke <mka@chromium.org>
Tested-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Owen Lin <owenlin@chromium.org>

[modify] https://crrev.com/c07868d62223a2b8b9d83969d118060f3d449b96/overlay-buddy/chromeos-base/chromeos-bsp-buddy/chromeos-bsp-buddy-0.0.1.ebuild
[rename] https://crrev.com/c07868d62223a2b8b9d83969d118060f3d449b96/overlay-buddy/chromeos-base/chromeos-bsp-buddy/chromeos-bsp-buddy-0.0.1-r14.ebuild

Project Member

Comment 7 by bugdroid1@chromium.org, Mar 8 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/c0793dcc228fe799955bbc7fdc8432d4c3e43d65

commit c0793dcc228fe799955bbc7fdc8432d4c3e43d65
Author: Matthias Kaehlcke <mka@chromium.org>
Date: Wed Mar 08 03:55:55 2017

chromeos-bsp-rikku: Only install codec rules and triggers for kepler

The other rules in media-lib/media-rules are not applicable and the
package will be removed.

BUG= chromium:685738 
TEST=precq passes

Change-Id: Ic1cb3b9d2dbb729da1bc09f34a3fb7e845d25190
Reviewed-on: https://chromium-review.googlesource.com/448836
Commit-Ready: Matthias Kaehlcke <mka@chromium.org>
Tested-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Owen Lin <owenlin@chromium.org>

[modify] https://crrev.com/c0793dcc228fe799955bbc7fdc8432d4c3e43d65/overlay-rikku/chromeos-base/chromeos-bsp-rikku/chromeos-bsp-rikku-0.0.1.ebuild
[rename] https://crrev.com/c0793dcc228fe799955bbc7fdc8432d4c3e43d65/overlay-rikku/chromeos-base/chromeos-bsp-rikku/chromeos-bsp-rikku-0.0.1-r11.ebuild

Project Member

Comment 9 by bugdroid1@chromium.org, Mar 8 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/156c873dfa3b6d1549a5a0936c07bd145864e18a

commit 156c873dfa3b6d1549a5a0936c07bd145864e18a
Author: Matthias Kaehlcke <mka@chromium.org>
Date: Wed Mar 08 21:23:02 2017

chromeos-bsp-jecht: Remove kepler dependencies

jecht does not ship with kepler

BUG= chromium:685738 
TEST=precq passes

Change-Id: Id5f52d49820d9827a9e1dc937acaed9aeeb72398
Reviewed-on: https://chromium-review.googlesource.com/448838
Commit-Ready: Matthias Kaehlcke <mka@chromium.org>
Tested-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Owen Lin <owenlin@chromium.org>

[modify] https://crrev.com/156c873dfa3b6d1549a5a0936c07bd145864e18a/overlay-jecht/chromeos-base/chromeos-bsp-jecht/chromeos-bsp-jecht-0.0.1.ebuild
[rename] https://crrev.com/156c873dfa3b6d1549a5a0936c07bd145864e18a/overlay-jecht/chromeos-base/chromeos-bsp-jecht/chromeos-bsp-jecht-0.0.1-r9.ebuild

Project Member

Comment 10 by bugdroid1@chromium.org, Mar 8 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/66a98f456082b45843263118de94bd12d474b147

commit 66a98f456082b45843263118de94bd12d474b147
Author: Matthias Kaehlcke <mka@chromium.org>
Date: Wed Mar 08 21:23:02 2017

chromeos-bsp-panther: Remove kepler dependencies

panther does not ship with kepler

BUG= chromium:685738 
TEST=precq passes

Change-Id: Ifb539e132e5a4ccf58e1aeab404dd8b79aada1cd
Reviewed-on: https://chromium-review.googlesource.com/448839
Commit-Ready: Matthias Kaehlcke <mka@chromium.org>
Tested-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Owen Lin <owenlin@chromium.org>

[modify] https://crrev.com/66a98f456082b45843263118de94bd12d474b147/overlay-panther/chromeos-base/chromeos-bsp-panther/chromeos-bsp-panther-0.0.1.ebuild
[rename] https://crrev.com/66a98f456082b45843263118de94bd12d474b147/overlay-panther/chromeos-base/chromeos-bsp-panther/chromeos-bsp-panther-0.0.1-r9.ebuild

Project Member

Comment 11 by bugdroid1@chromium.org, Mar 8 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/0eb42570a32ba3efcffc6994ab43a91b69bf2c37

commit 0eb42570a32ba3efcffc6994ab43a91b69bf2c37
Author: Matthias Kaehlcke <mka@chromium.org>
Date: Wed Mar 08 23:22:01 2017

chromeos-bsp-hana: Add missing dependency on baseboard-oak

BUG=b:36059332,  chromium:685738 
TEST=emerge-hana -aC media-rules
  build_packages --board=hana
  /etc/init/platform-cpusets.conf exists
  /lib/udev/rules.d/50-media.rules exists

Change-Id: I359d45ce8a610b2b9d4cba73c442cdf5c5a407f2
Reviewed-on: https://chromium-review.googlesource.com/451580
Commit-Ready: Matthias Kaehlcke <mka@chromium.org>
Tested-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Nicolas Boichat <drinkcat@chromium.org>

[rename] https://crrev.com/0eb42570a32ba3efcffc6994ab43a91b69bf2c37/overlay-hana/chromeos-base/chromeos-bsp-hana/chromeos-bsp-hana-0.0.1-r6.ebuild
[modify] https://crrev.com/0eb42570a32ba3efcffc6994ab43a91b69bf2c37/overlay-hana/chromeos-base/chromeos-bsp-hana/chromeos-bsp-hana-0.0.1.ebuild

Project Member

Comment 12 by bugdroid1@chromium.org, Mar 8 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/0eb42570a32ba3efcffc6994ab43a91b69bf2c37

commit 0eb42570a32ba3efcffc6994ab43a91b69bf2c37
Author: Matthias Kaehlcke <mka@chromium.org>
Date: Wed Mar 08 23:22:01 2017

chromeos-bsp-hana: Add missing dependency on baseboard-oak

BUG=b:36059332,  chromium:685738 
TEST=emerge-hana -aC media-rules
  build_packages --board=hana
  /etc/init/platform-cpusets.conf exists
  /lib/udev/rules.d/50-media.rules exists

Change-Id: I359d45ce8a610b2b9d4cba73c442cdf5c5a407f2
Reviewed-on: https://chromium-review.googlesource.com/451580
Commit-Ready: Matthias Kaehlcke <mka@chromium.org>
Tested-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Nicolas Boichat <drinkcat@chromium.org>

[rename] https://crrev.com/0eb42570a32ba3efcffc6994ab43a91b69bf2c37/overlay-hana/chromeos-base/chromeos-bsp-hana/chromeos-bsp-hana-0.0.1-r6.ebuild
[modify] https://crrev.com/0eb42570a32ba3efcffc6994ab43a91b69bf2c37/overlay-hana/chromeos-base/chromeos-bsp-hana/chromeos-bsp-hana-0.0.1.ebuild

Project Member

Comment 13 by bugdroid1@chromium.org, Mar 10 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/overlays/overlay-rowan-private/+/0d8820589f0cf02007ce327ae69fab2e8083fcd6

commit 0d8820589f0cf02007ce327ae69fab2e8083fcd6
Author: Matthias Kaehlcke <mka@chromium.org>
Date: Fri Mar 10 21:05:16 2017

Project Member

Comment 14 by bugdroid1@chromium.org, Mar 18 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/aaf7b57134f16a12d46699ef12fb331d87299e49

commit aaf7b57134f16a12d46699ef12fb331d87299e49
Author: Matthias Kaehlcke <mka@chromium.org>
Date: Sat Mar 18 09:13:44 2017

media-libs: Remove package media-rules

The rules and triggers have been moved to the platform/board overlays
and the media-libs/go2001-rules package.

BUG= chromium:685738 
TEST=precq passes

Change-Id: I7dbc0bc7e3767ce0fe332f477ddf23a78e57306f
Reviewed-on: https://chromium-review.googlesource.com/436525
Commit-Ready: Matthias Kaehlcke <mka@chromium.org>
Tested-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Owen Lin <owenlin@chromium.org>

[delete] https://crrev.com/b0758975e1efd8f2e60ca14b3ea1cdcdca491b6b/media-libs/media-rules/files/50-media.rules
[delete] https://crrev.com/b0758975e1efd8f2e60ca14b3ea1cdcdca491b6b/media-libs/media-rules/media-rules-0.1.0.ebuild
[delete] https://crrev.com/b0758975e1efd8f2e60ca14b3ea1cdcdca491b6b/media-libs/media-rules/media-rules-0.1.0-r13.ebuild
[delete] https://crrev.com/b0758975e1efd8f2e60ca14b3ea1cdcdca491b6b/media-libs/media-rules/files/udev-trigger-codec.conf

Comment 15 by mka@chromium.org, May 1 2017

Status: Fixed (was: Unconfirmed)
Labels: VerifyIn-61

Comment 17 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment