Add btyacc/3.0 #8947
Add btyacc/3.0 #8947
Conversation
This comment has been minimized.
This comment has been minimized.
|
I have no clue how to handle that MacOS CI error. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
friendlyanon
force-pushed
the
package/btyacc
branch
from
c1bc69b to
9d97539
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
From discussions on the FirebirdSQL mailing list, it appears that this is just a build requirement of developer workflows when working on FirebirdSQL. However, I'm not sure whether this serves a general use-case or not. Does flex/bison/similar do basically the same as this does? If no, then I don't see a reason why this couldn't be packaged as well. |
|
We are also missing some CMake targets https://github.com/ChrisDodd/btyacc/blob/0256b69f36f01877f1f4e76f80a5f561beabcfef/CMakeLists.txt#L69-L71 |
This comment has been minimized.
This comment has been minimized.
|
|
||
| add_executable(test_package "${bin}/main.c") | ||
|
|
||
| enable_testing() |
There was a problem hiding this comment.
Usually we don't use ctest because it hides the output and also because we don't use assert statement on test package code. But I'm okay as it doesn't affect the test result.
There was a problem hiding this comment.
The reason I used CTest here was because I don't know whether run spawns a shell, where I can pipe things, or not.
|
All green in build 2 (
|
Specify library name and version: btyacc/3.0
This is a build requirement of fbclient.
Issue: #8618
conan-center hook activated.