[MCOL-3981] doesn't build with old (2.8.x) cmake Created: 2020-05-01 Updated: 2020-11-12 Resolved: 2020-05-05 |
|
| Status: | Closed |
| Project: | MariaDB ColumnStore |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.5.1 |
| Type: | Bug | Priority: | Major |
| Reporter: | Sergei Golubchik | Assignee: | Daniel Lee (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Sprint: | 2020-7 | ||||||||
| Description |
|
Old cmake (that is used on some buildbot builders) doesn't understand DEFINES_FILE in the BISON_TARGET. As a workaround, specify --defines= explicitly in COMPILE_FLAGS. On the other hand, newer cmake that supports DEFINES_FILE needs it for proper dependency tracking. So, as a fix, use both, yes, it's redundant.
|
| Comments |
| Comment by Sergei Golubchik [ 2020-05-03 ] | ||
|
also, please, add
to .gitignore | ||
| Comment by Roman [ 2020-05-04 ] | ||
|
Plz review. | ||
| Comment by Patrick LeBlanc (Inactive) [ 2020-05-04 ] | ||
|
Looks fine, and BB seems to approve. Needs to also get into 1.4 though. | ||
| Comment by Patrick LeBlanc (Inactive) [ 2020-05-04 ] | ||
|
For QA, this is an unobservable change. | ||
| Comment by Patrick LeBlanc (Inactive) [ 2020-05-05 ] | ||
|
Talked with Roman; this isn't an issue for 1.4 b/c that is enterprise-only, and enterprise builders use an updated cmake. |