Re: [PATCH] [contrib] Add process_make.py

2025-07-18 Thread Filip Kastl
> Hi, > > Thanks for the review! No, there is no reason for it being CamelCase. I had > written > the code in CamelCase originally then 'sed s/CamelCase/snake_case/g'd the > functions. > I forgot to update this one :) > > Looks like I also forgot to update the Colors class as well. I'd actuall

Re: [PATCH] [contrib] Add process_make.py

2025-07-18 Thread Dhruv Chawla
ch: +return c_match +return re.match(self.ignored_cxx, line) + + + +# Main routine +#################### + + +def main() -> None: +a

Re: [PATCH] [contrib] Add process_make.py

2025-07-18 Thread Filip Kastl
Hi, I've tried running it and quickly looked through the code. Looks good to me! I'm not a mantainer, though. Just two nitpicks from me inline... On Wed 2025-07-16 21:47:00, dhr...@nvidia.com wrote: > From: Dhruv Chawla > > This is a script that makes it easier to visualize the output from ma

[PATCH] [contrib] Add process_make.py

2025-07-16 Thread dhruvc
From: Dhruv Chawla This is a script that makes it easier to visualize the output from make. It filters out most of the output, leaving only (mostly) messages about files being compiled, installed and linked. It is not 100% accurate in the matching, but I feel it does a good enough job. To use it