From: Carlos Santos <casantos@datacom.com.br>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v3 1/3] package/waf: add a blind Config.in.host
Date: Fri, 4 Jan 2019 08:05:41 -0200 (BRST) [thread overview]
Message-ID: <49851202.241713.1546596341117.JavaMail.zimbra@datacom.com.br> (raw)
In-Reply-To: <20190104095811.33a189e4@windsurf>
> From: "Thomas Petazzoni" <thomas.petazzoni@bootlin.com>
> To: "Carlos Santos" <casantos@datacom.com.br>
> Cc: "Yann Morin" <yann.morin.1998@free.fr>, "Peter Korsgaard" <peter@korsgaard.com>, "Mahyar Koshkouei"
> <mahyar.koshkouei@gmail.com>, "Thomas De Schampheleire" <thomas.de_schampheleire@nokia.com>, "buildroot"
> <buildroot@buildroot.org>, "Arnout Vandecappelle" <arnout@mind.be>
> Sent: Sexta-feira, 4 de janeiro de 2019 6:58:11
> Subject: Re: [Buildroot] [PATCH v3 1/3] package/waf: add a blind Config.in.host
> Hello Carlos,
>
> On Thu, 3 Jan 2019 21:34:03 -0200 (BRST), Carlos Santos wrote:
>
>> Well, I'm happy that my work helped you guys to analyze the pros and
>> cons of having Config.in.host for all host packages. Looks like it's
>> not a good idea, so I will mark the patches as refused in patchwork.
>
> I don't think we can conclude yet "it's not a good idea". At this
> point, my feeling is more: the consequences are worse than I initially
> expected, which requires some more thoughts on how we want to handle
> that.
>
> Again, what was your motivation for adding those Config.in.host options
> in the first place, specifically for waf ? Was it just about getting
> this whole discussion started, and use waf just as a simple first
> example ?
It started when I realized that the trailing slashes in WAF_SITE and
other host packages. Then Arnout said "we need Config.in.host" and I
decided to give it a try. It may be a silly explanation but sometimes
we do things just for fun. :-)
--
Carlos Santos (Casantos) - DATACOM, P&D
next prev parent reply other threads:[~2019-01-04 10:05 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-23 17:19 [Buildroot] [PATCH v3 1/3] package/waf: add a blind Config.in.host Carlos Santos
2018-12-23 17:19 ` [Buildroot] [PATCH v3 2/3] doc/manual: document the waf packages may need to select host-waf Carlos Santos
2018-12-26 21:31 ` Thomas Petazzoni
2018-12-23 17:19 ` [Buildroot] [PATCH v3 3/3] package/mpv: selec host-waf Carlos Santos
2018-12-26 21:30 ` [Buildroot] [PATCH v3 1/3] package/waf: add a blind Config.in.host Thomas Petazzoni
2018-12-26 21:55 ` Yann E. MORIN
2019-01-03 21:50 ` Thomas Petazzoni
2019-01-03 22:06 ` Yann E. MORIN
2019-01-03 22:14 ` Yann E. MORIN
2019-01-03 23:34 ` Carlos Santos
2019-01-04 8:58 ` Thomas Petazzoni
2019-01-04 10:05 ` Carlos Santos [this message]
2019-01-04 10:10 ` Thomas Petazzoni
2019-01-04 11:06 ` Arnout Vandecappelle
2019-01-04 11:51 ` 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=49851202.241713.1546596341117.JavaMail.zimbra@datacom.com.br \
--to=casantos@datacom.com.br \
--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.