diff mbox series

[2/2] support/scripts/graph-build-time: replace confusing colors

Message ID 20190130201156.5927-2-patrickdepinguin@gmail.com
State Accepted
Headers show
Series [1/2] support/scripts/graph-build-time: add download times | expand

Commit Message

Thomas De Schampheleire Jan. 30, 2019, 8:11 p.m. UTC 
From: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>

The color for 'extract' is very similar to the one for 'install-images'.
Both are cyan-like.

Replace the former by a pale blue to make all colors sufficiently distinct.

Signed-off-by: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
---
 support/scripts/graph-build-time | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Matt Weber Feb. 4, 2019, 11:07 a.m. UTC | #1 
Thomas De,

On Wed, Jan 30, 2019 at 2:12 PM Thomas De Schampheleire
<patrickdepinguin@gmail.com> wrote:
>
> From: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
>
> The color for 'extract' is very similar to the one for 'install-images'.
> Both are cyan-like.
>
> Replace the former by a pale blue to make all colors sufficiently distinct.
>
> Signed-off-by: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>

A bit trickier to test and get a install images with enough time to
visually see it :-)

Tested-by: Matthew Weber <matthew.weber@rockwellcollins.com>
Thomas De Schampheleire Feb. 4, 2019, 12:48 p.m. UTC | #2 
El lun., 4 feb. 2019 a las 12:07, Matthew Weber
(<matthew.weber@collins.com>) escribió:
>
> Thomas De,
>
> On Wed, Jan 30, 2019 at 2:12 PM Thomas De Schampheleire
> <patrickdepinguin@gmail.com> wrote:
> >
> > From: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
> >
> > The color for 'extract' is very similar to the one for 'install-images'.
> > Both are cyan-like.
> >
> > Replace the former by a pale blue to make all colors sufficiently distinct.
> >
> > Signed-off-by: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
>
> A bit trickier to test and get a install images with enough time to
> visually see it :-)

I spoofed the build-time.log file when testing :-)

/Thomas
diff mbox series

Patch

diff --git a/support/scripts/graph-build-time b/support/scripts/graph-build-time
index 2fdb6a476d..ba3cdad85b 100755
--- a/support/scripts/graph-build-time
+++ b/support/scripts/graph-build-time
@@ -76,7 +76,7 @@  steps = ['download', 'extract', 'patch', 'configure', 'build',
 default_colors = ['#8d02ff', '#e60004', '#009836', '#2e1d86', '#ffed00',
                   '#0068b5', '#f28e00', '#940084', '#97c000']
 
-alternate_colors = ['#ffbe0a', '#00e0e0', '#3f7f7f', '#ff0000', '#00c000',
+alternate_colors = ['#ffbe0a', '#96bdff', '#3f7f7f', '#ff0000', '#00c000',
                     '#0080ff', '#c000ff', '#00eeee', '#e0e000']