Issue 35717 in chromium-os: shill: /etc/dbus-1/system.d/flimflam.conf should be installed by shill

[chrom...@googlecode.com]

Status: Started
Owner: pet...@chromium.org
CC: ps...@chromium.org, qui...@chromium.org
Labels: Type-Bug Pri-2 Area-Network Mstone-24 Connectivity-Manager

New issue 35717 by pet...@chromium.org: shill:
/etc/dbus-1/system.d/flimflam.conf should be installed by shill
http://code.google.com/p/chromium-os/issues/detail?id=35717

Currently the file is owned and installed by flimflam. We should install a
shill.conf from shill instead.

[chrom...@googlecode.com]

Issue 35717: shill: /etc/dbus-1/system.d/flimflam.conf should be installed
by shill
http://code.google.com/p/chromium-os/issues/detail?id=35717

This issue is now blocking issue chromium-os:32415.
See http://code.google.com/p/chromium-os/issues/detail?id=32415

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

[chrom...@googlecode.com]

Updates:
Cc: chrome-...@google.com

Comment #2 on issue 35717 by bugdro...@chromium.org: shill:

/etc/dbus-1/system.d/flimflam.conf should be installed by shill

http://code.google.com/p/chromium-os/issues/detail?id=35717#c2

Project: chromiumos/platform/shill
Branch : master
Author : Darin Petkov <pet...@chromium.org>
Commit : 309c024521655236d0679c3be4541dbe19a9df51

Code Review +2: Paul Stewart
Verified +1: Darin Petkov
Change-Id : Id3b4e9882fe6d62eb963b408a1e0b7cd9865e72d
Reviewed-at : https://gerrit.chromium.org/gerrit/36772

shill: Add dbus configuration file from shill.

The file is not installed yet -- this will be followed by an ebuild
patch. Remove "cashew" user. The new file is named after the dbus
interface and we temporarily install duplicate configs (from flimflam
and shill) but that seems to work OK.

BUG=chromium-os:35717
TEST=gmerged shill, rebooted, tested connectivity, inspected logs for
crashes or dbus issues

A shims/org.chromium.flimflam.conf

[chrom...@googlecode.com]

Comment #3 on issue 35717 by bugdro...@chromium.org: shill:

/etc/dbus-1/system.d/flimflam.conf should be installed by shill

http://code.google.com/p/chromium-os/issues/detail?id=35717#c3

Project: chromiumos/overlays/chromiumos-overlay

Branch : master
Author : Darin Petkov <pet...@chromium.org>

Commit : d14b5a584f02b58760138c7ebf5a11fac9d98795

Code Review +2: Paul Stewart
Verified +1: Darin Petkov

Change-Id : I3dcdacd50130922569cc77389fa6ee4bbd6026a3
Reviewed-at : https://gerrit.chromium.org/gerrit/36773

shill: Install the dbus configuration file.

BUG=chromium-os:35717
TEST=gmerged shill, rebooted, tested connectivity, inspected logs

CQ-DEPEND=Id3b4e9882fe6d62eb963b408a1e0b7cd9865e72d

M chromeos-base/shill/shill-9999.ebuild

[chrom...@googlecode.com]

Updates:
Status: Fixed
Labels: Iteration-68

Comment #4 on issue 35717 by pet...@chromium.org: shill:

/etc/dbus-1/system.d/flimflam.conf should be installed by shill
http://code.google.com/p/chromium-os/issues/detail?id=35717

(No comment was entered for this change.)

[chrom...@googlecode.com]

Updates:
Status: Verified

Comment #5 on issue 35717 by deep...@chromium.org: shill:

/etc/dbus-1/system.d/flimflam.conf should be installed by shill
http://code.google.com/p/chromium-os/issues/detail?id=35717

Marking as Verified. We do have org.chromium.flimflam.conf in
/etc/dbus-1/system.d.
Checked in 3196.0.0.