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

Madgula Rohan authored 9 years ago and Sjoerd Simons committed 9 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

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

Make the behaviour more consistent by using the standard gnome
autogen.sh from https://wiki.gnome.org/Projects/GnomeCommon/Migration

.
This makes autogen correctly behave when NOCONFIGURE is set amongst
other things.

Also remove configure,{README,INSTALL}{.in,} from EXTRA_DIST and leave it up to
autofoo to import a standard INSTALL file.

Reviewed-by: Justin Kim <justin.kim@collabora.com>
Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D2063

Aleksander Morgado authored 9 years ago and Sjoerd Simons committed 9 years ago

If the -sections file isn't being stored in git, we need to explicitly request
to rebuild sections, or we may end up with a broken (re)build; e.g.:

    ./swfdec-unused.txt:1: warning: 2 unused declarations.They should be added to swfdec-sections.txt in the appropriate place.
    touch tmpl-build.stamp
    gtk-doc: Building XML
    cd . && \
            gtkdoc-mkdb --module=swfdec --source-dir=../swfdec --output-format=xml --expand-content-files="" --main-sgml-file=swfdec-docs.sgml --sgml-mode --output-format=xml --source-dir=../swfdec-gtk
    ../swfdec/swfdec_sandbox.c:45: warning: Section SwfdecSandbox is not defined in the swfdec-sections.txt file.
    ../swfdec-gtk/swfdec_gtk_socket.c:39: warning: Section SwfdecGtkSocket is not defined in the swfdec-sections.txt file.
    ../swfdec-gtk/swfdec_playback_pulse.c:34: warning: Symbol name not found at the start of the comment block.
    ../swfdec-gtk/swfdec_gtk_system.c:47: warning: Section SwfdecGtkSystem is not defined in the swfdec-sections.txt file.
    ./swfdec-unused.txt:1: warning: 2 unused declarations.They should be added to swfdec-sections.txt in the appropriate place.
    touch sgml-build.stamp
    gtk-doc: Building HTML
    rm -rf ./html
    mkdir ./html
    cd ./html && gtkdoc-mkhtml swfdec ../swfdec-docs.sgml
    warning: failed to load external entity "../xml/Internals.xml"
    ../swfdec-docs.sgml:36: element include: XInclude error : could not load ../xml/Internals.xml, and no fallback was found
    make[2]: *** [html-build.stamp] Error 6
    make[2]: Leaving directory `/srv/git/swfdec/doc'
    make[1]: *** [all-recursive] Error 1
    make[1]: Leaving directory `/srv/git/swfdec'
    make: *** [all] Error 2

Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: Aleksander Morgado <aleksander.morgado@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D2043

Aleksander Morgado authored 9 years ago and Sjoerd Simons committed 9 years ago

Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: Aleksander Morgado <aleksander.morgado@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D2016

Aleksander Morgado authored 9 years ago and Sjoerd Simons committed 9 years ago

Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: Aleksander Morgado <aleksander.morgado@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D2002

Aleksander Morgado authored 9 years ago and Sjoerd Simons committed 9 years ago

This change also renames the public enumerations so that they are properly
processed by glib-mkenums.

Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: Aleksander Morgado <aleksander.morgado@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D2001

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

"Usage of this option is discouraged" —dpkg-shlibdeps(1)

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

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

It turns out we weren't installing the static library. Do that,
because Debian policy says so.

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

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

There are none.

Also clean up MAINTAINERCLEANFILES: Makefile.in is automatically cleaned
so it doesn't need to be mentioned.

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

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

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

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

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

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

Basically the same as https://phabricator.apertis.org/D1435

.

Refs T770

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

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

It's added by autoreconf, so the correct place to clean it is
maintainer-clean, which is already done.

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

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

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

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

Also remove a comment that does not add any more information than the
line it documents.

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

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

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

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

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

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

The autogeneration for those files wasn't really adding anything,
and resulted in us fighting with the Autotools.

Similar to <https://phabricator.apertis.org/D539

>.

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

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

libseaton dependency is removed as it will not be maintained in future.GKeyFile
serves this purpose of storing all downloads information

Depends on D391. Part of T364

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

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

Part of D270

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

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

Part of D206

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

Dilip Balaraju authored 9 years ago and Sjoerd Simons committed 9 years ago

Part of D212

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

Dilip Balaraju authored 9 years ago and Sjoerd Simons committed 9 years ago

Part of D201

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

Dilip Balaraju authored 9 years ago and Sjoerd Simons committed 9 years ago

Added macros NEWPORT_DEBUG, NEWPORT_CRITICAL, NEWPORT_WARNING
for g_debug,g_critical,g_warning respectively.
And replaced few g_debug with NEWPORT_CRITICAL/NEWPORT_WARNING.

Part of D199

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

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

network preferencei management requires getting the current Network type
which is not supported by GNetworkMonitor,so it is replaced with connman service.

Part of D198

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

Dilip Balaraju authored 9 years ago and Sjoerd Simons committed 9 years ago

Due to namespace changes in libseaton, API's are updated according.
So to build package in needs libseaton version >=0.2.0

Part of D197

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

Dilip Balaraju authored 9 years ago and Sjoerd Simons committed 9 years ago

Added two interfaces one for service and another for download object.
Description for StartDownload and parameter description for GetDownloadForUrl methods updated.
gtk-doc updated with new design structure

Part of D195

Bug: https://bugs.apertis.org/show_bug.cgi?id=218


Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk&gt;,smcv>
Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

Dilip Balaraju authored 9 years ago and Sjoerd Simons committed 9 years ago

Ribchester was only used to get the download paths based on mimetypes.
Now this feature is removed as applications  provides the required download path to newport.
Now only one method i.e StartDownload() is available and no longer
StartApplicationDownload() is required, so ribchester dependency and it's code chunks is removed.

Part of D192

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>