Use sysroot for dpkg-shlibdeps when building debian package

Use sysroot for dpkg-shlibdeps when building debian package

Previously we were using the host environment to calculate
shlibdeps.

BUG=

[Sam Clegg, 2016-10-06 18:42:01]

sbc@chromium.org changed reviewers:
+ thestig@chromium.org, thomasanderson@chromium.org

[Sam Clegg, 2016-10-06 18:42:01]

This change depends on having a var/lib/dpkg/ correctly configured in the
sysroot images (https://codereview.chromium.org/2391183002/)

[Lei Zhang, 2016-10-06 18:49:21]

Looks similar to my version of this CL, so I'm all for it.

https://codereview.chromium.org/2389263007/diff/1/chrome/installer/linux/debi...
File chrome/installer/linux/debian/build.sh (right):

https://codereview.chromium.org/2389263007/diff/1/chrome/installer/linux/debi...
chrome/installer/linux/debian/build.sh:123: usage() {
Update?

https://codereview.chromium.org/2389263007/diff/1/chrome/installer/linux/debi...
chrome/installer/linux/debian/build.sh:280: # The grep -v is for a duplicate
libc6 dep caused by Lucid glibc silliness.
Can you also check if this can be removed?

[Tom (Use chromium acct), 2016-10-06 19:33:01]

thomasanderson@google.com changed reviewers:
+ thomasanderson@google.com

[Tom (Use chromium acct), 2016-10-06 19:33:01]

lgtm % thestig@'s comments

[Tom (Use chromium acct), 2016-10-11 00:55:37]

Can we land this minus the stuff at chrome/installer/linux/debian/build.sh:170?

I need it for a dependent patch set.

[Sam Clegg, 2016-10-11 18:32:12]

On 2016/10/11 00:55:37, Tom Anderson wrote:
> Can we land this minus the stuff at
chrome/installer/linux/debian/build.sh:170?
> 
> I need it for a dependent patch set.

I would just clone the changes.  Does that work?   It doesn't make sense to me
to add a new command line flag but then not use it anywhere.

[Tom (Use chromium acct), 2016-10-11 18:36:52]

On 2016/10/11 18:32:12, Sam Clegg wrote:
> On 2016/10/11 00:55:37, Tom Anderson wrote:
> > Can we land this minus the stuff at
> chrome/installer/linux/debian/build.sh:170?
> > 
> > I need it for a dependent patch set.
> 
> I would just clone the changes.  Does that work?   It doesn't make sense to me
> to add a new command line flag but then not use it anywhere.

Ok, that works

[Tom (Use chromium acct), 2016-10-12 20:16:03]

Description was changed from

==========
Use sysroot for dpkg-shlibdeps when building debian package

Previously we were using the host environment to calculate
shlibdeps.

BUG=
==========

to

==========
Use sysroot for dpkg-shlibdeps when building debian package

Previously we were using the host environment to calculate
shlibdeps.

BUG=
==========

[Sam Clegg, 2016-10-12 20:34:08]

On 2016/10/11 18:36:52, Tom Anderson wrote:
> On 2016/10/11 18:32:12, Sam Clegg wrote:
> > On 2016/10/11 00:55:37, Tom Anderson wrote:
> > > Can we land this minus the stuff at
> > chrome/installer/linux/debian/build.sh:170?
> > > 
> > > I need it for a dependent patch set.
> > 
> > I would just clone the changes.  Does that work?   It doesn't make sense to
me
> > to add a new command line flag but then not use it anywhere.
> 
> Ok, that works

Closing in favor of https://codereview.chromium.org/2411423002/