Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com>

In commit 63adbe18, this comment
documents the changes in the states:

    Appstore Server allows its client to set only the following states
    DOWNLOADING, INSTALLING and INSTALLED. Any other states set
    by the client is not allowed like DOWNLOAD_READY. Hence, changed it
    to DOWNLOADING.

Do the same for the other tests.

Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com>

This was already done for 2 rule files in 1b0d591c. This is also needed
for tools/usr.bin.frome-tool.in and tests/usr.lib.installed-tests.frome.test-service.in
to have apparmor letting dbus-messages pass.

Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com>

During tests, the applications database is stored in different /tmp/
sub-folders.

When accessing the database, sqlite3 tries to lock the file with fcntl()
and this action needs to be allowed with 'k' in the apparmor profile
for /usr/bin/frome and the installed test programs.

Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com>

Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com>

The applications database is stored in
${HOME}/.local/share/frome/applications.db. When accessing the database,
sqlite3 tries to lock the file with fcntl() and this action needs to be
allowed with 'k' in the apparmor profile for /usr/bin/frome.

Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com>

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

Signed-off-by: Andrew Lee (李健秋) <ajqlee@debian.org>

We wish to use only the Apertis CI pipeline workflow to build source
packages from Apertis GitLab and then submit them to OBS.

For packages where Apertis is the upstream the simplest approach is to
set them up as native packages to avoid having to generate orig tarballs
and manage changes on top of them.

Signed-off-by: Andrew Lee (李健秋) <andrew.lee@collabora.co.uk>

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

Newer hotdoc (0.10) introduced several breaking changes which cause the
documentation to be misbuilt, so let's explicitly depend on the old version.

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

This dh_apparmor is a debhelper program that will generate apparmor
profile reload scripts.

Here we call dh_apparmor in debian/rules to generate apparmor profile
reload script for each profile that ship in the binary packages.

Signed-off-by: Andrew Lee (李健秋) <andrew.lee@collabora.co.uk>

These apparmor profiles that shipped in this package used to load by
dpkg trigger which took too long as it reload all available profiles.

We are trying to minimize the the load time and also avoid
work-in-progress profile to be loaded when a developer is editing the
profile on the filesystem.

So that we need to call dh_apparmor in debian/rules to generate
apparmor profile reload script for each profile that ship in the
package.

Signed-off-by: Andrew Lee (李健秋) <andrew.lee@collabora.co.uk>

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

The `gbp dch` command was failing with:

    re.error: bad escape \T at position 38

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

Builds were failing:

    In file included from /usr/include/glib-2.0/glib/glist.h:32,
                     from /usr/include/glib-2.0/glib/ghash.h:33,
                     from /usr/include/glib-2.0/glib.h:50,
                     from /usr/include/glib-2.0/gobject/gbinding.h:28,
                     from /usr/include/glib-2.0/glib-object.h:23,
                     from /usr/include/glib-2.0/gio/gioenums.h:28,
                     from /usr/include/glib-2.0/gio/giotypes.h:28,
                     from /usr/include/glib-2.0/gio/gio.h:26,
                     from src/app-db.h:14,
                     from src/app-db.c:13:
    src/app-db.c: In function ‘frome_app_db_find_entry’:
    /usr/include/glib-2.0/glib/gmem.h:212:30: error: returning ‘GomResource *’ {aka ‘struct _GomResource *’} from a function with incompatible return type ‘FromeAppDBEntry *’ {aka ‘struct _FromeAppDBEntry *’} [-Werror=incompatible-pointer-types]
     #define g_steal_pointer(pp) ((__typeof__(*pp)) (g_steal_pointer) (pp))
                                 ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    src/app-db.c:213:10: note: in expansion of macro ‘g_steal_pointer’
       return g_steal_pointer (&entry);
              ^~~~~~~~~~~~~~~
    cc1: all warnings being treated as errors
    make[1]: *** [Makefile:1748: src/libfrome_internal_la-app-db.lo] Error 1

The code was correct since FromeAppDBEntry is a GomResource subclass, but add a
FROME_APP_DB_ENTRY() cast to make things a bit more robust and make the
compiler happy.

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

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

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

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

Thushara Malali Somesha authored 8 years ago and Emanuele Aina committed 8 years ago

Signed-off-by: Thushara Malali Somesha <ThusharaM.MalaliSomesha@in.bosch.com>
Reviewed-by: Emanuele Aina <emanuele.aina@collabora.com>
Differential Revision: https://phabricator.apertis.org/D5430

Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: Justin Kim <justin.kim@collabora.com>
Differential Revision: https://phabricator.apertis.org/D6118

This commit adds the debian autopkgtest to execute the
frome tests using gnome-desktop-testing-runner

Signed-off-by: Luis Araujo <luis.araujo@collabora.co.uk>
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D6028

Appstore Server allows its client to set only the following states
DOWNLOADING, INSTALLING and INSTALLED. Any other states set
by the client is not allowed like DOWNLOAD_READY. Hence, changed it
to DOWNLOADING.

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



Reviewed-by: Emanuele Aina <emanuele.aina@collabora.co.uk>
Signed-off-by: Nandini Raju <Nandini.Raju@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D5427

Allow mildenhall-settings to communicate with frome service.

Signed-off-by: Manoj Prabhakar Nallabothula <manojprabhakar.n@in.bosch.com>
Differential Revision: https://phabricator.apertis.org/D5357

mildenhall-appstore needs to be able to access it.

The device ID is used by both, the account and download API so I'm
exposing id as a service-wide property.

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



Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Reviewed-by: Justin Kim <justin.kim@collabora.com>
Differential Revision: https://phabricator.apertis.org/D5341

mildenhall-appstore needs to be able to access it.

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Reviewed-by: Justin Kim <justin.kim@collabora.com>
Differential Revision: https://phabricator.apertis.org/D5340

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

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

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

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

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

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

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

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

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

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

This API was actually part of the previous release but I forgot to
update it.

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

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

Needed to communicate with the app store server.

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Reviewed-by: Justin Kim <justin.kim@collabora.com>
Differential Revision: https://phabricator.apertis.org/D5186

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Reviewed-by: Justin Kim <justin.kim@collabora.com>
Differential Revision: https://phabricator.apertis.org/D5185