cros_extract_deps -j writes JSON, sorted by package name.

cros_extract_deps -j writes JSON, sorted by package name.

Rationale: Consumers of this output should not have to eval() python
code in order to use the output. Also potentially useful for dumping
out of buildbot for consumption by other web-based tools.

[piman, 2010-08-18 16:04:31]

LGTM, though I think you can simply remove the option and always dump as json,
since the output looks compatible with the input of the other tools.

[seano, 2010-08-18 16:25:41]

Thanks, I threw the option in there in case there were tools I was unaware
of.

I'll make json the default, and take responsibility for fixing any
consumers.


On Wed, Aug 18, 2010 at 6:04 PM, <piman@chromium.org> wrote:

> LGTM, though I think you can simply remove the option and always dump as
> json,
> since the output looks compatible with the input of the other tools.
>
>
> http://codereview.chromium.org/3148022/show
>

[seano, 2010-08-18 16:45:41]

Made JSON output the default & Tested on my box.

Submitting.

On 2010/08/18 16:25:41, seano wrote:
> Thanks, I threw the option in there in case there were tools I was unaware
> of.
> 
> I'll make json the default, and take responsibility for fixing any
> consumers.
> 
> 
> On Wed, Aug 18, 2010 at 6:04 PM, <mailto:piman@chromium.org> wrote:
> 
> > LGTM, though I think you can simply remove the option and always dump as
> > json,
> > since the output looks compatible with the input of the other tools.
> >
> >
> > http://codereview.chromium.org/3148022/show
> >
>

[jimhebert, 2010-08-21 00:22:22]

On Wed, Aug 18, 2010 at 9:04 AM, <piman@chromium.org> wrote:

> LGTM, though I think you can simply remove the option and always dump as
> json,
> since the output looks compatible with the input of the other tools.
>

Unfortunately, it's not.  Bug filed,
http://code.google.com/p/chromium-os/issues/detail?id=5980

[jimhebert, 2010-08-21 00:43:30]

Bah, nevermind. PEBKAC. Withdrawn.

jim

On Fri, Aug 20, 2010 at 5:22 PM, Jim Hebert <jimhebert@chromium.org> wrote:

> On Wed, Aug 18, 2010 at 9:04 AM, <piman@chromium.org> wrote:
>
>> LGTM, though I think you can simply remove the option and always dump as
>> json,
>> since the output looks compatible with the input of the other tools.
>>
>
> Unfortunately, it's not.  Bug filed,
> http://code.google.com/p/chromium-os/issues/detail?id=5980
>