[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: Move KDE_VERSION* to a header by themselves.
From:       Marc Mutz <Marc.Mutz () uni-bielefeld ! de>
Date:       2002-02-23 20:19:02
[Download RAW message or body]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Saturday 23 February 2002 15:58, Rob Kaper wrote:
> On Saturday 23 February 2002 15:38, Marc Mutz wrote:
> > What's wrong with kdeconfig.h or kdeversion.h?
>
> It is not as consistent as kde.h, we name most of our classes after
> the Q equivalents, so this makes sense for filenames as well.

Have you _looked_ at qt.h? It includes _all_ qt header files. That has 
nothing to do with what I proposed.

The qt (approximate) equivalent of what I propose is qglobal.h or 
qconfig.h, both of which we can't use since s/q/k/ gives already 
existing header files in both cases.

> Furthermore, this belongs to the global KDE namespace, and to me a
> file named kde(.h) makes a lot of sense for that as well.

The way I see it:

- - kde.h would be OK for me, but as you implied, people'd consider this 
the kde equivalent of qt.h, which it is not.
- - kdeconfig.h would be preferable if we had something more than the 
version defines to put there.
- - kdeversion.h is preferable if we only put the version stuff there, 
since in this case the name carries the exact semantics of the contents

Marc

PS: To stress that kde{,-}{config,version}.h is reasonable choice, I 
attach two files (note the artsversion.h :-)

- -- 
Marc Mutz <mutz@kde.org>
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (GNU/Linux)
Comment: For info see http://www.gnupg.org

iD8DBQE8d/k33oWD+L2/6DgRAoUyAKCLeghRDl9t8dK3+IWaJJqKNHo79ACbBF1M
AQ1YvCLokNa4FQwOTwRC0s4=
=rcJe
-----END PGP SIGNATURE-----

["locate-version-h" (text/plain)]

/opt/kde-HEAD/qt-copy/src/3rdparty/libjpeg/jversion.h
/opt/kde-HEAD/include/arts/artsversion.h
/opt/kde-HEAD/include/noatun/conversion.h
/usr/src/linux-2.4.8/arch/i386/math-emu/version.h
/usr/src/linux-2.4.8/drivers/net/cipe/version.h
/usr/src/linux-2.4.8/drivers/net/sk98lin/h/skversion.h
/usr/src/linux-2.4.8/drivers/scsi/aacraid/include/version.h
/usr/src/linux-2.4.8/pcmcia-cs-3.1.29/include/pcmcia/version.h
/usr/src/linux-2.4.8/include/linux/version.h
/usr/src/linux-2.4.8/include/pcmcia/version.h
/usr/src/linux-2.4.8/alsa-driver-0.5.11/include/version.h.in
/usr/src/linux-2.4.8/alsa-driver-0.5.11/include/version.h
/usr/X11R6/include/X11/fonts/codeconv/cconvversion.h
/usr/X11R6/include/X11/fonts/codeconv/xttversion.h
/usr/share/webmin/postfix/help/opt_mail_version.html
/usr/include/sp/lib/version.h
/usr/include/gnu/libc-version.h
/usr/include/linux/version.h
/usr/include/libxml2/libxml/xmlwin32version.h
/usr/include/libxml2/libxml/xmlversion.h
/usr/include/noatun/conversion.h

["locate-config-h" (text/plain)]

/opt/kde-HEAD/qt-copy/src/3rdparty/libjpeg/jconfig.h
/opt/kde-HEAD/qt-copy/tools/designer/designer/config.h
/opt/kde-HEAD/qt-copy/include/qconfig.h
/opt/kde-HEAD/include/kio/slaveconfig.h
/opt/kde-HEAD/include/arts/mcopconfig.h
/opt/kde-HEAD/include/kgame/kgamedialogconfig.h
/opt/kde-HEAD/include/mimelib/config.h
/opt/kde-HEAD/include/ksimpleconfig.h
/opt/kde-HEAD/include/ksconfig.h
/opt/kde-HEAD/include/kconfig.h
/usr/lib/qt2/include/qconfig.h
/usr/lib/glib/include/glibconfig.h
/usr/lib/perl5/5.6.1/i386-linux/CORE/config.h
/usr/lib/linuxconf/help.eng/shellmod/config.help
/usr/src/linux-2.4.8/drivers/net/cipe/config.h
/usr/src/linux-2.4.8/drivers/acpi/include/acconfig.h
/usr/src/linux-2.4.8/drivers/char/epcaconfig.h
/usr/src/linux-2.4.8/drivers/sound/sound_config.h
/usr/src/linux-2.4.8/pcmcia-cs-3.1.29/cardmgr/yacc_config.h
/usr/src/linux-2.4.8/3rdparty/sam9407/kconfig.h
/usr/src/linux-2.4.8/include/net/ipconfig.h
/usr/src/linux-2.4.8/include/linux/rhconfig.h
/usr/src/linux-2.4.8/include/linux/config.h
/usr/src/linux-2.4.8/alsa-driver-0.5.11/include/config.h.in
/usr/X11R6/include/DPS/dpsconfig.h
/usr/X11R6/include/X11/extensions/XKBconfig.h
/usr/share/doc/ntp-4.1.0/config.htm
/usr/share/autoconf/acconfig.h
/usr/include/sp/include/config.h
/usr/include/kio/slaveconfig.h
/usr/include/orb/orbit_config.h
/usr/include/qt2/qconfig.h
/usr/include/IIOP/IIOP-config.h
/usr/include/g++-3/stl_config.h
/usr/include/linux/rhconfig.h
/usr/include/linux/config.h
/usr/include/kpathsea/config.h
/usr/include/libxslt/xsltconfig.h
/usr/include/jconfig.h
/usr/include/_G_config.h
/usr/include/mimelib/config.h
/usr/include/libart_lgpl/art_config.h
/usr/include/ksimpleconfig.h
/usr/include/freetype2/freetype/config/ftconfig.h
/usr/include/ksconfig.h
/usr/include/python2.1/config.h
/usr/include/libexslt/exsltconfig.h
/usr/include/libgnome/gnome-config.h
/usr/include/kconfig.h
/usr/include/ORBitutil/orbit-os-config.h


[prev in list] [next in list] [prev in thread] [next in thread]