yonaskolb/XcodeGen

[BUG] Optional source path doesn't work if executing xcodegen from outer directory

Ouverte

#1 453 ouverte le 22 févr. 2024

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)Swift (895 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (8 714 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Given a project config:

name: OptionalSourcePath

targets:
  OptionalSourcePath:
    type: application
    platform: iOS
    deploymentTarget: "16.0"
    sources:
      - path: "First"
      - path: "Second"
        optional: true

and a directory structure of:

❯ tree dir
dir
├── First
├── Second
└── project.yml

3 directories, 1 file

calling xcodegen from within dir:

cd dir
xcodegen -s project.yml

generates a correct Xcode project.

However, calling xcodegen from outside of dir:

❯ basename $PWD
Downloads
❯ ls dir
First  project.yml  Second
❯ xcodegen -s dir/project.yml

Generates an Xcode project where the "Second" dir is missing from file tree: missing

❯ xcodegen --version
Version: 2.39.1

Guide contributeur

Direction de recherche
Étudiez comment XcodeGen résout les chemins sources lorsque le répertoire de travail actuel diffère du répertoire du projet. Concentrez vous sur les fichiers SpecParsing ou ProjectGenerator où les chemins sources sont traités. L'attribut optionnel peut affecter la résolution des chemins. Assurez vous que les chemins relatifs sont résolus par rapport à l'emplacement du fichier projet, et non par rapport au répertoire de travail actuel.
Stack technique
swift
Domaine
cli
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65