[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