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
Status tracked in 5.7
5.5
Undecided
Unassigned
5.6
Low
Unassigned
5.7
Low
Unassigned
Percona XtraBackup
High
Sergei Glushchenko
2.2
High
Sergei Glushchenko
2.3
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
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.