From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Mon, 27 Oct 2014 17:59:51 +0100 Subject: [Buildroot] [PATCH 2/2] polarssl: disable assembly for more scenarios In-Reply-To: <544E789C.8090402@zacarias.com.ar> References: <1414418838-3487-1-git-send-email-gustavo@zacarias.com.ar> <1414418838-3487-2-git-send-email-gustavo@zacarias.com.ar> <20141027174859.45788b19@free-electrons.com> <544E789C.8090402@zacarias.com.ar> Message-ID: <20141027175951.3f27a70b@free-electrons.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Dear Gustavo Zacarias, On Mon, 27 Oct 2014 13:53:48 -0300, Gustavo Zacarias wrote: > > It's not really great to have to build things differently debugging on > > BR2_ENABLE_DEBUG, because the prompt of BR2_ENABLE_DEBUG is "build > > packages with debugging symbols". So the user may expect that we build > > the same set of features, just with debugging symbols. > > > > Though I agree that with BR2_ENABLE_DEBUG also activating > > --enable-debug, a lot more than just debugging symbols actually happens. > > > > Therefore, committed, thanks! > > Well i'm not very happy to build things differently (asm vs no asm) if > debugging is on since it can change the result and make it pointless, > but it wouldn't be too good to disable asm in general for thumb either > so i threw the dice on the lesser evil. Yeah, I'm fine with the trade-off as well. Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com