Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 13/30] package/matchbox: fix option prompts
Date: Fri, 24 Apr 2015 18:50:40 +0200	[thread overview]
Message-ID: <20150424165040.GA4244@free.fr> (raw)
In-Reply-To: <553971A3.7040808@mind.be>

Arnout, All,

On 2015-04-24 00:26 +0200, Arnout Vandecappelle spake thusly:
> On 04/22/15 20:09, Yann E. MORIN wrote:
> > Use the package names instead of some fancy human-readable strings. ;-)
> > 
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > ---
> >  package/matchbox/Config.in                   | 2 +-
> >  package/matchbox/matchbox-common/Config.in   | 4 ++--
> >  package/matchbox/matchbox-desktop/Config.in  | 2 +-
> >  package/matchbox/matchbox-keyboard/Config.in | 2 +-
> >  package/matchbox/matchbox-panel/Config.in    | 2 +-
> >  5 files changed, 6 insertions(+), 6 deletions(-)
> > 
> > diff --git a/package/matchbox/Config.in b/package/matchbox/Config.in
> > index 7d06bb2..e58aaf2 100644
> > --- a/package/matchbox/Config.in
> > +++ b/package/matchbox/Config.in
> > @@ -1,5 +1,5 @@
> >  config BR2_PACKAGE_MATCHBOX
> > -	bool "MatchBox Window Manager"
> > +	bool "matchbox"
> 
>  Not so sure about this one... The upstream name is matchbox-window-manager, and
> it is useful information that it's a window manager. OTOH that's already in the
> comment right above it.

Well, even though I agree on principle:
  - "matchbox-window-manager" is a bit long
  - as you said, it's in the "Window Manager" section
  - the global stuff is named "matchbox", and I suppose users would
    expect to see just "matchbox"

>  So in the end I'm OK either way.

I can see if naming that package accordingly to the upstream name, and
if it proves to cumbersome, I'll just let it as it is now...

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

  reply	other threads:[~2015-04-24 16:50 UTC|newest]

Thread overview: 82+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-22 18:09 [Buildroot] [PATCH 0/30] matchbox: huge cleanups (branch yem/matchbox) Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 01/30] package/matchbox: make matchbox-lib a real package Yann E. MORIN
2015-04-23 21:29   ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 02/30] package/matchbox: make matchbox-wm " Yann E. MORIN
2015-04-23 21:35   ` Arnout Vandecappelle
2015-04-23 21:44     ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 03/30] package/matchbox-wm: drop useless commented options Yann E. MORIN
2015-04-23 21:33   ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 04/30] package/matchbox: make matchbox-panel a real package Yann E. MORIN
2015-04-23 21:36   ` Arnout Vandecappelle
2015-04-23 21:46   ` Arnout Vandecappelle
2015-04-23 21:47     ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 05/30] package/matchbox: make matchbox-desktop " Yann E. MORIN
2015-04-23 21:49   ` Arnout Vandecappelle
2015-04-24 17:37     ` Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 06/30] package/matchbox: make matchbox-common " Yann E. MORIN
2015-04-23 21:52   ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 07/30] package/matchbox: make matchbox-fakekey " Yann E. MORIN
2015-04-23 21:54   ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 08/30] package/matchbox: make matchbox-keyboard " Yann E. MORIN
2015-04-23 21:55   ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 09/30] package/matchbox: make match-startup-monitor " Yann E. MORIN
2015-04-23 22:00   ` Arnout Vandecappelle
2015-04-24 17:40     ` Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 10/30] package/matchbox: remove superfluous conditional includes Yann E. MORIN
2015-04-23 22:01   ` Arnout Vandecappelle
2015-04-24 17:43     ` Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 11/30] package/matchbox: point to new home Yann E. MORIN
2015-04-23  8:58   ` Alexandre Belloni
2015-04-23 16:38     ` Yann E. MORIN
2015-04-23 22:01   ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 12/30] package/matchbox: get rid of intermediate non-package matchbox Yann E. MORIN
2015-04-23 22:27   ` Arnout Vandecappelle
2015-04-24 16:51     ` Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 13/30] package/matchbox: fix option prompts Yann E. MORIN
2015-04-23 22:26   ` Arnout Vandecappelle
2015-04-24 16:50     ` Yann E. MORIN [this message]
2015-04-22 18:09 ` [Buildroot] [PATCH 14/30] package/matchbox-common: fix dependencies Yann E. MORIN
2015-05-01 20:01   ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 15/30] package/matchbox-common: break dependency chain for display Yann E. MORIN
2015-05-01 20:02   ` Arnout Vandecappelle
2015-05-02 12:00     ` Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 16/30] package/matchbox-fakekey: add missing dependency Yann E. MORIN
2015-05-01 20:07   ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 17/30] package/matchbox-startup-monitor: add misisng dependency Yann E. MORIN
2015-05-01 20:19   ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 18/30] package/matchbox: finally add prompts to all sub-packages Yann E. MORIN
2015-05-01 20:23   ` Arnout Vandecappelle
2015-05-02 12:28     ` Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 19/30] package/matchbox: enable support for the Xsession manager Yann E. MORIN
2015-05-01 20:28   ` Arnout Vandecappelle
2015-05-02 13:10     ` Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 20/30] package/matchbox: cleanup handling of support for compositing Yann E. MORIN
2015-05-01 20:29   ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 21/30] package/matchbox: make it a menu Yann E. MORIN
2015-05-01 20:32   ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 22/30] package/matchbox: forcibly disable support for compositing Yann E. MORIN
2015-05-01 20:31   ` Arnout Vandecappelle
2015-05-02 13:04     ` Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 23/30] package/matchbox: drop useless dependencies Yann E. MORIN
2015-05-01 20:41   ` Arnout Vandecappelle
2015-05-01 20:58     ` Arnout Vandecappelle
2015-05-02 13:40       ` Yann E. MORIN
2015-05-02 14:12         ` Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 24/30] package/matchbox-keyboard: enforce disabling support for cairo Yann E. MORIN
2015-05-01 20:45   ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 25/30] package/matchbox-common: explicitly disable PDA folders when not selected Yann E. MORIN
2015-05-01 20:44   ` Arnout Vandecappelle
2015-04-22 18:09 ` [Buildroot] [PATCH 26/30] package/matchbox: wrap long lines Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 27/30] package/matchbox: xlib-libXcursor is an optional dependency Yann E. MORIN
2015-05-01 21:02   ` Arnout Vandecappelle
2015-05-02 14:33     ` Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 28/30] package/matchbox-panel: add option to use ACPI for the battery applet Yann E. MORIN
2015-05-01 21:39   ` Arnout Vandecappelle
2015-05-02 15:22     ` Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 29/30] package/matchbox-panel: add support for dnotify Yann E. MORIN
2015-05-01 21:44   ` Arnout Vandecappelle
2015-05-02 15:23     ` Yann E. MORIN
2015-04-22 18:09 ` [Buildroot] [PATCH 30/30] package/matchbox-panel: fix WiFi monitoring Yann E. MORIN
2015-05-01 21:45   ` Arnout Vandecappelle
2015-04-23 21:24 ` [Buildroot] [PATCH 0/30] matchbox: huge cleanups (branch yem/matchbox) Arnout Vandecappelle
2015-04-23 21:27   ` Yann E. MORIN

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150424165040.GA4244@free.fr \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox