Wireshark-dev: Re: [Wireshark-dev] UseLemon.cmake call with lemon fullpath
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Joerg Mayer <jmayer@xxxxxxxxx>
Date: Mon, 11 Apr 2011 04:32:08 +0200
On Wed, Apr 06, 2011 at 09:40:54PM -0300, Jo?o Henrique Freitas wrote:
> wireshark (1.4.4) uses the script cmake/modules/UseLemon.cmake to call lemon.
> 
> I think it's need to call lemon tool with full path. Like this:
> 
> --- cmake/modules/UseLemon.cmake        2011-03-01 14:02:20.000000000 -0300
> +++ /home_local/usuario/w/UseLemon.cmake        2011-04-06
> 21:29:02.000000000 -0300
> @@ -12,7 +12,7 @@
>        ADD_CUSTOM_COMMAND(
>           OUTPUT
>           ${_out}
> -         COMMAND lemon
> +         COMMAND ${_lemonpardir}/lemon
>             t=${_lemonpardir}/lempar.c

No, that looks incorrect to me: In case of out of tree builds, the lemon
executable will not be located inside the source directory.

Are you encountering any problems without that patch? Are you doing in tree
builds?

ciao
   Jörg
-- 
Joerg Mayer                                           <jmayer@xxxxxxxxx>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.