All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick <kpa_info@yahoo.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v3 0/5] Python 3, simple, external package support
Date: Wed, 13 Feb 2013 13:33:56 +0100	[thread overview]
Message-ID: <511B8834.5050104@yahoo.fr> (raw)
In-Reply-To: <511B68EF.2000409@yahoo.fr>

On 02/13/2013 11:20 AM, Patrick wrote:
> Hi Maxime,
>
> On 02/13/2013 10:40 AM, Maxime Ripard wrote:
>> Hi Patrick,
>>
>> I don't get why this python-common directory is needed. If the two
>> python are mutually exclusive, why can't we just declare the PYTHON
>> variable in their respective package makefiles?
>
> You are right it could be done as suggested.
>
> The Config.in file is
>> useless as well, just make a depends on !PYTHON3 in python, and !PYTHON
>> in python3, like Thomas suggested.
>
> As  I replied to Thomas this result in strange behaviour of the menu.
> When selecting/deselecting one of the python entry the menu do not
> handle properly the "circular reference" and it's almost unusable.
>

Maxim, Thomas,

Does it seems acceptable to you if I keep the menu like proposed in this 
patch series (including the file python-common/Config.in) but move the 
PYTHON variable definition in both python.mk and python3.mk and remove 
the python-common.mk file ?

Comments are welcomes

Patrick

      reply	other threads:[~2013-02-13 12:33 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-13  9:20 [Buildroot] [PATCH v3 0/5] Python 3, simple, external package support Patrick Gerber
2013-02-13  9:20 ` [Buildroot] [PATCH v3 1/5] Python menu reorganization Patrick Gerber
2013-02-13  9:20 ` [Buildroot] [PATCH v3 2/5] Add python3 config directory simlink Patrick Gerber
2013-02-13  9:33   ` Thomas Petazzoni
2013-02-13 10:07     ` Patrick
2013-02-13 10:13       ` Thomas Petazzoni
2013-02-13 10:20         ` Patrick
2013-02-13  9:20 ` [Buildroot] [PATCH v3 3/5] Add common declaration to ease the support of both version Patrick Gerber
2013-02-13  9:20 ` [Buildroot] [PATCH v3 4/5] python-bottle: Convert package to be used with python2 or python3 Patrick Gerber
2013-02-13  9:20 ` [Buildroot] [PATCH v3 5/5] python-serial: " Patrick Gerber
2013-02-13  9:40 ` [Buildroot] [PATCH v3 0/5] Python 3, simple, external package support Maxime Ripard
2013-02-13 10:20   ` Patrick
2013-02-13 12:33     ` Patrick [this message]

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=511B8834.5050104@yahoo.fr \
    --to=kpa_info@yahoo.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.