Use chroot name instead of full path chroot command

Use chroot name instead of full path chroot command

chroot name is desired for -c option instead of the full path chroot
command

BUG=


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=171145

[Wei James(wistoch), 2012-12-04 08:48:51]

no response from tansell. 

+mmoss

[Michael Moss, 2012-12-04 16:15:21]

LGTM

[Wei James(wistoch), 2012-12-05 00:24:49]

On 2012/12/04 16:15:21, Michael Moss wrote:
> LGTM

thanks!

[commit-bot: I haz the power, 2012-12-05 00:26:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/james.wei@intel.com/11418284/1

[commit-bot: I haz the power, 2012-12-05 00:29:57]

Step "update" is always a major failure.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2012-12-05 00:43:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/james.wei@intel.com/11418284/1

[commit-bot: I haz the power, 2012-12-05 03:35:58]

Change committed as 171145

[mithro, 2012-12-05 04:27:38]

On 2012/12/04 08:48:51, Wei James(wistoch) wrote:
> no response from tansell. 
> 
> +mmoss

Sorry, I've been in a course this week.

What is this CL trying to fix? It breaks if /usr/local/bin isn't in PATH...

Tim

[Wei James(wistoch), 2012-12-05 04:31:08]

On 2012/12/05 04:27:38, tansell wrote:
> On 2012/12/04 08:48:51, Wei James(wistoch) wrote:
> > no response from tansell. 
> > 
> > +mmoss
> 
> Sorry, I've been in a course this week.
> 
> What is this CL trying to fix? It breaks if /usr/local/bin isn't in PATH...
> 
> Tim

in this place, the name of the chroot required, it is not to execute the script.


sudo schroot -c /usr/local/bin/"${target%bit}" -p -- apt-get clean

if you run the above command, it will report "chroot not found".

[Tim Ansell (dont use), 2012-12-05 23:22:31]

Ahh, looks like you where right.

I guess we should be checking the error code here?

Tim


On 5 December 2012 15:31, <james.wei@intel.com> wrote:

> On 2012/12/05 04:27:38, tansell wrote:
>
>> On 2012/12/04 08:48:51, Wei James(wistoch) wrote:
>> > no response from tansell.
>> >
>> > +mmoss
>>
>
>  Sorry, I've been in a course this week.
>>
>
>  What is this CL trying to fix? It breaks if /usr/local/bin isn't in
>> PATH...
>>
>
>  Tim
>>
>
> in this place, the name of the chroot required, it is not to execute the
> script.
>
>
>
> sudo schroot -c /usr/local/bin/"${target%bit}" -p -- apt-get clean
>
> if you run the above command, it will report "chroot not found".
>
>
>
>
https://chromiumcodereview.**appspot.com/11418284/<https://chromiumcodereview...
>