Skip to content
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

Added support for custom folder links (and shorthands for audio-related system folders) - again #162

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

zetenynagy
Copy link

@zetenynagy zetenynagy commented Nov 28, 2023

Hey @aonez - sorry, I renamed my branch and that closed the pull request.

I have implemented what you requested.
Here are the changes:

  • renamed --custom-path-link to --add-drop-link
  • --add-drop-link now takes multiple arguments, similar to --add-file and --add-folder
  • added --skip-drop-link-check to skip custom drop folder source verification
  • added --make-absent-link-folder to create nonexistent custom drop folder at requested location if it fails the verification
  • moved items around in the readme and in -h

I didn't really have any time to properly test functionality though, so if someone could take a look, that'd be great. I will also try to find the time in the coming days.

(Original pull request was #151 )

Audio Unit (Component), VST, VST3, CLAP
- renamed --custom-path-link to --add-drop-link
- --add-drop-link now takes multiple arguments
- added --skip-drop-link-check to skip custom drop folder source verification
- added --make-absent-link-folder to create nonexistent custom drop folder source that fails the verification
- moved items around in the readme and in -h
- Removed SKIP_DROP_LINK_CHECK & MAKE_ABSENT_FOLDER
- Always try to create the target if missing
- Always try to create the link
@aonez
Copy link
Member

aonez commented Nov 28, 2023

Thanks so much @zetenynagy. I've went ahead and simplified a little the custom code and removed --skip-drop-link-check and --make-absent-link-folder, just creating the folder if it does not exist.

To be consistent with the rest of the code probably we don't even need to check and create the target folder, but feels right to do it.

@mhhollomon maybe you can try it on your environment.

@aonez aonez added this to the 1.2.2 milestone Nov 28, 2023
@mhhollomon
Copy link

mhhollomon commented Nov 28, 2023 via email

@zetenynagy
Copy link
Author

zetenynagy commented Nov 28, 2023

This is why I figured adding skipping the check and creating the missing folder as optional flags could be a good idea, since it is a niche use case, but can be convenient in very specific applications.

I agree with @mhhollomon - if the optional flags can't stay, I think creating the folder shouldn't happen automatically.

@aonez - what do you think?

@mhhollomon
Copy link

UPDATE

Bad news. Turns out per Apple Technical support, there is no way to sign a DMG that includes an alias into /Library that will actually allow you to copy through that alias.

(CF https://developer.apple.com/forums//thread/742317) (Note that there is special dispensation for /Application)

I will be switching to a pkg file instead.

@zetenynagy
Copy link
Author

zetenynagy commented Nov 29, 2023

@mhhollomon - this is actually a more complex issue, and is dependent on many variable factors (signed/unsigned, macOS version, .dmg file system, format, double-clicking vs dragging directly on the link folder icon, etc.), so it could work for you still. See this thread: https://forum.juce.com/t/dmg-with-symlinks-to-library-audio-plug-ins/52161/15

It apparently is fixed in Ventura and newer - I'll try testing it on Sonoma possibly later today.

edit: tested a .dmg with links to both the Components and VST folders on macOS 14.1.1 and you're right, dragging it to the icon does not work - but if I open the folder via the link, and drag the file over, it works flawlessly. This .dmg was not made with create-dmg though, so I will try making my own and will test to see if it works.

@mhhollomon
Copy link

mhhollomon commented Nov 29, 2023

@zetenynagy - awesome! - thank you for the information - finding anything on the subject has been a chore.

My box is Sonoma and it doesn't work - but I'll need to go through the thread you linked and see if I need to change anything.

@zetenynagy
Copy link
Author

finding anything on the subject has been a chore.

Yeah, it's such a niche topic.

Apparently on Sonoma, people are "bypassing" the fact that dragging onto the link itself is a no-go by letting the user know that they need to double-click the link and drag the file to the newly opened folder window. It's a somewhat crude solution, but it works.

@mhhollomon
Copy link

mhhollomon commented Nov 29, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants