Re: [coreboot] [PATCH] Remove all build/ prefixes in the build output

[Uwe Hermann]

On Wed, Oct 28, 2009 at 10:28:48AM -0600, Myles Watson wrote:
> Thanks for finding the build I missed.

No problem.
> I'd prefer having "(This may take a while)" on the next line so that it
> doesn't wrap.

Hm? I don't understand what exactly wraps here? It should be one line
only, no wrapping.
> I prefer
> print '    SCONFIG     Output File:', os.path.basename(
>
> outputfilename)
>
> over custom splits and joins.
> print '    SCONFIG   ', join(outputfilename.split('/')[-5:], '/')
>
> Can we go back to os.path.basename?  If so:

We could, but that's not consistent. This is the output with os.path.basename:

SCONFIG    config.py
    SCONFIG    static.c
    SCONFIG    static.dot

and this with the splits and joins:

SCONFIG    mainboard/asus/p2b-f/config.py
    SCONFIG    mainboard/asus/p2b-f/static.c
    SCONFIG    mainboard/asus/p2b-f/static.dot

As we do print the paths (relative to the build dir) for the other
files in the build output, I think we should do that here too.
> Acked-by: Myles Watson

Thanks, r4880.

Uwe.