Hei Michael, 

Am 2013-03-19 16:32, schrieb Michael Olbrich:
> On Thu, Mar 14, 2013 at 01:16:36PM +0100, Alexander Dahl wrote:
>> We use -Wcast-align on ARMv5 platform and libmodbus3 sets -Werror
>> which leads to a failing build in one case. This patch fixes the
>> alignment problem. Upstream pull requested.
> 
> Hmm, this is just unit test stuff, right? We don't actually need this.

Yes, I think so. The targetinstall rule does not install these binaries
anyway.

> Maybe we should do this instead:
> 
> LIBMODBUS3_MAKE_OPT   := -C src
> LIBMODBUS3_INSTALL_OPT        := -C src install

My colleague removed 'tests' from Makefile.am, I don't know which one
would be the best choice.

Greets
Alex

-- 
»With the first link, the chain is forged. The first speech censured,
the first thought forbidden, the first freedom denied, chains us all
irrevocably.« (Jean-Luc Picard, quoting Judge Aaron Satie)
*** GnuPG-FP: 02C8 A590 7FE5 CA5F 3601  D1D5 8FBA 7744 CC87 10D0 ***

-- 
ptxdist mailing list
[email protected]

Reply via email to