Target sql_embedded should depend on GenDigestServerSource

Bug #1461129 reported by Sergei Glushchenko on 2015-06-02
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MySQL Server
Unknown
Unknown
Percona Server moved to https://jira.percona.com/projects/PS
Status tracked in 5.7
5.5
New
Undecided
Unassigned
5.6
Triaged
Low
Unassigned
5.7
Triaged
Low
Unassigned
Percona XtraBackup moved to https://jira.percona.com/projects/PXB
Fix Released
High
Sergei Glushchenko
2.2
Fix Released
High
Sergei Glushchenko
2.3
Fix Released
High
Sergei Glushchenko

Bug Description

Copy of https://bugs.mysql.com/bug.php?id=77226

Description:
Source file 'sql_yacc.yy' needs generated file 'lex_token.h'. This file is generated by 'GenDigestServerSource', which is dependency for 'sql', but not for 'sql_embedded'. One cannot see it normally, but if you disable 'sql' and leave only 'sql_embedded' by editing 'CMakeLists.txt' you will get compile error.

How to repeat:
Check 'sql/CMakeLists.txt' and 'libmysqld/CMakeLists.txt'.

Suggested fix:
--- i/libmysqld/CMakeLists.txt
+++ w/libmysqld/CMakeLists.txt
@@ -81,7 +81,7 @@ SET(SQL_EMBEDDED_SOURCES

 ADD_CONVENIENCE_LIBRARY(sql_embedded ${SQL_EMBEDDED_SOURCES})
 DTRACE_INSTRUMENT(sql_embedded)
-ADD_DEPENDENCIES(sql_embedded GenError GenServerSource)
+ADD_DEPENDENCIES(sql_embedded GenError GenServerSource GenDigestServerSource)

 # On Windows, static embedded server library is called mysqlserver.lib
 # On Unix, it is libmysqld.a

summary: - arget sql_embedded should depend on GenDigestServerSource
+ Target sql_embedded should depend on GenDigestServerSource
tags: added: upstream

Percona now uses JIRA for bug reports so this bug report is migrated to: https://jira.percona.com/browse/PXB-431

Percona now uses JIRA for bug reports so this bug report is migrated to: https://jira.percona.com/browse/PS-2114

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.