microsoft / microsoft/vscode-cpptools-api
Include flags with relative directories always resolve relative to workspace
Personne n'a encore pris cette issue.
- Langage dominant
- TypeScript
- Étoiles
- 33
- Forks
- 20
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
Problem
If a SourceFileConfiguration includes a entry in compilerFragments like /I..\foo, cpptools will attempt to resolve that include directory relative to the VS Code workspace root and silently discard the flag if the directory does not exist. However, it's possible that the compiler process was invoked with a different working directory than the workspace root, and the relative include directory would have resolved successfully relative to the compiler's working directory.
Suggested fix
Add a new readonly directory?: string; property to SourceFileConfiguration that specifies the working directory of the compiler process. Relative paths in compilerFragments should resolve relative to directory, if it is defined.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par localiser la déclaration de SourceFileConfiguration et le code qui résout compilerFragments par rapport à la racine de l’espace de travail de VS Code. Vérifiez comment les répertoires de travail du compilateur sont représentés, puis assurez-vous qu’un répertoire défini est utilisé pour les chemins d’inclusion relatifs et que la nouvelle propriété facultative est exposée dans l’API publique.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- api
- Type d'issue
- Fonctionnalité
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 55/100