Skip to contentSkip to conversation

libdrm: Remove xorg dependency for intel driver

PATCH New Bernd Kuhls 2 participants 6 messages Mar 11, 2014 – Mar 12, 2014

lore
Share & tools
lore
#0 lore raw

PATCH v5 1/6 libdrm: Remove xorg dependency for intel driver

---
 package/libdrm/Config.in |    1 -
 1 file changed, 1 deletion(-)
package/libdrm/Config.in +0 -1
index 7192e94..24ff7de 100644
@@ -13,7 +13,6 @@ menu "DRM Drivers"
 
 config BR2_PACKAGE_LIBDRM_INTEL
 	bool "intel"
-	depends on BR2_PACKAGE_XORG7
 	select BR2_PACKAGE_LIBATOMIC_OPS
 	select BR2_PACKAGE_XLIB_LIBPCIACCESS
 	depends on BR2_i386 || BR2_x86_64
Signature · 1 line
1.7.10.4
#1 lore raw

PATCH v5 1/6 libdrm: Remove xorg dependency for intel driver

Hi bernd,

Quoted 2 lines · Bernd Kuhls

config BR2_PACKAGE_LIBDRM_INTEL bool "intel" - depends on BR2_PACKAGE_XORG7

Remove that dependency will give a config error, so i was wrong with the dependency on BR2_PACKAGE_XORG7 in BR2_PACKAGE_MESA3D_DRI_DRIVER_I965 because the intel dri driver depends on the libdrm that depends on the xlib_pciaccess which is inside the xorg7 package.

#2 lore raw

PATCH v5 1/6 libdrm: Remove xorg dependency for intel driver

Sebastien Bourdelin <sebastien.bourdelin-4ysUXcep3aM1wj+D4I0NRVaTQe2KTcn/ @public.gmane.org> wrote in news:531F851F.40002 at savoirfairelinux.com:

Quoted 10 lines

Hi bernd,

Quoted 2 lines · Bernd Kuhls

config BR2_PACKAGE_LIBDRM_INTEL bool "intel" - depends on BR2_PACKAGE_XORG7

Remove that dependency will give a config error, so i was wrong with the dependency on BR2_PACKAGE_XORG7 in BR2_PACKAGE_MESA3D_DRI_DRIVER_I965 because the intel dri driver depends on the libdrm that depends on the xlib_pciaccess which is inside the xorg7 package.

Hi,

xlib_pciaccess itself does not depend on xorg stuff, I was able to compile the libdrm intel driver with patch v5 without activating xorg7.

Regards, Bernd

#3 lore raw

PATCH v5 1/6 libdrm: Remove xorg dependency for intel driver

Bernd Kuhls <[email protected]> wrote in news:3p65vaxf8m.ln2 at ID-313208.user.individual.net:

Quoted 2 lines

xlib_pciaccess itself does not depend on xorg stuff, I was able to compile the libdrm intel driver with patch v5 without activating xorg7.

PS: Should we also move libpciaccess out of the xorg section?

Regards, Bernd

#4 lore raw

PATCH v5 1/6 libdrm: Remove xorg dependency for intel driver

Hi bernd,

Quoted 5 lines · Bernd Kuhls

Bernd Kuhls <[email protected]> wrote in news:3p65vaxf8m.ln2 at ID-313208.user.individual.net:

Quoted 2 lines

xlib_pciaccess itself does not depend on xorg stuff, I was able to compile the libdrm intel driver with patch v5 without activating xorg7.

As far as i understand, the package XORG7 is a kind of a container which doesn't compile anything, it only adds global toolchain dependencies for all the xorg stuff.
Did you try to build the xlib_libpciaccess with a toolchain without this options ?

Quoted 1 line

PS: Should we also move libpciaccess out of the xorg section?

I dont think so, even if the libpciaccess is a generic PCI access library, it come as a part of the Xorg project and perhaps there is other xlib that doesn't require the toolchain options above so should we move them too ?
An other option could be to don't set dependencies on the XORG7 package and move the dependencies inside each sub-packages.

#5 lore raw

PATCH v5 1/6 libdrm: Remove xorg dependency for intel driver

Sebastien Bourdelin <[email protected]> wrote in news:5320096D.1020900 at savoirfairelinux.com:

Quoted 8 lines
Quoted 3 lines
Quoted 3 lines

xlib_pciaccess itself does not depend on xorg stuff, I was able to compile the libdrm intel driver with patch v5 without activating xorg7.

As far as i understand, the package XORG7 is a kind of a container which doesn't compile anything, it only adds global toolchain dependencies for all the xorg stuff.
Did you try to build the xlib_libpciaccess with a toolchain without this options ?

Hi,

no, and I do not think I will try to do se because I would like to see my patches committed to buildroot in the nearby future;) I will post a new series without this patch.

Regards, Bernd