summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Prepare release 1.7.0 (and specification 0.20)dbus-1.7.0Simon McVittie2013-02-223-12/+20
|
* Include config.h as the first thing in every .c fileSimon McVittie2013-02-224-0/+6
| | | | | | | | | ...except for CheckForAbstractSockets.c, which runs before config.h is generated, and sd-daemon.c, which is externally-maintained. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59971 Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Reviewed-by: Colin Walters <walters@verbum.org>
* Don't warn for functions deprecated since GLib 2.26Simon McVittie2013-02-221-0/+3
| | | | | | | | Also warn if we inadvertently use a function introduced since then. Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59971 Reviewed-by: Colin Walters <walters@verbum.org>
* Add support for systems without syslog.hMatt Fischer2013-02-211-3/+12
| | | | | | | | | This patch disables the use of syslog for systems which do not have it, such as QNX. Log messages are still printed to stderr. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=61176 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Fix inotify usage for QNXMatt Fischer2013-02-211-0/+4
| | | | | | | | | QNX's copy of sys/inotify.h is broken, and doesn't include stdint.h even though it refers to types from it. Therefore, it must be included manually. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=61176 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* NEWSSimon McVittie2013-02-181-0/+6
|
* Eliminate unwanted whitespace from the man pages' XML sourceSimon McVittie2013-02-186-28/+22
| | | | | | | | | | As demanded by the git commit hook set up by autogen.sh, this eliminates trailing whitespace on each line, and blank lines at EOF. We might as well do this now, since every line in these files has changed anyway. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59805 Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
* Remove doclifter "signature" from Docbook man pages' sourceSimon McVittie2013-02-186-7/+1
| | | | | | | | | | | This no longer serves any purpose, and might mislead contributors into thinking that this XML is not the source for the man pages. (The man(7)-formatted man pages used to be the canonical source for the XML, but now it's the other way round.) Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59805 Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
* Removed precarious cross compile shell script.Ralf Habacker2013-02-181-110/+0
| | | | | | | | | | | | | CMake provides a standardized way to cross compile packages by using -DCMAKE_TOOLCHAIN_FILE at configure time. Also recent distributions like opensuse provides up to date native mingw binary packages and cross compile packages which reduces the cross compile setup to package installation and setup of a cross tool chain file as documented at http://www.vtk.org/Wiki/CMake_Cross_Compiling. https://bugs.freedesktop.org/show_bug.cgi?id=59733 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* More NEWSSimon McVittie2013-02-141-0/+9
|
* Add @DBUS_VERSION@ to the generated man pagesSimon McVittie2013-02-145-0/+5
| | | | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59805 Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de> [split out of previous patch -smcv] Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Turn all man pages' source into configure-generated filesSimon McVittie2013-02-149-11/+16
| | | | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59805 Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de> [dropped whitespace changes per Ralf's review -smcv] Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Use Docbook XML as the source for all man pagesSimon McVittie2013-02-149-1342/+41
| | | | | | | | This means we no longer need man2html, which is nice. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59805 Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de> Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Fill in a manual and source for all man pagesSimon McVittie2013-02-146-0/+13
| | | | | | | | | I only filled in a version for dbus-daemon, whose XML is already generated by configure. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59805 Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de> Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Generate man pages from xml docbook sources for cmake buildsystem.Ralf Habacker2013-02-141-9/+27
| | | | | | [removed commented line -smcv] Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59805 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Updated man docbook xml sources from man page source using doclifter.Ralf Habacker2013-02-147-282/+653
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59805 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Moved docbook sources used by cmake into doc subdir and adapted cmake build ↵Ralf Habacker2013-02-145-8/+10
| | | | | | | system. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59805 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* More NEWS for 1.7Simon McVittie2013-02-121-0/+14
|
* Merge branch 'dbus-1.6'Simon McVittie2013-02-121-0/+3
|\
| * NEWS for 1.6Simon McVittie2013-02-121-0/+3
| |
| * Create missing directories in cmake <build-root>/bus/session.d and ↵Ralf Habacker2013-01-151-0/+3
| | | | | | | | | | | | | | <build-root>/bus/system.d Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41319 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* | Add poll constants for QNXMatt Fischer2013-02-121-0/+13
| | | | | | | | | | | | | | | | | | The QNX operating system uses different values for its poll constants, so they must be added into dbus-sysdeps.h in order for poll() to work correctly. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=60339 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* | Update sd-daemon.[ch] from systemdSimon McVittie2013-02-123-45/+171
| | | | | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=60681
* | Fixed cmake cross compile timestamp creating.Ralf Habacker2013-02-121-6/+3
| | | | | | | | | | | | | | We only need to distinct "Windows" from unix like systems Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59733 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* | Uses cmake provided expat find package.Ralf Habacker2013-02-122-68/+8
| | | | | | | | | | | | | | | | The cmake provided expat find package is more up to date. There is no need to maintain an additional one. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59733 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* | Merge <servicehelper> from included config fileKrzysztof Konopko2013-02-121-1/+8
| | | | | | | | | | | | | | | | | | | | | | <servicehelper> is not supported in the included config file, i. e. it's not merged in merge_included(). There's clearly no reason it shouldn't be supported in the included config file along with <user>, <type> and others. It's quite reasonable for a client willing to override the default servicehelper, e. g. in system-local.conf. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=51560 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* | build: Dist autogen.shColin Walters2013-02-051-0/+1
| | | | | | | | | | | | | | For convenience of people who have to patch the autotools. See also http://people.gnome.org/~walters/docs/build-api.txt https://bugs.freedesktop.org/show_bug.cgi?id=60330
* | CMake build system fix: Lets check for xmlto doc book generator first.Ralf Habacker2013-01-291-9/+7
| | | | | | | | | | | | | | | | | | | | We shouldn't try to build the documentation with meinproc *and* xmlto. Prefer xmlto, since it's also the one we use under Autotools. We still need to support meinproc as a fallback, because xmlto isn't available on Windows. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59733 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* | Fixed cmake warning related to WIN32 macro when configuring on cygwinRalf Habacker2013-01-151-0/+3
| | | | | | | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59401 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* | Create missing directories in cmake <build-root>/bus/session.d and ↵Ralf Habacker2013-01-151-0/+3
| | | | | | | | | | | | | | <build-root>/bus/system.d Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41319 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* | NEWSSimon McVittie2012-11-191-13/+12
| | | | | | | | | | | | As well as documenting recent changes, this sorts out some divergence between the master and dbus-1.6 versions of NEWS, so the 1.6 entries are the same as in the corresponding releases.
* | dbus-sysdeps-pthread.c: don't fail if !HAVE_MONOTONIC_CLOCK under -Werror=unusedSimon McVittie2012-11-191-0/+2
| | | | | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=47239
* | configure: redo pthread check to check for more thingsSimon McVittie2012-11-191-16/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In principle, anything in the pthread namespace might either be in the platform-specific thread library (libpthread or libpthreads or libthreads or ...), or in libc. In particular, it seems that pthread_mutexattr_init and pthread_mutexattr_settype are in libpthread, not libc, on Linux. We previously didn't (intentionally) look for them in libpthread, only in libc; so this check deserved to fail. However, a faulty configure check for pthread_cond_timedwait worked around this on Linux by checking for -lpthread and adding it to THREAD_LIBS if pthread_cond_timedwait *was* found in libc (even though that behaviour makes no sense). The practical impact was that D-Bus would fail to compile on platforms where pthread_cond_timedwait is in a special threading library that is not linked by default, and at least one of (pthread_mutexattr_init, pthread_mutexattr_settype) is also in a special threading library. This is the case on at least OpenBSD (fd.o #54416). So far I've only added checks for the new symbols introduced by using recursive pthreads mutexes. If we get reports of compilation failures on weird platforms, we can check for more symbols. Also clarify the indentation, which was turning into quite a mess, and use AS_IF instead of if/elif/else/fi in accordance with Autoconf best-practice. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=47239 Reviewed-by: Colin Walters <walters@verbum.org>
* | Don't include generated documentation in the tarballSimon McVittie2012-11-131-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Building it unconditionally causes problems for minimal installations (OSTree), and building it opportunistically means the tarball isn't guaranteed to contain it, depending who releases libdbus and which packages they happen to have installed at the time. If this documentation is important enough that we need to ship it precompiled in tarballs, we should guarantee it; or if it isn't important enough to justify that, we should just drop it. I don't think we really need it in the tarballs at all: most users will get their libdbus from a binary distribution (in which case I expect the distribution's dbus maintainers to set appropriate build-dependencies), and those who build from source can either install xmlto, read the documentation on our website, or at worst, read the source XML. (We don't put the Doxygen-generated API reference HTML in the tarball either, and I haven't heard any complaints.) Bug: https://bugs.freedesktop.org/show_bug.cgi?id=55426 Reviewed-by: Colin Walters <walters@verbum.org>
* | Remove redundant close() callsMichel HERMIER2012-11-091-6/+3
| | | | | | | | | | | | | | | | The dup2() calls immediately afterwards will close the "destination" fd if necessary. [commit message added -smcv] Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* | Merge branch 'dbus-1.6'Simon McVittie2012-11-095-10/+39
|\ \ | |/ | | | | | | Conflicts: NEWS
| * NEWSSimon McVittie2012-11-091-1/+7
| |
| * Don't leak temporary fds pointing to /dev/nullMichel HERMIER2012-11-092-0/+2
| | | | | | | | | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=56927 [commit message added -smcv] Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
| * Fix building with newer ValgrindArun Raghavan2012-11-092-10/+10
| | | | | | | | | | | | | | | | | | Newer valgrind (tried with 3.8.0) defines macros so that a terminating semi-colon is required. This fixes usage to follow that convention. [edited to remove comments that are no longer useful -smcv] Bug: https://bugs.freedesktop.org/show_bug.cgi?id=55932 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* | Use InterlockedExchange to get a full memory barrier on WindowsSimon McVittie2012-11-091-2/+12
| | | | | | | | | | | | | | See the bug for extensive discussion. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41423 Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
* | include README.valgrind in tarballsSimon McVittie2012-11-091-0/+1
| |
* | Add documentation on running clients with ValgrindArun Raghavan2012-11-091-0/+24
| | | | | | | | | | | | | | | | Quick documentation on how Valgrind can be run in clients without triggering false positives. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=55933 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* | build: Ensure docs are enabled for distcheckColin Walters2012-11-091-0/+1
| | | | | | | | | | | | | | https://bugs.freedesktop.org/show_bug.cgi?id=55426 Signed-off-by: Colin Walters <walters@verbum.org> Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* | spec: fix command name REJECT -> REJECTEDPavel Strashkin2012-11-021-5/+5
| | | | | | | | | | Signed-off-by: Pavel Strashkin <pavel.strashkin@nexenta.com> Signed-off-by: Colin Walters <walters@verbum.org>
* | dbus-spawn: fix spacing mistake in commentRay Strode2012-10-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | commit 90f939f155bd120f44ff3906296707a6c00cd462 had two problems. 1) a small whitespace error in the added comment 2) the wrong bug reference at the bottom I'm using 1) as an excuse to add additional commit for the sake of 2). https://bugs.freedesktop.org/show_bug.cgi?id=56043
* | dbus-spawn: set SIGPIPE to SIG_IGN before activating servicesRay Strode2012-10-161-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | dbus's service activation code sets SIGPIPE to SIG_DFL as a sort of poor man's prctl(... PR_SET_PDEATHSIG) to detect when the parent goes away. It neglects to reignore the infamous signal before performing activation, however. This means if, for instance, journald is restarted all services activated after it will die with SIGPIPE when logging messages unless they explicitly ignore SIGPIPE themselves. This commit changes dbus's service activation code to correctly ignore SIGPIPE to protect activated services from a gruesome, premature death. Reviewed-by: Lennart Poettering <lennart@poettering.net> https://bugzilla.redhat.com/show_bug.cgi?id=839258
* | Set correct address when using --address=systemd:Simon Peeters2012-10-083-13/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When dbus gets launched through systemd, we need to create an address string based on the sockets passed. The _dbus_append_addres_from_socket() function is responsible for extracting the address information from the file-descriptor and formatting it in a dbus friendly way. This fixes bus activation when running dbus under a systemd session. https://bugs.freedesktop.org/show_bug.cgi?id=50962 Signed-off-by: Simon Peeters <peeters.simon@gmail.com>
* | Merge remote-tracking branch 'origin/master'Simon McVittie2012-10-041-3/+4
|\ \
| * | build: Make --disable-xml-docs build work againColin Walters2012-10-021-3/+4
| | | | | | | | | | | | | | | | | | | | | We can't build the .html files without xmlto, so don't add them to a dist_ variable. https://bugs.freedesktop.org/show_bug.cgi?id=55426
* | | Merge branch 'dbus-1.6'Simon McVittie2012-10-042-1/+5
|\ \ \ | |/ / |/| / | |/ | | | | | | Conflicts: NEWS bus/activation-helper.c configure.ac
View Lorry Controller Status