[all-commits] [llvm/llvm-project] 7b1b12: [lldb][Windows] Enforce exec permission using Plat...

Dmitry Vasilyev via All-commits all-commits at lists.llvm.org
Tue May 14 02:57:58 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7b1b1279414217ea7f2402a03dfb5a18ea5a5367
      https://github.com/llvm/llvm-project/commit/7b1b1279414217ea7f2402a03dfb5a18ea5a5367
  Author: Dmitry Vasilyev <dvassiliev at accesssoftek.com>
  Date:   2024-05-14 (Tue, 14 May 2024)

  Changed paths:
    M lldb/source/Target/Platform.cpp

  Log Message:
  -----------
  [lldb][Windows] Enforce exec permission using Platform::Install() from Windows host (#91887)

Target::Install() set 0700 permissions for the main executable file.
Platform::Install() just copies permissions from the source. But the
permission eFilePermissionsUserExecute is missing on the Windows host. A
lot of tests failed in case of Windows host and Linux target because of
this issue. There is no API to provide the exec flag. This patch set the
permission eFilePermissionsUserExecute for all files installed via
Platform::Install() from the Windows host. It fixes a lot of tests in
case of Windows host and Linux target.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list