Re: [PATCH libdrm v2 00/13] hide library symbols by default

2018-09-21 Thread Lucas De Marchi
On Thu, Sep 20, 2018 at 01:16:09PM +0100, Emil Velikov wrote: > On 14 September 2018 at 00:57, Lucas De Marchi > wrote: > > Rely on -fvisibility=hidden to hide the symbols. Previous version of > > this series applying only to drm_intel.so is > > > > Reviewed-by: Eric Engestrom > > > > bu

Re: [PATCH libdrm v2 00/13] hide library symbols by default

2018-09-20 Thread Emil Velikov
On 14 September 2018 at 00:57, Lucas De Marchi wrote: > Rely on -fvisibility=hidden to hide the symbols. Previous version of > this series applying only to drm_intel.so is > > Reviewed-by: Eric Engestrom > > but it's not included here since I changed the approach for the build > system ch

Re: [PATCH libdrm v2 00/13] hide library symbols by default

2018-09-19 Thread Lucas De Marchi
On Fri, Sep 14, 2018 at 10:28:46AM +0100, Eric Engestrom wrote: > On Thursday, 2018-09-13 16:57:11 -0700, Lucas De Marchi wrote: > > Rely on -fvisibility=hidden to hide the symbols. Previous version of > > this series applying only to drm_intel.so is > > > > Reviewed-by: Eric Engestrom > > >

Re: [PATCH libdrm v2 00/13] hide library symbols by default

2018-09-19 Thread Lucas De Marchi
On Fri, Sep 14, 2018 at 10:28:46AM +0100, Eric Engestrom wrote: > On Thursday, 2018-09-13 16:57:11 -0700, Lucas De Marchi wrote: > > Rely on -fvisibility=hidden to hide the symbols. Previous version of > > this series applying only to drm_intel.so is > > > > Reviewed-by: Eric Engestrom > > >

Re: [PATCH libdrm v2 00/13] hide library symbols by default

2018-09-14 Thread Eric Engestrom
On Thursday, 2018-09-13 16:57:11 -0700, Lucas De Marchi wrote: > Rely on -fvisibility=hidden to hide the symbols. Previous version of > this series applying only to drm_intel.so is > > Reviewed-by: Eric Engestrom > > but it's not included here since I changed the approach for the build > s

[PATCH libdrm v2 00/13] hide library symbols by default

2018-09-13 Thread Lucas De Marchi
Rely on -fvisibility=hidden to hide the symbols. Previous version of this series applying only to drm_intel.so is Reviewed-by: Eric Engestrom but it's not included here since I changed the approach for the build system change. drm_private can also be removed from other symbols but it pr