Re: [Opensc-devel] [PATCH x2] support VPATH / out-of-source-tree builds

[Anthony F. <ant...@gm...>]

Ludovic --

Thanks for the reply.  Sorry that it took me this long to get back to
you -- but I think I understand why I'm seeing the problem (and why
you aren't).

On Fri, Mar 29, 2013 at 4:09 AM, Ludovic Rousseau
<lud...@gm...> wrote:
>
> 2013/3/28 Anthony Foiani <ant...@gm...>:
> > (It might be possible to use autoconf to sub in the right paths.  I
> > don't know autoconf & friends well enough to say.)
>
> Fixed in revision 6579.

Looks good, thanks!

> I can't reproduce the problem [unable to build pcsc-wirecheck & friends].
>
> I do
> $ mkdir a
> $ cd a
> $ ../configure
> $ make
> and have no error (after the fix in revision 6579. See above)
>
> Please give me the commands you use to get the issue.

You can find it attached, but first, my analysis:

Try it again with a fresh svn checkout with only a bootstrap done on
the native machine.

If you've built from the source tree for the *native* arch, at least
once, then you're golden -- because pcsc-wirecheck-dist.c will be
created with the native executable (even if its in "noinst_", it's
still built for the "--host" arch).  The makefile tests for existance
(not regular "newer-than"), and if it exists, it just "touch"es it, so
no executable is needed.

If you try to cross-compile from a fresh checkout (as I did), then you
run into this issue.  First, the pcsc-wirecheck-dist.c file is not
created (because the local/native makefile tries to run a binary that
is build for the host); second, if you add in my patch to use native
CC directly for pcsc-wirechec-gen.c, it is created in the build tree,
not the source tree -- hence the need for all the -I and -isystem
garbage.  :(

> > (And even if you don't care for the rest of the patch, you include two
> > ".in" files on a list of _SOURCES in there somewhere; that should
> > probably be fixed regardless of the rest of this patch.)
>
> Fixed in revision 6580.

Excellent.

Best regards,
Anthony Foiani