Hi,

"Adam D. Barratt" <a...@adam-barratt.org.uk> writes:
> Apologies for the delay in getting back to you.  I did have one quick
> query on the diff, as I couldn't immediately see how the below hunk
> relates to the socket location:
>
> +--- akonadi-1.3.1.orig/server/src/storage/dbconfig.cpp
> ++++ akonadi-1.3.1/server/src/storage/dbconfig.cpp
> [...]
> +@@ -142,7 +143,7 @@
> + 
> +       // verify settings and apply permanent changes (written out below)
> +       if ( mDriverName == QLatin1String( "QMYSQL" ) ) {
> +-        if ( mInternalServer && mConnectionOptions.isEmpty() )
> ++        if ( mInternalServer /* && mConnectionOptions.isEmpty() */ )
> +           mConnectionOptions = defaultOptions;
> +         if ( mInternalServer && (mServerPath.isEmpty() || 
> !QFile::exists(mServerPath) ) )
> +           mServerPath = defaultServerPath;

This makes akonadi ignore the QMYSQL/Options setting from
~/.config/akonadi/akonadiserverrc.  This was necessary as this contains
the path for the socket which this patch changes.

There is another possible issue with the patch: after an upgrade the
user might have to restart all processes using akonadi (including the
backend) as new processes will no longer use the old socket and thus
fail.  There has been one report for upgrading to akonadi 1.3.1-4 in
testing that looked like this[1], but the submitter said it happened
only after restarting the computer (which should not happen).

I haven't tested this as the old version does not work at all where I
wrote the patch (and the upgrading case was less important before
squeeze was released).  In any case this will also happen during the
upgrade from squeeze to wheezy as upstream adopted the patch.

For a stable update, we might still want to add another patch to look in
the old locations as well.  I can try to do this next week, but I don't
think it will get as much testing (and I am unfamiliar with Qt/KDE).

Regards,
Ansgar

[1] <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=545139#50>



-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/87k4cq3h3x....@marvin.43-1.org

Reply via email to