minor fixes to faddr2line

[PATCH 0/2] minor fixes to faddr2line

Posted by Pankaj Raghav (Samsung) 1 week, 3 days ago

From: Pankaj Raghav <p.raghav@samsung.com>

The first commit fixes the issue I am facing in NixOS.

The second commit uses a tempfile instead of shell variable while trying
to read the elf file.

Pankaj Raghav (2):
  scripts/faddr2line:use /usr/bin/env bash for portability
  scripts/faddr2line:fix "Argument list too long" error

 scripts/faddr2line | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)


base-commit: 846bd2225ec3cfa8be046655e02b9457ed41973e
-- 
2.50.1

Re: [PATCH 0/2] minor fixes to faddr2line

Posted by Josh Poimboeuf 6 hours ago

On Sun, Sep 21, 2025 at 12:03:56PM +0200, Pankaj Raghav (Samsung) wrote:
> From: Pankaj Raghav <p.raghav@samsung.com>
> 
> The first commit fixes the issue I am facing in NixOS.
> 
> The second commit uses a tempfile instead of shell variable while trying
> to read the elf file.
> 
> Pankaj Raghav (2):
>   scripts/faddr2line:use /usr/bin/env bash for portability
>   scripts/faddr2line:fix "Argument list too long" error
> 
>  scripts/faddr2line | 15 ++++++++++-----
>  1 file changed, 10 insertions(+), 5 deletions(-)

Queued, thanks!

-- 
Josh

Re: [PATCH 0/2] minor fixes to faddr2line

Posted by Pankaj Raghav 1 week ago

On 9/21/25 12:03, Pankaj Raghav (Samsung) wrote:
> From: Pankaj Raghav <p.raghav@samsung.com>
> 
> The first commit fixes the issue I am facing in NixOS.
> 
> The second commit uses a tempfile instead of shell variable while trying
> to read the elf file.
> 
> Pankaj Raghav (2):
>   scripts/faddr2line:use /usr/bin/env bash for portability
>   scripts/faddr2line:fix "Argument list too long" error
> 
>  scripts/faddr2line | 15 ++++++++++-----
>  1 file changed, 10 insertions(+), 5 deletions(-)
> 

Gentle ping Josh :)

--
Pankaj