[PATCH] Fix compilation with FreeRdp 1.1 and master

Hardening rdp.effort at gmail.com
Tue Dec 10 02:02:27 PST 2013


On 10/12/2013 09:25, Hardening wrote:
> On 10/12/2013 00:42, Kristian Høgsberg wrote:
>> On Mon, Dec 09, 2013 at 10:16:39PM +0100, Hardening wrote:
>>> The API to use remoteFx encoding has changed between master and 
>>> stable 1.1
>>> branch. This patch fixes compilation for both.
>>> Please note that the freerdp/version.h file is generated in the very 
>>> last versions
>>> of FreeRdp so be sure to update/install the last versions.
>> Yeah, this doesn't compile for me with an older FreeRdp because of the
>> missing freerdp/version.h.  If we're going to require a newer FreeRdp
>> for this to compile, can we just drop the #if?  Or is version.h
>> included by freedrp.h or something so we can avoid the version.h
>> #include?
>>
>>
> Until recent changes there were no safe way to know which version you 
> were compiling against.
> The FREERDP_VERSION_[MAJOR|MINOR|REVISION] macros have never been 
> accurate to detect this (not updated when they should). These macros 
> were included in the freerdp/freerdp.h and are now auto-generated in 
> the freerdp/version.h file.
>
> You're right i think i can have a better patch that will take care of 
> the installation that are missing the freerdp/version.h file.
Sorry to disappoint but my previous patch is the only way to deal 
between stable-1.1 (which is fact is a stabilization in progress branch) 
and master. The patch works only with last version of stable-1.1 or 
master (the config.h is here from a longer time in master). To mitigate 
things, nobody ships FreeRdp 1.1 for now.
Sorry about that, things have not been done cleanly in FreeRdp, but 
we'll take care of that in the future.

Regards



More information about the wayland-devel mailing list