Skip to content

Commit

Permalink
Remove space-in-path replacement
Browse files Browse the repository at this point in the history
This replacement causes issues with importing indexstores.

In 965c373 this replacement was added to work around issues with `ar_wrapper`. I don't see `ar_wrapper` used anywhere in Bazel, so I believe this is safe to remove.
  • Loading branch information
brentleyjones committed Mar 13, 2024
1 parent 477df3a commit 6755b56
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions swift/internal/derived_files.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -113,10 +113,10 @@ def _intermediate_frontend_file_path(target_name, src):
"""
objs_dir = "{}_objs".format(target_name)

owner_rel_path = owner_relative_path(src).replace(" ", "__SPACE__")
safe_name = paths.basename(owner_rel_path)
owner_rel_path = owner_relative_path(src)
basename = paths.basename(owner_rel_path)

return paths.join(objs_dir, paths.dirname(owner_rel_path)), safe_name
return paths.join(objs_dir, paths.dirname(owner_rel_path)), basename

def _intermediate_object_file(actions, target_name, src):
"""Declares a file for an intermediate object file during compilation.
Expand Down

0 comments on commit 6755b56

Please sign in to comment.