[iOS] Swift compiler to use absolute path to SDK [chromium/src : main]

0 views
Skip to first unread message

Sylvain Defresne (Gerrit)

unread,
11:52 AM (5 hours ago) 11:52 AM
to Justin Novosad, Rohit Rao, Chromium LUCI CQ, chromium...@chromium.org, ios-r...@chromium.org
Attention needed from Justin Novosad and Rohit Rao

Sylvain Defresne added 1 comment

Patchset-level comments
File-level comment, Patchset 3 (Latest):
Sylvain Defresne . resolved

Please ask someone working on siso to review this.

I was explicitly asked to not use absolute path in any command-line in the generated ninja files. I think this request to not use absolute path (even for commands that are only run locally) still holds.

Open in Gerrit

Related details

Attention is currently required from:
  • Justin Novosad
  • Rohit Rao
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I50c7228c9fea615ca646229b25c0fb39e6e320cd
Gerrit-Change-Number: 7600704
Gerrit-PatchSet: 3
Gerrit-Owner: Justin Novosad <ju...@chromium.org>
Gerrit-Reviewer: Justin Novosad <ju...@chromium.org>
Gerrit-Reviewer: Rohit Rao <rohi...@chromium.org>
Gerrit-Reviewer: Sylvain Defresne <sdef...@chromium.org>
Gerrit-Attention: Justin Novosad <ju...@chromium.org>
Gerrit-Attention: Rohit Rao <rohi...@chromium.org>
Gerrit-Comment-Date: Tue, 24 Feb 2026 16:52:18 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

Sylvain Defresne (Gerrit)

unread,
11:53 AM (5 hours ago) 11:53 AM
to Justin Novosad, Rohit Rao, Chromium LUCI CQ, chromium...@chromium.org, ios-r...@chromium.org
Attention needed from Justin Novosad and Rohit Rao

Sylvain Defresne added 1 comment

Patchset-level comments
Sylvain Defresne . resolved

If we still need to use relative paths, you can change `swiftc.py` to check whether any component in `-sdk` path is a symlink and to resolve it at run time.

Gerrit-Comment-Date: Tue, 24 Feb 2026 16:53:26 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages