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

Fix contextual menu command and command-line #9

Open
wants to merge 2 commits into
base: stable
Choose a base branch
from

Conversation

arkichek
Copy link

@arkichek arkichek commented Nov 12, 2022

Xcode always passes targets as arguments to plugin, thats why a condition !arguments.isEmpty is always true, and final command is incorrect. Also have added missing DERIVED_SOURCES_DIR environment support.
Fixes #8

@arkichek arkichek changed the title Fix contextual menu command and command-line. Fixes #8 Fix contextual menu command and command-line. Nov 12, 2022
@arkichek arkichek changed the title Fix contextual menu command and command-line. Fix contextual menu command and command-line Nov 12, 2022
@twprzybysz
Copy link

@djbe can you take a look and merge this feature asap?

@twprzybysz
Copy link

@djbe 🚀

@acosmicflamingo
Copy link

Interesting...looks like others have found the same missing DERIVED_SOURCES_DIR issue as I have. What is blocking this PR from being merged?

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

Successfully merging this pull request may close these issues.

Doesn't run as Context Menu Command in Xcode
3 participants