[LEDE-DEV] merge: add OpenWrt branding

Hannu Nyman hannu.nyman at iki.fi
Tue Oct 24 07:37:30 PDT 2017


Zoltan HERPAI kirjoitti 24.10.2017 klo 14:46:
> Given that we've decided to sail under the same flag for
> the benefit of the whole community, and acknowledge the
> achievements of the LEDE project, let's start the final
> steps of the merge.
>      
> The git and other sources are untouched until the infra merge.
>      
> ...

> @@ -1,10 +1,13 @@
> -     _________
> -    /        /\      _    ___ ___  ___
> -   /  LE    /  \    | |  | __|   \| __|
> -  /    DE  /    \   | |__| _|| |) | _|
> - /________/  LE  \  |____|___|___/|___|                      lede-project.org
> - \        \   DE /
> -  \    LE  \    /  -----------------------------------------------------------
> -   \  DE    \  /    %N (%V, %C)
> -    \________\/    -----------------------------------------------------------
> -
> +  _______                     ________        __
> + |       |.-----.-----.-----.|  |  |  |.----.|  |_
> + |   -   ||  _  |  -__|     ||  |  |  ||   _||   _|
> + |_______||   __|_____|__|__||________||__|  |____|
> +          |__| W I R E L E S S   F R E E D O M
> + -----------------------------------------------------
> + DESIGNATED DRIVER (%C, %R)
> + -----------------------------------------------------
> +  * 2 oz. Orange Juice         Combine all juices in a
> +  * 2 oz. Pineapple Juice      tall glass filled with
> +  * 2 oz. Grapefruit Juice     ice, stir well.
> +  * 2 oz. Cranberry Juice
> + -----------------------------------------------------


Please do not re-introduce hard-coded version branding into various files.

All version info was centralised to include/version.mk by this commit:

https://github.com/lede-project/source/commit/3193053df7d27bde82fbcc310e4404cc43f39313#diff-d08225593c57fc07ce549597f7fd4c68

and further refined with

https://github.com/lede-project/source/commit/5f3c96c28503b8a0aad1514ef144b15e670d338b#diff-d08225593c57fc07ce549597f7fd4c68

Having hard-coded versions scattered around in various files proved to be 
cumbersome at several Openwrt releases, as some of the hard-coded names were 
forgotten to be changed in connection to several releases.

In LEDE sources the variables like "%N (%V, %C)" take nicely care of the 
version branding also in /etc/banner, also including the version name.

Using hard-coded name like "DESIGNATED DRIVER (%C, %R)" would be a clear step 
backwards.



Ps.

And are the drinks really back on the menu?





More information about the Lede-dev mailing list