swiftlang/swift

[SR-2489] -resource-dir lib/swift/macosx should be treated as -resource-dir lib/swift/

Open

#45,094 opened on Aug 25, 2016

View on GitHub
 (0 comments) (0 reactions) (0 assignees)Swift (69,989 stars) (10,719 forks)batch import
compilergood first issueimprovementlegacy driver

Description
Previous ID SR-2489
Radar None
Original Reporter @belkadan
Type Improvement
Votes 0
Component/s Compiler
Labels Improvement, Driver, StarterBug
Assignee None
Priority Medium

md5: 55654e442cde4ed3e942cc7a23ba8985

Issue Description:

Although -resource-dir is pretty much only useful for people working on the compiler, it's a common mistake to pass a particular platform directory (like lib/swift/macosx/) instead of the whole thing (lib/swift/). The Swift driver could check for this (by looking for something that ought to be in lib/swift/) and popping up to the parent directory if it doesn't find it (with a warning).

Contributor guide

Tech stack
swift
Domain
clideveloper experience
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Swift compiler driver codebasefile path handling
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70
Research direction
Locate the Swift driver source code where resource dir is parsed (likely in lib/Driver/). The issue suggests checking for a known file (like 'lib/swift/') in the provided directory, and if not found, warn and pop up to parent. No pull requests or comments exist, so the contributor should propose an implementation in a new PR. The original JIRA issue (SR 2489) may have additional context.