[Zesty] No default apps shown on first start in kvm oem end user

thank you for your bug report, is that specific to the oem mode? Also could you report the bug using ubuntu-bug or at least manually attach the syslog/journal log to the bug?

apport information

apport information

apport information

Zeitgeist service didn't start because of pre-start script failing on missing activity db.

The attachment "zeitgeist-service-optional-prestart.patch" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

Trevinho disagrees with me, but I want to put on the record that I think comment #6's patch is wrong and that instead the VACUUM mode should explicitly check if the database exists and "return 0" in that case.

Rationale:

It's reasonable to say that it's okay to run without having run the main program first. The counter argument says that this is *not* reasonable and you would expect an error in this case. I think that there is no one true way here, and you should write programs with their intended usage in mind. In this case I think the intented use is that you could imagine a user writing a shell script such as:

  #!/bin/sh

  set -e

  zeitgeist-vacuum # or zeitgeist --vacuum
  zeitgeist

(of course this is a simplified version of the existing user unit's control flow)

and it's just silly to have to detect the DB not existing explicitly there. It's also a bit shady to ignore VACUUM failures that could point to a more severe problem.

For the record, I think the code to implement my change would look like this (untested).

    var file = File.new_for_path (db_path);
    if (!file.query_exists ())
    {
        debug ("Database '%s' doesn't exist, but that's okay".printf (db_path));
        return 0;
    }

With this change the unit file can remain unmodified.

Since there's disagreement, I won't unilaterally upload my suggestion.

As I wrote in IRC, unfortunately even using the new

  zeitgeist --vacuum

Would fail in such scenario, as it would return a non-zero value (see [1]), thus my idea of just changing the unit file.

Which also is due to the fact that a standalone tool is right to return an error in such cases, while it's up to script to handle these failures.

[1] https://cgit.freedesktop.org/zeitgeist/zeitgeist/tree/src/zeitgeist-daemon.vala#n478

corrado@corrado-z-feb27:~$ uname -a
Linux corrado-z-feb27 4.10.0-9-generic #11-Ubuntu SMP Mon Feb 20 13:47:35 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
corrado@corrado-z-feb27:~$

installed from: Ubuntu 17.04 "Zesty Zapus" - Alpha amd64 (20170227)
/home/corrado/.local/share/zeitgeist is empty.
copy /home/corrado/.local/share/zeitgeist from a different installation from:
Ubuntu 17.04 "Zesty Zapus" - Alpha amd64 (20161106)
seems solving the problem

see: https://bugs.launchpad.net/ubuntu/+source/zeitgeist/+bug/1667898

uploaded, please test tomorrow once it hits an image

This bug was fixed in the package zeitgeist - 1.0-0ubuntu1

---------------
zeitgeist (1.0-0ubuntu1) zesty; urgency=medium

  [ Jeremy Bicha ]
  * New upstream release (LP: #1665902)
  * debian/control:
    - Drop Build-Depends on intltool
  * Drop patches applied in new release
    - Give-the-D-Bus-service-files-the-correct-names.patch
    - Add-a-systemd-user-service-for-each-D-Bus-session-se.patch
    - fix_autocomplete.diff
    - fix-test-crash.patch
    - thread-default-context.patch
  * Refresh patches

  [ Iain Lane ]
  * Fix debian/patches/startup-database-vacuum.patch
    - The vacuum mode is now included upstream
    - Move the wrapper script to a file in debian/ and install that
    - Call the wrapper and allow it to fail so that zeitgeist starts for new
      users (LP: #1666495)
  * debian/patches/0001-Fix-placeholder-in-systemd-service-template.patch:
    Cherry-pick. Fix substitution in systemd unit file.

 -- Iain Lane <email address hidden> Wed, 01 Mar 2017 14:09:01 +0000