-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
LLVM.LLVM package does not register PATH #11767
Comments
I'm not familiar with LLVM. It looks like a collection of command-line tools. I'm not sure if the installer registers the path or not. It may require a terminal restart to get the path if the installer does set the path. |
@denelon It provides an option to register the path, and the manifest seems not to be using that option. |
microsoft/winget-cli#222 is a feature request to refresh the path after install. Would that address this situation? |
I don't think so, the default option from the installer is not to register path, but when someone install a package via CLI the default expectation is to get a path (IMO at least), so I think the manifest should be somehow updated. |
Is setting the path just passing a flag to the installer? If that is the case, the switches could be modified to pass the flag. If it requires passing the desired path to the installer, I'm not sure that would be an option at this point. |
It gives three options in GUI:
I'm not sure what flag should be given (or whether it even receives a flag) when running it via CLI since I don't see a documentation for that. |
The manifest has: InstallerSwitches:
InstallLocation: /D=<INSTALLPATH> https://github.com/microsoft/winget-cli/tree/master/schemas/JSON/manifests/v1.0.0 It looks like this might be suitable for "Custom". You may need to test with a local manifest to see which one handles the scenario correctly. You can copy the manifest files to your local system and then |
Only if I know what "custom" flag it receives 😄 |
Maybe you should first try to know which tool is used to pack the installer. For example, when installed via winget, VSCode can register path/association/context menu by using switch |
Seems it uses NSIS per the build script. |
This is not enough... See VSCode installer build script , you'll need to know the possible tasks/sections in install script. |
CPACK_NSIS_MODIFY_PATH is the CPack API to add a relevant option page to the NSIS installer. This is the underlying source to interact with NSIS, but I'm not sure what it really does. |
Maybe you can try the command listed here? By passing |
Has anybody been able to determine which flags need to be passed to the installer to register the path for LLVM? |
Chocolatey manually registers the path (see |
This comment was marked as outdated.
This comment was marked as outdated.
I think I did respond 🤨 |
This comment was marked as outdated.
This comment was marked as outdated.
Can someone remove that label please? |
Was this resolved by LLVM with their latest update? |
No; I was just looking at the wrong file. Sorry for the ghost ping |
Still not resolved, ehh. |
@nlhnt, Some of the work in that feature is related to registering applications. Maybe there will be a way to leverage the path logic after that is complete. I'll bring it up to the team. Is LLVM open source? Maybe they would have insight into command line switches to register a path via silent install. |
LLVM is open source. I started looking into the project since they use nullsoft for their installer but it's hard to tell how they have it configured since they use cmake on top (with the solution file equivalent here https://github.com/llvm/llvm-project/blob/main/llvm/CMakeLists.txt). I've put in an issue (llvm/llvm-project#54724) and will be able to determine from there if they have nullsoft configured to allow setting the path. |
LLVM uses cmake to generate their nullsoft installer, so they haven't enabled any commandline flags aside from the default nullsoft ones. In order to enable a silent install of LLVM that sets the path someone would have to contribute that option to the previously mentioned CMakeLists file. This was explained in the most recent comment in response to the LLVM issue, and I will be adjusting the question into a feature request since I'm not sure how to implement this myself. |
Try |
Hello @saschanaz, This issue has been identified as requiring a fix from a third party or external repository. Since there has been no recent activity on this issue, it will be automatically closed. Template: msftbot/noRecentActivity/areaExternal |
Brief description of your issue
... thus the clang command is not exposed to system.
Steps to reproduce
winget install LLVM.LLVM
Expected behavior
It should register PATH.
Actual behavior
It does not.
Environment
The text was updated successfully, but these errors were encountered: