Bug 14745 - xorg/util/macros - Add macros to check fvisbility support and generic warn cflags
Summary: xorg/util/macros - Add macros to check fvisbility support and generic warn cf...
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Server/General (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Xorg Project Team
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2008-02-28 22:59 UTC by Paulo César Pereira de Andrade
Modified: 2008-11-29 20:48 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
0001-Add-XORG_CHECK_SYMBOLS-and-XORG_WARN_CFLAGS-macros.patch (3.31 KB, patch)
2008-02-28 22:59 UTC, Paulo César Pereira de Andrade
no flags Details | Splinter Review
0001-Add-XORG_CHECK_SYMBOLS-and-XORG_WARN_CFLAGS-macros.patch (3.34 KB, patch)
2008-03-18 18:07 UTC, Paulo César Pereira de Andrade
no flags Details | Splinter Review
0001-Add-XORG_CHECK_SYMBOLS-and-XORG_WARN_CFLAGS-macros.patch (3.47 KB, patch)
2008-03-19 19:52 UTC, Paulo César Pereira de Andrade
no flags Details | Splinter Review
0001-Add-XORG_CHECK_SYMBOLS-and-XORG_WARN_CFLAGS-macros.patch (3.49 KB, patch)
2008-05-10 15:49 UTC, Paulo César Pereira de Andrade
no flags Details | Splinter Review

Description Paulo César Pereira de Andrade 2008-02-28 22:59:18 UTC
Created attachment 14706 [details] [review]
0001-Add-XORG_CHECK_SYMBOLS-and-XORG_WARN_CFLAGS-macros.patch

I did not add the other modified files, including
configure.ac's and Makefile.am's a I would like to
know how people would prefer, i.e. have modules
being able to be compiled with hidden symbols in a
"by directory" basis, or a global flag?

  Other macros can be useful, but I added in this
patch only the minimally necessary ones. Other macros
would be useful to avoid too much cut&pasting and
then having several files with different versions
of the same original shell/m4 code.


  Later I will check the symbols in the new "compilable"
modules. But I can tell in advance that there are some
that compile, but still have missing symbols, i.e. instant
apparent sever crash for a new user, possibly with an
unusable console.
Comment 1 Paulo César Pereira de Andrade 2008-03-18 18:07:40 UTC
Created attachment 15276 [details] [review]
0001-Add-XORG_CHECK_SYMBOLS-and-XORG_WARN_CFLAGS-macros.patch

  New version that applies cleanly to git master.
Comment 2 Paulo César Pereira de Andrade 2008-03-19 19:52:34 UTC
Created attachment 15325 [details] [review]
0001-Add-XORG_CHECK_SYMBOLS-and-XORG_WARN_CFLAGS-macros.patch

  This updated version checks of #defined __GNUC__ instead
of including Xfuncproto.h. It is not required to check
gcc version as it is "trying" to compile the small code
to check for visibility support.

  Not including Xfuncproto.h is required, because it would
include the already installed Xfuncproto.h in the default
search path, that may not match the new Xfuncproto.h.
Comment 3 Paulo César Pereira de Andrade 2008-05-10 15:49:23 UTC
Created attachment 16467 [details] [review]
0001-Add-XORG_CHECK_SYMBOLS-and-XORG_WARN_CFLAGS-macros.patch

  Remaking patch with a minor modification.

  To be cached, variables must contain "_cv_", so
corrected accordingly the variable.
Comment 4 Paulo César Pereira de Andrade 2008-11-29 20:48:58 UTC
visibility was add to XORG_CFLAGS, and
compiler warnings macro to util-macros 1.2.0.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.