-
Notifications
You must be signed in to change notification settings - Fork 6.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unable to read %LOCALAPPDATA% on Windows service accounts #12285
Comments
@BillyONeal @ras0219 @ras0219-msft Can you take a look? Thanks. |
ras0219-msft
added a commit
that referenced
this issue
Jul 13, 2020
* [vcpkg] Enable binary caching by default. Support `x-` migration. Fix passing multiple copies of single parameter arguments. * [vcpkg] Handle x- prefixes for general arguments * [vcpkg] Fix #12285 and improve documentation of default binary cache path * [vcpkg] Revert x- prefix homogenization for per-command arguments * [vcpkg] Only use accelerated compiler detection for Windows Desktop + Ninja. Improve breadcrumbs for users encountering issues. * [vcpkg] Fix compiler tracking not pre-downloading Ninja. Fix compiler tracking not looking in -err.log. * [vcpkg] Update toolsrc/src/vcpkg/binarycaching.cpp Co-authored-by: Billy O'Neal <[email protected]> * [vcpkg] Format Co-authored-by: Robert Schumacher <[email protected]> Co-authored-by: Billy O'Neal <[email protected]>
strega-nil
pushed a commit
to strega-nil/vcpkg
that referenced
this issue
May 5, 2021
* [vcpkg] Enable binary caching by default. Support `x-` migration. Fix passing multiple copies of single parameter arguments. * [vcpkg] Handle x- prefixes for general arguments * [vcpkg] Fix microsoft#12285 and improve documentation of default binary cache path * [vcpkg] Revert x- prefix homogenization for per-command arguments * [vcpkg] Only use accelerated compiler detection for Windows Desktop + Ninja. Improve breadcrumbs for users encountering issues. * [vcpkg] Fix compiler tracking not pre-downloading Ninja. Fix compiler tracking not looking in -err.log. * [vcpkg] Update toolsrc/src/vcpkg/binarycaching.cpp Co-authored-by: Billy O'Neal <[email protected]> * [vcpkg] Format Co-authored-by: Robert Schumacher <[email protected]> Co-authored-by: Billy O'Neal <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
After the merging of #12091 (cc @ras0219), the instances of vcpkg running in our Windows CI exited reporting that
%LOCALAPPDATA%
cannot be read. This environment variable is undefined as the CI jobs are running on a Windows service account (running as SYSTEM).As a hacky workaround,
$Env:LOCALAPPDATA = $Env:TEMP
solves the problem, but a proper fix would be much appreciated.Environment
[...]
)To reproduce
Steps to reproduce the behavior:
Run
vcpkg install
on a Windows service account.See error below:
Expected behavior
The command should succeed on service accounts. If %LOCALAPPDATA% is undefined, maybe it should fallback to one of the following directories:
C:\Windows\System32\config\systemprofile\AppData\Local
C:\Windows\SysWOW64\config\systemprofile\AppData\Local
The text was updated successfully, but these errors were encountered: