Looks like the upstream version was still using an old version scheme.

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

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

It's preventing apertis-tests/dbus/check-dbus-services to ping the
services (T3122).

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Reviewed-by: André Magalhães <andre.magalhaes@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D5300

Needed so Newport can be D-Bus activated.

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Apertis: https://phabricator.apertis.org/T3091

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

This serves two purposes:

1. hotdoc won't warn about any incorrect documentation in
   the generated sources.
2. We don't need to remove html in other comments anymore.

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

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/D4487

 * debian/gbp.conf: extend default attributes and add dch
 * Integrated TAP test driver for installed test support
 * Download path and apparmor validation is done through "CbyProcessInfo"
 * AppArmor: update for new path of newport-client
 * AppArmor: allow logging to Journal stdout socket
 * newport-client AppArmor: update profile so it works again
 * Set libexecdir to /usr/lib
 * Add autopkgtest tests
 * Add debian/source/apertis-component marking this as a target package

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

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

There is currently one that runs the GNOME installed-tests.

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

The conservative default is the same as ${libdir}, i.e.
/usr/lib/MULTIARCH, but there's no real point in multiarch for most
libexec binaries (in particular, for our tests) and it makes finding
the tests a little harder.

Signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D4515

newport-client uses GTestDBus, so we need enough permissions to
run our own new dbus-daemon.

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

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

This is what we get if we're started by the GTestDBus code in the
newport-client test.

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

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

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

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

Integrated canterbury-0 dependency to perform validation
for download path and apparmor labels of clients.

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

newport tests are meant to run only when installed,
and it is added under installed_test_programs.

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

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

Reviewed-by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>
Signed-off-by: Héctor Orón Martínez <hector.oron@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D4329

Necessary for GQuark

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

+ activate the devhelp extension and update the build-depends
+ use tar-ustar to allow us to have filenames longer than
  99 characters in archives.
+ use -Wno-portability in order to use the hotdoc Makefile

The text for the standalone markdown documentation was taken
from the gtk-doc files removed in a previous commit.

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

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

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

This is not how we expose documentation, and this file
was only a half-completed template.

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

Latter commits will replace it with hotdoc

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

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

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

Otherwise, we can't autobuild newport and tumbler on the same machine.
tumbler specifically depends on libcurl4-gnutls-dev.

libcurl4-nss-dev is still the preferred one for now, although
it seems that the Mildenhall HMIs are the only things that depend
on it, so we should maybe standardize on libcurl3-gnutls as our
preferred implementation in future.

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

This is done by handling all curl related
errors in the switch statement.

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D2766

local variable "UrlInfo" which was shadowing "UrlInfo"
variable declared in the parent block is renamed to
"url_info_new"

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D2765

Variable declarations are moved to top of the block
as per coding guidelines.

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D2758

This resolves explicit inclusion of gvfs/remote-volume-monitors
in its apparmor profile as the rule already exists in
abstractions/chaiwala-base.

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D2899

Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

Madgula Rohan authored 9 years ago and Sjoerd Simons committed 8 years ago

Newport is now able to acquire the bus name and
able to create the folder under
user data directory for download database

Reviewed-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
Signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D2597

Madgula Rohan authored 9 years ago and Sjoerd Simons committed 9 years ago

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D2350

Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

Madgula Rohan authored 9 years ago and Sjoerd Simons committed 9 years ago

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D1895

Madgula Rohan authored 9 years ago and Sjoerd Simons committed 9 years ago

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D1894

Madgula Rohan authored 9 years ago and Sjoerd Simons committed 9 years ago

removed uinProgressCount and uinSpeedLimit variables from DownloadMgr structure,and these values are read during runtime of newport.

Reviewed-by: Aleksander Morgado <aleksander.morgado@collabora.co.uk>
signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D1323

Madgula Rohan authored 9 years ago and Sjoerd Simons committed 9 years ago

Added addtional check for string length in group name creation as this is used for name of temporary download file.

Reviewed-by: Aleksander Morgado <aleksander.morgado@collabora.co.uk>
signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D1230

Madgula Rohan authored 9 years ago and Sjoerd Simons committed 9 years ago

Removed unnecessary addition of download file name to the group name as download path includes the download file name.

Reviewed-by: Aleksander Morgado <aleksander.morgado@collabora.co.uk>
signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D1229

Madgula Rohan authored 9 years ago and Sjoerd Simons committed 9 years ago

Added missing statement of marking the current state as NEWPORT_DOWNLOAD_FAILED when newport fails to create file for download.

Reviewed-by: Aleksander Morgado <aleksander.morgado@collabora.co.uk>
signed-off-by: Rohan Madgula <Rohan.Madgula@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D1228