[Libreoffice] Do we support classic and industrial theme? (was: [PATCH] configure.in examples: options shouldn't have quote)
Michael Meeks
michael.meeks at suse.com
Mon Dec 12 06:25:24 PST 2011
Hi Korrawit,
On Sun, 2011-12-11 at 21:39 +0700, Korrawit Pruegsanusak wrote:
> [3] diff as follows:
> diff --git a/configure.in b/configure.in
> index f8aed0e..132e022 100644
> --- a/configure.in
> +++ b/configure.in
> @@ -8034,7 +8034,7 @@ fi
> WITH_THEMES=""
> for theme in $with_theme; do
> case $theme in
> - default|crystal|hicontrast|oxygen|tango) : ;;
> + default|crystal|hicontrast|oxygen|tango|classic|industrial) : ;;
> *) AC_MSG_ERROR([Unknown value for --with-theme: $theme]) ;;
> esac
> WITH_THEMES="$WITH_THEMES $theme"
I suggest we push your patch; industrial is used as layer in a stack of
layers used to build other themes, so we can't remove those icons from
git; but the classic stuff we're well rid of - and it won't keep up with
new icon names / movements etc. I suspect. The classic tar-ball is still
in git as a static un-maintained blob; we could prolly remove that too
in master - if you knock up a new patch that can be git am'd ;-) ...
HTH,
Michael.
--
michael.meeks at suse.com <><, Pseudo Engineer, itinerant idiot
More information about the LibreOffice
mailing list