Skip to content

Conversation

@jszumski
Copy link
Contributor

@jszumski jszumski commented Oct 6, 2025

Fixes a truncated DEVELOPER_DIR by quoting the path in case it has spaces like "/Applications/Xcode 26b2.app". Tools invoked from the inner output base will use the truncated value like this rules_swift_package_manager call:

INFO: Repository rules_swift_package_manager++swift_deps+swiftpkg_knit instantiated at:
  <builtin>: in <toplevel>
Repository rule swift_package defined at:
  /private/var/tmp/_bazel_jszumski/ed7e7efa198ffcfaf8285220a74bc6f4/rules_xcodeproj.noindex/build_output_base/external/rules_swift_package_manager+/swiftpkg/internal/swift_package.bzl:226:32: in <toplevel>
ERROR: /private/var/tmp/_bazel_jszumski/ed7e7efa198ffcfaf8285220a74bc6f4/rules_xcodeproj.noindex/build_output_base/external/rules_swift_package_manager+/swiftpkg/internal/repository_utils.bzl:68:13: An error occurred during the fetch of repository 'rules_swift_package_manager++swift_deps+swiftpkg_knit':
   Traceback (most recent call last):
	File "/private/var/tmp/_bazel_jszumski/ed7e7efa198ffcfaf8285220a74bc6f4/rules_xcodeproj.noindex/build_output_base/external/rules_swift_package_manager+/swiftpkg/internal/swift_package.bzl", line 53, column 33, in _swift_package_impl
		repo_rules.check_spm_version(repository_ctx, env = env)
	File "/private/var/tmp/_bazel_jszumski/ed7e7efa198ffcfaf8285220a74bc6f4/rules_xcodeproj.noindex/build_output_base/external/rules_swift_package_manager+/swiftpkg/internal/repo_rules.bzl", line 74, column 31, in _check_spm_version
		spm_ver = spm_versions.get(repository_ctx, env = env)
	File "/private/var/tmp/_bazel_jszumski/ed7e7efa198ffcfaf8285220a74bc6f4/rules_xcodeproj.noindex/build_output_base/external/rules_swift_package_manager+/swiftpkg/internal/spm_versions.bzl", line 39, column 49, in _get_version
		exec_out = repository_utils.exec_spm_command(
	File "/private/var/tmp/_bazel_jszumski/ed7e7efa198ffcfaf8285220a74bc6f4/rules_xcodeproj.noindex/build_output_base/external/rules_swift_package_manager+/swiftpkg/internal/repository_utils.bzl", line 68, column 13, in _execute_spm_command
		fail(err_msg_tpl.format(
Error in fail: Failed to execute SPM command. working_directory: , args: ["xcrun", "swift", "package", "--version"], return_code: 1
xcrun: error: missing DEVELOPER_DIR path: /Applications/Xcode

@jszumski jszumski requested a review from a team as a code owner October 6, 2025 21:05
@jszumski jszumski force-pushed the jszumski/fix-path-to-xcode-spaces branch from 374de40 to 31b544f Compare October 6, 2025 21:06
@jszumski jszumski force-pushed the jszumski/fix-path-to-xcode-spaces branch from 31b544f to 355cf89 Compare October 7, 2025 14:33
@brentleyjones brentleyjones merged commit ac524b3 into main Oct 7, 2025
9 of 12 checks passed
@brentleyjones brentleyjones deleted the jszumski/fix-path-to-xcode-spaces branch October 7, 2025 15:47
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.

4 participants