Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 455310 - x11-libs/libva-1.1.0 with dev-libs/wayland-1.0.x: va_display_wayland.c:95:9: error: too many arguments to function 'wl_display_get_fd'
Summary: x11-libs/libva-1.1.0 with dev-libs/wayland-1.0.x: va_display_wayland.c:95:9: ...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Alexis Ballier
URL: http://cgit.freedesktop.org/libva/com...
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2013-02-03 16:39 UTC by Dennis Schridde
Modified: 2013-06-19 02:24 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log (build.log,36.07 KB, text/plain)
2013-02-03 16:39 UTC, Dennis Schridde
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dennis Schridde 2013-02-03 16:39:58 UTC
Created attachment 337818 [details]
build.log

va_display_wayland.c: In function 'display_handle_global':
va_display_wayland.c:59:23: warning: assignment makes pointer from integer without a cast [enabled by default]
va_display_wayland.c:61:18: warning: assignment makes pointer from integer without a cast [enabled by default]
va_display_wayland.c: In function 'va_open_display_wayland':
va_display_wayland.c:95:9: error: too many arguments to function 'wl_display_get_fd'
In file included from ../../va/wayland/va_wayland.h:31:0,
                 from va_display_wayland.c:28:
/usr/include/wayland-client.h:141:5: note: declared here

I assume libva needs a versioned dependency on <dev-libs/wayland-1.0, since I have dev-libs/wayland-1.0.3 installed and libva built successfully some time ago.
Comment 1 Dennis Schridde 2013-02-04 22:07:25 UTC
The first two errors seem to stem from wl_display_bind missing in wayland-1.0 - apparently there is no undeclared-function warning...?
Comment 2 Dennis Schridde 2013-02-04 23:57:02 UTC
This upstream commit fixes it: http://cgit.freedesktop.org/libva/commit/?id=b1cf861a087234eb797c402040403a538adbe0cc

The change to configure.ac also confirms that the version of wayland required for libva-1.1.0 was 0.95.0.
Comment 3 Dennis Schridde 2013-02-14 18:48:25 UTC
Could someone please adjust the DEPEND of libva-1.1.0 to ">=dev-libs/wayland-0.95 <dev-libs/wayland-1.0" and add a -r1 version with DEPEND=">=dev-libs/wayland-1.0" and the mentioned patch?

I just tested it using a pre_src_prepare env hack and it works.
Comment 4 Alexis Ballier gentoo-dev 2013-02-14 19:07:45 UTC
applied, thanks!
Comment 5 Dennis Schridde 2013-02-14 19:35:50 UTC
I hate to say it, but your patch to the ebuild looks wrong.

Imagine I build libva against wayland-1 and then downgrade to wayland-0.95 for whatever reason. Now libva will try to use the wayland-1 ABI against the wayland-0.95 library. Which will fail, because they are ABI incompatible.

I still think that the correct solution is using DEPEND.
Comment 6 Alexis Ballier gentoo-dev 2013-02-14 19:59:06 UTC
(In reply to comment #5)
> I hate to say it, but your patch to the ebuild looks wrong.
> 
> Imagine I build libva against wayland-1 and then downgrade to wayland-0.95
> for whatever reason. Now libva will try to use the wayland-1 ABI against the
> wayland-0.95 library. Which will fail, because they are ABI incompatible.
> 
> I still think that the correct solution is using DEPEND.

hu?
so wayland changes its abi without changing soname ? if that's the case, then its a problem within wayland upstream

if that's not the case, then libva will need to be rebuilt and will not try to use the wayland-1 abi, so i dont see any problem here...
Comment 7 Alexis Ballier gentoo-dev 2013-06-19 02:24:28 UTC
(In reply to Alexis Ballier from comment #6)
> (In reply to comment #5)
> > I hate to say it, but your patch to the ebuild looks wrong.
> > 
> > Imagine I build libva against wayland-1 and then downgrade to wayland-0.95
> > for whatever reason. Now libva will try to use the wayland-1 ABI against the
> > wayland-0.95 library. Which will fail, because they are ABI incompatible.
> > 
> > I still think that the correct solution is using DEPEND.
> 
> hu?
> so wayland changes its abi without changing soname ? if that's the case,
> then its a problem within wayland upstream
> 
> if that's not the case, then libva will need to be rebuilt and will not try
> to use the wayland-1 abi, so i dont see any problem here...