Re: [PATCH] Remove grok_architecture call from gdb.commit_logs.commit_logs

["Serhei Makarov" <me@serhei.io>]

On Thu, Aug 5, 2021, at 2:42 PM, Keith Seitz via Bunsen wrote:
> `grok_architecture' is called from gdb's commit_logs function.
> However at this moment, we actually can't do anything to ascertain
> the architecture. This function is currently passing the `testdir'
> variable, which is non-sensical.
> 
> Architecture is actually grok'd in annotate_dejagnu_log, so simply
> remove the call here.
The history of this line is that I was importing results from Sergio's buildbots,
where they were organized in directory and the name of the directory was the most obvious
(iirc the only) source for some of the arch/distro information.

Looks like this is the result of a messed up refactoring. At some point grok_architecture was
local to commit_logs and did a more flexible match for architecture names.

Therefore, ok to commit the patch. (I'll do it myself later on if you're busy, but it might be
good to test that you have commit access.)

All the best,
      Serhei