On Sat, Apr 12, 2025 at 08:19:50PM +0200, Stefan Weil wrote:
> Am 12.04.25 um 20:08 schrieb Arthur Sengileyev:
>
> > In msys2 distribution objdump from gcc is using single tab character
> > prefix, but objdump from clang is using 4 white space characters instead.
> > The script will not identify any dll dependencies for a QEMU build
> > generated with clang. This in turn will fail the build, because there
> > will be no files inside dlldir and no setup file will be created.
> > Instead of checking for whitespace in prefix use lstrip to accommodate
> > for differences in outputs.
> >
> > Signed-off-by: Arthur Sengileyev <arthur.sengile...@gmail.com>
> > ---
> > scripts/nsis.py | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/scripts/nsis.py b/scripts/nsis.py
> > index af4e064819..8f469634eb 100644
> > --- a/scripts/nsis.py
> > +++ b/scripts/nsis.py
> > @@ -23,7 +23,7 @@ def find_deps(exe_or_dll, search_path, analyzed_deps):
> > output = subprocess.check_output(["objdump", "-p", exe_or_dll],
> > text=True)
> > output = output.split("\n")
> > for line in output:
> > - if not line.startswith("\tDLL Name: "):
> > + if not line.lstrip().startswith("DLL Name: "):
> > continue
> > dep = line.split("DLL Name: ")[1].strip()
>
>
> Thanks. I use nearly the same code `if not line.strip().startswith("DLL
> Name: "):` in my builds for WoA.
>
> @Stefan, can this trivial patch still be applied for 10.0?Yes, it is included in Michael Tokarev's latest trivial patches pull request. > I had planned to replace the whole code with objdump by platform independent > Python code, but that's a larger change, and I missed the deadline. > > > Reviewed-by: Stefan Weil <s...@weilnetz.de> > >
signature.asc
Description: PGP signature
