[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