[Package Issue]: Git.Git

[zsien]

Please confirm these before moving forward

• I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
• I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue

Other

Brief description of your issue

After installing Git.Git using winget, running commands like git log or git diff results in an error when Git tries to invoke the default pager less. The following errors are displayed:

error: cannot spawn less: Permission denied
fatal: unable to execute pager 'less'

Steps to reproduce

• Install Git.Git via winget: winget install -e --id Git.Git.
• Open your terminal (e.g., CMD, PowerShell, or Windows Terminal).
• Run a command that uses the pager, such as git log or git diff.
• Observe the error messages regarding less.

Actual behavior

Git fails to spawn the less pager due to a permission error, resulting in the failure of commands like git log and git diff.

Expected behavior

Git should correctly invoke less as the pager and display output without any permission errors.

Environment

Windows: Windows.Desktop v10.0.19044.4529
系统体系结构: X64
软件包: Microsoft.DesktopAppInstaller v1.25.340.0

Winget 目录
-------------------------------------------------------------------------------------------------------------------
日志                   %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir
用户设置               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json
可移植链接目录(用户)   %LOCALAPPDATA%\Microsoft\WinGet\Links
可移植链接目录(计算机) C:\Program Files\WinGet\Links
可移植包根目录(用户)   %LOCALAPPDATA%\Microsoft\WinGet\Packages
可移植包根目录         C:\Program Files\WinGet\Packages
可移植包根目录 (x86)   C:\Program Files (x86)\WinGet\Packages
安装程序下载           %USERPROFILE%\Downloads
配置模块               %LOCALAPPDATA%\Microsoft\WinGet\Configuration\Modules

链接
----------------------------------------------------------------------------
隐私声明             https://aka.ms/winget-privacy
许可协议             https://aka.ms/winget-license
第三方声明           https://aka.ms/winget-3rdPartyNotice
主页                 https://aka.ms/winget
Windows 应用商店条款 https://www.microsoft.com/en-us/storedocs/terms-of-sale

管理员设置                                状态
------------------------------------------------
LocalManifestFiles                        已禁用
BypassCertificatePinningForMicrosoftStore 已禁用
InstallerHashOverride                     已禁用
LocalArchiveMalwareScanOverride           已禁用
ProxyCommandLineOptions                   已禁用
DefaultProxy                              已禁用

Screenshots and Logs

No response

[Dragon1573]

⚠ Unable to reproduce in Windows Sandbox instance.

Note

There's a fetal error when first invoking git log -n 1. It was caused by a feature provided by Windows Sandbox. It can map any folder on host to Sandbox instance. I'm running the test using exactly the same repository on both side. There might encounter conflicts.

This problem will not exist if there's only 1 Windows user accessing the repository.

--> Installing WinGet
--> Disabling safety warning when running installers
Tip: you can type 'Update-EnvironmentVariables' to update your environment variables, such as after installing a new software.

--> Configuring Winget
已启用管理员设置 'LocalManifestFiles'。
已启用管理员设置 'LocalArchiveMalwareScanOverride'。
已启用管理员设置 'ProxyCommandLineOptions'。
将管理员设置 'DefaultProxy' 设置为 'http://alist.dragon1573.local:57890'。

PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs> winget install --accept-source-agreements -e --id Git.Git
“msstore”源要求在使用前查看以下协议。
Terms of Transaction: https://aka.ms/microsoft-store-terms-of-transaction
源要求将当前计算机的 2 个字母的地理区域发送到后端服务才能正常工作，(例如"US")。

已找到 Git [Git.Git] 版本 2.48.1
此应用程序由其所有者授权给你。
Microsoft 对第三方程序包概不负责，也不向第三方程序包授予任何许可证。
正在下载 https://github.com/git-for-windows/git/releases/download/v2.48.1.windows.1/Git-2.48.1-64-bit.exe
  ██████████████████████████████  66.3 MB / 66.3 MB
已成功验证安装程序哈希
正在启动程序包安装...
已成功安装

PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs> Update-EnvironmentVariables

PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs> git --version
git version 2.48.1.windows.1

PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs> git log -n 1
fatal: detected dubious ownership in repository at 'C:/Users/WDAGUtilityAccount/Desktop/winget-pkgs/'
'C:/Users/WDAGUtilityAccount/Desktop/winget-pkgs/' is owned by:
        (inconvertible) (S-1-5-21-2612129988-1302729843-3266673762-1001)
but the current user is:
        1D5B2862-EA9A-4/WDAGUtilityAccount (S-1-5-21-2047949552-857980807-821054962-504)
To add an exception for this directory, call:

        git config --global --add safe.directory C:/Users/WDAGUtilityAccount/Desktop/winget-pkgs/

PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs> git config --global --add safe.directory C:/Users/WDAGUtilityAccount/Desktop/winget-pkgs/

PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs> git log -n 1
commit eacf058694e0c17bc50637a5fa846f4266451642 (HEAD -> master, origin/master, origin/HEAD)
Author: Dragon1573 <49941141+Dragon1573@users.noreply.github.com>
Date:   Wed Mar 12 15:13:26 2025 +0800

    Bugfix (SandboxTest): Trim unexpected "UTC" suffix

    - The unexpected suffix is responsed by GitHub RESTful API
    - It will lead to an error when parsing it to DateTime object for token caching.

PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs> git diff HEAD upstream/master --stat
 .pre-commit-config.yaml |  17 ----
 Tools/SandboxTest.ps1   | 216 +++++-------------------------------------------
 2 files changed, 22 insertions(+), 211 deletions(-)

PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs> git diff HEAD upstream/master --stat
 .pre-commit-config.yaml |  17 ----
 Tools/SandboxTest.ps1   | 216 +++++-------------------------------------------
 2 files changed, 22 insertions(+), 211 deletions(-)

PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs> git diff HEAD upstream/master
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
deleted file mode 100644
index ef80600a3a7..00000000000
--- a/.pre-commit-config.yaml
+++ /dev/null
@@ -1,17 +0,0 @@
-repos:
-  - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v5.0.0
-    hooks:
-      - id: check-yaml
-        name: 'Check YAML Syntax'
-      - id: end-of-file-fixer
-        name: 'Fix End of Files'
-      - id: trailing-whitespace
-        name: 'Trim Trailing Whitespace'
-  - repo: local
-    hooks:
-      - id: prevent-alist
-        name: 'Prevent Local File Server'
-        language: pygrep
-        entry: 'http(s?)://.*\.local/.*'
-        files: 'manifests/.*'
diff --git a/Tools/SandboxTest.ps1 b/Tools/SandboxTest.ps1
index e6d9fbdcb13..869cfc4f04c 100644
--- a/Tools/SandboxTest.ps1
+++ b/Tools/SandboxTest.ps1
@@ -32,16 +32,11 @@ Param(
     # WinGetOptions
     [Parameter(HelpMessage = 'Additional options for WinGet')]
     [string] $WinGetOptions,
-    # GitHubToken
-    [Parameter(HelpMessage = 'Token for GitHub API Requests')]
-    # It is possible that the environment variable may not exist, in which case this may be null
-    [string] $GitHubToken = $env:GITHUB_TOKEN,
     # Switches
     [switch] $SkipManifestValidation,
     [switch] $Prerelease,
     [switch] $EnableExperimentalFeatures,
-    [switch] $Clean,
-    [string] $Proxy
+    [switch] $Clean
 )

 enum DependencySources {
@@ -51,7 +46,7 @@ enum DependencySources {

 # Script Behaviors
 $ProgressPreference = 'SilentlyContinue'
-$ErrorActionPreference = 'Stop' # This gets overridden most places, but is set explicitly here to help catch errors
+$ErrorActionPreference = 'Stop' # This gets overriden most places, but is set explicitly here to help catch errors
 if ($PSBoundParameters.Keys -notcontains 'InformationAction') { $InformationPreference = 'Continue' } # If the user didn't explicitly set an InformationAction, Override their preference
 $script:OnMappedFolderWarning = ($PSBoundParameters.Keys -contains 'WarningAction') ? $PSBoundParameters.WarningAction : 'Inquire'
 $script:UseNuGetForMicrosoftUIXaml = $false
@@ -61,7 +56,6 @@ $script:DependenciesBaseName = 'DesktopAppInstaller_Dependencies'
 $script:ReleasesApiUrl = 'https://api.github.com/repos/microsoft/winget-cli/releases?per_page=100'
 $script:DependencySource = [DependencySources]::InRelease
 $script:UsePowerShellModuleForInstall = $false
-$script:CachedTokenExpiration = 30 # Days

 # File Names
 $script:AppInstallerMsixFileName = "$script:AppInstallerPFN.msixbundle" # This should exactly match the name of the file in the CLI GitHub Release
@@ -81,7 +75,6 @@ $script:UiLibsHash_NuGet = '6B62BD3C277F55518C3738121B77585AC5E171C154936EC58D87

PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs>

Important

The account WDAGUtilityAccount is an administrator account of Windows Sandbox instance. During the test (launched via Tools/SandboxTest.ps1), a default PowerShell session will automatically launched with admin privileges and I use this session directly.

Can't sure if it's related to the Windows account privileges. 😕