funccount rename to count
- Dominant language
- C
- Stars
- 22.7k
- Forks
- 4.1k
- Avg merge
- 10d 4h
- Merged PRs (30d)
- 3
Description
When I wrote funccount, it traced kernel function calls only, but I expected we'd add user-level functions as well. Hence the name. But Sasha not only added uprobes, but also tracepoints and USDT. Now it does more than count functions, it counts everything. Should I rename it to "count"?
I really like how trace is called trace. And these tools are similar, accepting events described in the same way. It sounds more consistent to have them as trace and count.
This is going to break anyone who has automated it, or has learned funccount (and I've written a lot of documentation that calls it funccount). I'd propose adding a shell wrapper called funccount that did:
```shell
echo "NOTICE: funccount is now called count. Executing count."
if [ -e count.py ]; then
./count.py "$@"
else
./count "$@"
fi
```
So this transition would not be painful.
@goldshtn @yonghong-song ... I'll file a separate ticket for argdist. :-)
Contributor guide
No contributing guide indexed for this repository
Research direction
Review the count.py and count entry points, then locate the documentation that still calls the tool funccount. Check how the proposed shell wrapper would preserve existing invocations and how related tools such as trace are named. Done means the rename and compatibility transition have maintainer agreement, with affected documentation identified.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- linux, python, shell
- Domain
- cli, documentation, operating-systems
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100