The policy in Apertis is to increment library revision when there is
a change in the code, but that no interface were added or removed.

Signed-off-by: Frédéric Dalleau <frederic.dalleau@collabora.com>

When converting the D-Bus API to be autogenerated from XML, the signals were
left out.

Signed-off-by: Frédéric Dalleau <frederic.dalleau@collabora.com>

Take advantage of the new capabilities of gdbus-codegen to generate
only the interface information.

When doing more strict checking, GCC will complain and fail the
compilation.

Signed-off-by: Andrew Shadura <andrew.shadura@collabora.co.uk>
Reviewed-by: Frédéric Dalleau <frederic.dalleau@collabora.com>
Differential Revision: https://phabricator.apertis.org/D7745

check-dbus-services uses Ping method to check the availability
of services; we need to whitelist the method to make it work.

Apertis: T4426

Signed-off-by: Andrew Shadura <andrew.shadura@collabora.co.uk>
Reviewed-by: Frédéric Dalleau <frederic.dalleau@collabora.com>
Differential Revision: https://phabricator.apertis.org/D7744

Signed-off-by: Andrew Shadura <andrew.shadura@collabora.co.uk>
Reviewed-by: Emanuele Aina <emanuele.aina@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D7390

Due to a postinst failure, ACL for rhosydd isn't being created,
leading to rhosydd service failing to start. Temporarily work
this around by allowing anyone to run rhosydd.

Apertis: https://phabricator.apertis.org/T4401



Signed-off-by: Andrew Shadura <andrew.shadura@collabora.co.uk>
Reviewed-by: Emanuele Aina <emanuele.aina@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D7389

Signed-off-by: Héctor Orón Martínez <hector.oron@collabora.co.uk>

Signed-off-by: Héctor Orón Martínez <hector.oron@collabora.co.uk>

  Support usrmerge
  Apertis: https://phabricator.apertis.org/T398



Signed-off-by: Héctor Orón Martínez <hector.oron@collabora.co.uk>
Reviewed-by: Emanuele Aina <emanuele.aina@collabora.com>
Differential Revision: https://phabricator.apertis.org/D7236

This is similar to D6297 and D6298, but with the addition of
AX_SPLIT_VERSION to generate the two version.h headers.

Disable check-news: it is not compatible with autogenerating
version numbers in this way, and Apertis doesn't normally enforce
use of NEWS files.

Signed-off-by: Simon McVittie <smcv@collabora.com>
Reviewed-by: Emanuele Aina <emanuele.aina@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D6641

Signed-off-by: Simon McVittie <smcv@collabora.com>
Reviewed-by: Emanuele Aina <emanuele.aina@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D6640

If we're going to recommend Rhosydd as the exemplary package, we
should make sure it continues to be exemplary.

At compat level 10, the autoreconf and systemd addons are used by
default, so we don't need to invoke them explicitly or depend on them.
Similarly, we now do a parallel build by default.

Signed-off-by: Simon McVittie <smcv@collabora.com>
Reviewed-by: Emanuele Aina <emanuele.aina@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D6639

Signed-off-by: Simon McVittie <smcv@collabora.com>

Rhosydd only uses the SDK-API library canterbury-0.

Signed-off-by: Simon McVittie <smcv@collabora.com>

Differential Revision: https://phabricator.apertis.org/D6591

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Differential Revision: https://phabricator.apertis.org/D6005

This test won't get far without them.

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Differential Revision: https://phabricator.apertis.org/D6004

This removes some Makefile.am rules and simplifies the build system a
bit.

Signed-off-by: Philip Withnall <philip.withnall@collabora.co.uk>
Reviewed-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D4912

Mathieu Duponchelle authored 8 years ago and Andrej Shadura committed 8 years ago

The following issue types were raised by hotdoc:

* Invalid plural links: hotdoc does not know how to pluralize
  symbols such as `#RsdSubscription`.

* Invalid inter-documentation links: hotdoc does not yet install
  symbol databases, links to other documentation symbols are only
  possible when the link target is documented by gtk-doc.

* Invalid function links, for example when referring to a
  user-provided `main()` function.

Signed-off-by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>
Signed-off-by: Andrew Shadura <andrew.shadura@collabora.co.uk>
Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D4490

Mathieu Duponchelle authored 8 years ago and Andrej Shadura committed 8 years ago

And reduce our extensions requirements to the newly-consolidated
hotdoc packages.

+ Update depends in debian/control

Signed-off-by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>
Signed-off-by: Andrew Shadura <andrew.shadura@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D4491

 • Add apparmor rules to enable Apertis applications to talk to rhosydd.

Signed-off-by: Andrew Shadura <andrew.shadura@collabora.co.uk>

Madgula Rohan authored 8 years ago and Emanuele Aina committed 8 years ago

Any store or built-in applications should be able to communicate
with rhosydd service.

Signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D4296

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

+ Use the hotdoc autotools helpers

This requires using -Wno-portability in the call to AM_INIT_AUTOMAKE,
as discussed with Simon it's a reasonable requirement for using the
hotdoc.mk fragment Makefile.

The produced html is now cleanly installable and uninstallable.

+ Make documentation build parallel for croesor and rhosydd

The issue preventing this is fixed upstream, although in a
different manner than the linked github issue proposed.

+ Remove distribution of the generated documentation

Distributing build products in the source tarball is a bad practice,
and there's no compelling reason for doing this in Apertis libraries.

+ Use a sitemap

This means subpages do not need to be linked and updated in markdown
files.

+ Enable ‘smart index’

+ Enable devhelp books generation

+ Update the debian files

+ remove build-depends on hotdoc-dbus-extension, not sure why it
  was added in the first place.

Using this new feature of hotdoc means we no longer need to maintain
potentially huge lists of symbols in the markdown files.

As desired by Philip, we now pass the C sources to hotdoc in the
Makefile, which means the source list does not need updating.

However, combined with smart-index, this means every public symbol
is now exposed in the documentation.

Subsequent patches should filter out potentially undesirable files
or symbols, this requires more in-depth knowledge of the project
than I currently possess.

Signed-off-by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

Differential Revision: https://phabricator.apertis.org/D3784

This ignores:

+ The hotdoc depdir folder
+ The stamp files generated by the hotdoc fragment
+ The private folder for each hotdoc project
+ The output directory for each hotdoc project

Signed-off-by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

Differential Revision: https://phabricator.apertis.org/D3873

Signed-off-by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

Differential Revision: https://phabricator.apertis.org/D3872

Links to symbols use the following syntax: [](symbol-name), not
[symbol-name]().

Signed-off-by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

Differential Revision: https://phabricator.apertis.org/D3783

This is not yet supported by hotdoc.

Signed-off-by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

Differential Revision: https://phabricator.apertis.org/D3793

The syntax for linking to a recognized symbol is [](symbol-name),
not [symbol-name]().

Signed-off-by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

Differential Revision: https://phabricator.apertis.org/D3782

This updates it to the latest recommended style. See the commit for
rationale about the change.

Signed-off-by: Philip Withnall <philip.withnall@collabora.co.uk>
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D3927