-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
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
DistroAV install in /Library instead of ~/Library - symlink workaround create issues after rebranding #200761
Comments
By looking at the cask code, I think the reason why it installs in |
That is the understanding, but we could not find out where this is set. Hence checking here. For example, when the .pkg is manually downloaded & launched the only option is the User's Home. Which is inline with the settings to restrict to "CurrentUserHomeDirectory" Which also aligns with the Domains restricted in the distribution.in config : https://github.com/DistroAV/DistroAV/blob/2e29a55d513d83d743dcaa5dfcb1dbbe2db5395c/cmake/macos/resources/distribution.in#L8 Looking at it, it seems this is only enforced in the installer-gui. Which MIGHT make sense as if the package is installed from the command line, it need sudo (I guess brew is similar ?). The other location where the installation target is set is at : https://github.com/DistroAV/DistroAV/blob/2e29a55d513d83d743dcaa5dfcb1dbbe2db5395c/cmake/macos/defaults.cmake#L36 |
I'm purely guessing but maybe it is related to this line? Will the In the Distribution file of your pkg it also shows
|
Thanks for poiting this out. Will investigate. I found out as well that when installing the pacakge manually via installer command it will work properly if using : With sudo this fail ( as no home directory exists i guess
And targeting / does install in /Library (expected but not wanted) Either we have a settings not set properly (as @yunhao-jiang pointed out above) or something else does not enforce the proper path. Also, this should jsut fail if trying to install anywhere else than Home library. |
@yunhao-jiang I did soem check and changing things in helpers.cmake L39 does not appear to change anywhere else. Same when trying to modify in defaults.cmake. The only place i have found to change is in the cfreate-pacakge.cmake.in but this is not directly related to the install-path. DistroAV try to stick as close to the OBS plugin template which suggest, that other plugin on brew would have the same type of issues. I am out of options to figure things out. |
There is a part to this issue that I would consider somewhat of an oddity on the OBS side. |
Indeed, plugin are user based but OBS is system wide. It's a choice by OBS folks to have it as such. This vision and package deployment seems to conflict a bit on that. The cask does NOT leave anything being as of today. The main challenge was that we changed the name from OBS-ndi to distroAV and we decided to let the user "clean-up/uninstall" themselves. If old files still exist it prevent from running the new version until cleaned-up. Was fine until we got a surge of folks with the plugin installed in /Library, and up until recently could never reproduce/find the reason why :D. It's here :) (not saying it's homebrew's fault either). In a hindsight, we should have "uninstalled + cleaned-up" old files ourselves when changing names. For the longer term :
I guess this can be closed then... |
Verification
--force
.brew update-reset && brew update
and retried my command.brew doctor
, fixed as many issues as possible and retried my command.Description of issue
Cask for DistroAV (formerly obs-ndi) install in /Library... instead of ~/Library...
There is a workaround in the cask but this does not clean-up / delete / remove the old obs-ndi files, simply unlink them.
This created a surge of Support Request without understanding where it came from for the DistroAV project.
As a maintainer on the DistroAV Project, I would like to find a more reliable way to deploy / update and maintain the Cask for the MacOS users.
Here are the challenges we wish to address:
Context:
I am new to homebrew recipes crafting.
This is aimed to be : 1. fixed quickly 2. revised to be simplified long-term.
Any good practice / recommendation would be useful. (already read the cookbook)
Command that failed
brew install distroav
Output of command with
--verbose --debug
Output of
brew doctor
andbrew config
Output of
brew tap
The text was updated successfully, but these errors were encountered: