[Bug 307750] New: No path to mysqld set in server configuration

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 307750] New: No path to mysqld set in server configuration

Carsten Pfeiffer-2
https://bugs.kde.org/show_bug.cgi?id=307750

            Bug ID: 307750
          Severity: major
           Version: 2.6.0
          Priority: NOR
          Assignee: [hidden email]
           Summary: No path to mysqld set in server configuration
    Classification: Unclassified
                OS: Linux
          Reporter: [hidden email]
          Hardware: Other
            Status: NEW
         Component: Database
           Product: digikam

I cannot use Digikam with mysql anymore. It refuses to start the mysql process.
I suppose this is because of the following code in databaseserver.cpp:

const QString mysqldPath(DatabaseConfigElement::element(dbType).dbServerCmd);
//const QString mysqldPath("/usr/sbin/mysqld");
if ( mysqldPath.isEmpty() || mysqldPath.compare( QLatin1String(
"SERVERCMD_MYSQL-NOTFOUND" ) ))
{
        kDebug() << "No path to mysqld set in server configuration!";
        return DatabaseServerError(DatabaseServerError::StartError, i18n("No
path to mysqld set in server configuration."));
}

The method always aborts when the mysqldPath is either empty or !=
"SERVERCMD_MYSQL-NOTFOUND". So unless your mysqld binary is named like that,
digikam refuses to start it. You should rather use the == operator than
compare().

Reproducible: Always

Steps to Reproduce:
1. Configure digikam to use an internal mysql server
2. Invoke "Check Database Connection" or just restart digikam
Actual Results:  
Digikam is unable to start mysqld and thus cannot connect to the database.

Expected Results:  
Digikam starts mysqld properly.

I'd classify this as major, because I cannot access my images from digikam
anymore.

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 307750] No path to mysqld set in server configuration

Gilles Caulier-4
https://bugs.kde.org/show_bug.cgi?id=307750

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
            Version|2.6.0                       |3.0.0

--- Comment #1 from Gilles Caulier <[hidden email]> ---
yes, i can confirm code commented in 3.0.0.

Francesco, what do you think about ?

Gilles Caulier

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 307750] No path to mysqld set in server configuration

Carsten Pfeiffer-2
In reply to this post by Carsten Pfeiffer-2
https://bugs.kde.org/show_bug.cgi?id=307750

--- Comment #2 from Carsten Pfeiffer <[hidden email]> ---
Do you want me to send a patch?

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 307750] No path to mysqld set in server configuration

Carsten Pfeiffer-2
In reply to this post by Carsten Pfeiffer-2
https://bugs.kde.org/show_bug.cgi?id=307750

--- Comment #3 from Carsten Pfeiffer <[hidden email]> ---
*ping* pretty please, fix this before 3.0. It's really trivial.

I can also commit this myself, if you prefer that.

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 307750] No path to mysqld set in server configuration [patch]

Gilles Caulier-4
In reply to this post by Carsten Pfeiffer-2
https://bugs.kde.org/show_bug.cgi?id=307750

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|No path to mysqld set in    |No path to mysqld set in
                   |server configuration        |server configuration
                   |                            |[patch]

--- Comment #4 from Gilles Caulier <[hidden email]> ---
Carsten,

You is welcome to commit this patch in git/master...

Best

Gilles Caulier

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 307750] No path to mysqld set in server configuration [patch]

Carsten Pfeiffer-2
In reply to this post by Carsten Pfeiffer-2
https://bugs.kde.org/show_bug.cgi?id=307750

Carsten Pfeiffer <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
   Version Fixed In|                            |3.0
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/digi
                   |                            |kam/62ec1eae9e92b70f8dcc0e8
                   |                            |48c3f9bb1692f49c2

--- Comment #5 from Carsten Pfeiffer <[hidden email]> ---
Git commit 62ec1eae9e92b70f8dcc0e848c3f9bb1692f49c2 by Carsten Pfeiffer.
Committed on 10/01/2013 at 21:59.
Pushed by pfeiffer into branch 'master'.
FIXED-IN: 3.0

Make internal mysql server work again.

M  +1    -1    databaseserver/databaseserver.cpp

http://commits.kde.org/digikam/62ec1eae9e92b70f8dcc0e848c3f9bb1692f49c2

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 307750] No path to mysqld set in server configuration [patch]

Gilles Caulier-4
In reply to this post by Carsten Pfeiffer-2
https://bugs.kde.org/show_bug.cgi?id=307750

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Version Fixed In|3.0                         |3.0.0

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 307750] No path to mysqld set in server configuration [patch]

bugzilla_noreply
In reply to this post by Carsten Pfeiffer-2
https://bugs.kde.org/show_bug.cgi?id=307750

[hidden email] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|Database                    |Database-Mysql

--
You are receiving this mail because:
You are the assignee for the bug.