Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fluent-bit 3.1.5 #180727

Closed
wants to merge 1 commit into from
Closed

fluent-bit 3.1.5 #180727

wants to merge 1 commit into from

Conversation

BrewTestBot
Copy link
Member

Created by brew bump


Created with brew bump-formula-pr.

release notes
https://fluentbit.io/announcements/v3.1.5/

What's Changed

Full Changelog: fluent/fluent-bit@v3.1.4...v3.1.5

@github-actions github-actions bot added the bump-formula-pr PR was created using `brew bump-formula-pr` label Aug 10, 2024
@daeho-ro
Copy link
Sponsor Contributor

daeho-ro commented Aug 10, 2024

The patch should be updated.

diff --git a/lib/luajit-cmake/LuaJIT.cmake b/lib/luajit-cmake/LuaJIT.cmake
index c0dee5830..fb3c0c027 100644

--- a/lib/luajit-cmake/LuaJIT.cmake
+++ b/lib/luajit-cmake/LuaJIT.cmake
@@ -667,13 +667,13 @@ set(luajit_headers
   ${LJ_DIR}/luaconf.h
   ${LJ_DIR}/lualib.h
   ${CMAKE_CURRENT_BINARY_DIR}/luajit.h)
-install(FILES ${luajit_headers} DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/luajit)
+install(FILES ${luajit_headers} DESTINATION ${CMAKE_INSTALL_LIBEXECDIR}/include/luajit)
 install(TARGETS libluajit
     LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
     ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR})
 
 # Build the luajit binary
-if (LUAJIT_BUILD_EXE)
+if (FALSE)
   add_executable(luajit ${LJ_DIR}/luajit.c)
   target_link_libraries(luajit libluajit)
   target_include_directories(luajit PRIVATE

But still got error on the last line of LuaJIT.cmake

CMake Error at lib/luajit-cmake/LuaJIT.cmake:706 (add_executable):
  add_executable cannot create ALIAS target "luajit::lua" because target
  "luajit" does not already exist.

Don't know how to deal with it.

Copy link
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. To keep this pull request open, add a help wanted or in progress label.

@github-actions github-actions bot added the stale No recent activity label Aug 13, 2024
@github-actions github-actions bot closed this Aug 15, 2024
@github-actions github-actions bot deleted the bump-fluent-bit-3.1.5 branch August 15, 2024 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` stale No recent activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants