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

feat: Add tracy option for profiling on recipes #134

Merged
merged 11 commits into from
Nov 30, 2024
Prev Previous commit
Next Next commit
chore: make tests work
zaucy committed Nov 28, 2024
commit 0591725da4342aa635e6f283a1095991b6d7c94f
15 changes: 7 additions & 8 deletions test/MODULE.bazel
Original file line number Diff line number Diff line change
@@ -3,14 +3,7 @@ bazel_dep(name = "rules_ecsact", version = "0.5.0")
bazel_dep(name = "ecsact_codegen", version = "0.4.1")
bazel_dep(name = "boost.dll", version = "1.83.0.bzl.2")
bazel_dep(name = "boost.process", version = "1.83.0.bzl.2")
bazel_dep(name = "ecsact_runtime", version = "0.6.5")

# TODO: https://github.com/bazelbuild/bazel-central-registry/pull/1916
git_override(
module_name = "libarchive",
commit = "7c331f92acea5243c195cdc6fb46ecfa11ce1ce2",
remote = "https://github.com/zaucy/libarchive.git",
)
bazel_dep(name = "ecsact_runtime", version = "0.7.0")

bazel_dep(name = "toolchains_llvm", version = "1.0.0", dev_dependency = True)
bazel_dep(name = "hedron_compile_commands", dev_dependency = True)
@@ -37,3 +30,9 @@ register_toolchains(
"@llvm_toolchain//:all",
dev_dependency = True,
)

git_override(
module_name = "tracy",
commit = "9e3ab98805fec8f1fbcfd7747af4117bb984be19",
remote = "[email protected]:seaube/tracy.git",
)
2 changes: 1 addition & 1 deletion test/build_recipe/ecsact_build_test.cc
Original file line number Diff line number Diff line change
@@ -23,7 +23,7 @@ ecsact_execute_systems_error ecsact_execute_systems( //
local_dep::example();
// This is an incorrect way to use this function, but this test is only caring
// about the import
ecsact_system_execution_context_get(nullptr, {}, nullptr);
ecsact_system_execution_context_get(nullptr, {}, nullptr, nullptr);
ecsact_system_execution_context_action(nullptr, nullptr);
return ECSACT_EXEC_SYS_OK;
}
7 changes: 1 addition & 6 deletions test/build_recipe/ecsact_build_test_merge.cc
Original file line number Diff line number Diff line change
@@ -5,12 +5,7 @@
# error "This test should only have been built through 'ecsact build'"
#endif

void ecsact_system_execution_context_get(
struct ecsact_system_execution_context*,
ecsact_component_like_id,
void*,
...
) {
void ecsact_system_execution_context_get(struct ecsact_system_execution_context*, ecsact_component_like_id, void*, const void*) {
}

void ecsact_system_execution_context_action(struct ecsact_system_execution_context*, void*) {