Repository navigation
Docs/roadmap refresh - #45
Merged
Merged
Conversation
…stale parts Three entries were already finished and still listed as pending: - Fuzz Testing: three libFuzzer targets exist, with corpora and CI wiring. - Per-module test files: eight files now (cpu, full_flow, http_handlers, logger, async_executor, result_parsing, renderer_parsing, thread_stacks). - Parallel ctest: verified with `ctest -j6`. Worth recording why it needed work rather than just noting it passes -- the tests used to truncate each other's files because the profiler's intermediate artifacts had fixed names in a shared directory. They carry the pid now, so process isolation is enough and no RESOURCE_LOCK is needed. The counts in that section had drifted too: test_http_handlers.cpp has 28 cases, not 15, and the `output_type` it mentioned was renamed to `renderer`. The section now states the current totals (8 targets, 79 TEST cases) so it can be checked against reality instead of trusted. The Conan/vcpkg entry said "libs is wrong and requirements are missing openssl/zlib/gtest/absl". Checking it, the recipe has six defects and abseil is declared, so the old wording was both incomplete and partly wrong. Recorded from static reading, explicitly flagged as unverified because conan is not installed here: a missing `check_min_cppstd` import (NameError as soon as validate() runs), the nonexistent `profiler_lib`, copy paths that do not match where CMake installs, a duplicated backward-cpp requirement, a `REMOTE_PROFILER_ENABLE_SYMBOLIZE` macro that exists nowhere in the code, and placeholder metadata. Two further breakages turned up while checking the port: `portfile.cmake` has `SHA512 0`, and the placeholder homepage is in four places including the root vcpkg.json, which is the primary path. Verified: links resolve, both workflow files untouched, and every file the new entries reference exists.
…sting Two findings from the previous entry's own verification did not make it into that commit -- the edit script asserted on an ASCII quote where the file uses the full-width one, so it wrote nothing and I reported it as written. They are here now, plus the indentation corrected: the entry had landed a level too deep, reading as a sub-item of "自动化发布流程". What the additions record: - `ports/cpp-remote-profiler/portfile.cmake` carries `SHA512 0`, a placeholder checksum. vcpkg verifies it after downloading, so the port fails to install even once registered in vcpkg-configuration.json. That is a second, independent reason the port cannot be used, beyond it not being registered. - The placeholder `your-org` homepage is in four places, and one of them is the root `vcpkg.json` -- the manifest on the primary, documented install path. The earlier entry only mentioned conanfile.py. Verified: every list indent is even, and each file the section references exists.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.