ansys / ansys/python-usd-viewer
Should the auto setup check for products ?
- Vorherrschende Sprache
- Python
- Sterne
- 0
- Forks
- 0
- Ø Merge
- 1 T. 16 Std.
- Gemergte PRs (30 T.)
- 7
Beschreibung
This is mainly an open question and would probably be better as a discussion than an issue. Should the auto setup script check for installation using products that are expected to contain C++ compiler support ?
For example, I used `winget` to install visual studio and the auto setup script is not working. With a tiny modification:
```python
# Products that include C++ compiler support
vs_products = [
"Microsoft.VisualStudio.Product.BuildTools",
"Microsoft.VisualStudio.Product.Community",
"Microsoft.VisualStudio.Product.Professional",
"Microsoft.VisualStudio.Product.Enterprise",
]
vs_found = False
for vswhere_path in vswhere_paths:
if Path(vswhere_path).exists():
try:
# Search for any product that includes C++ build tools
result = subprocess.run(
[vswhere_path, "-latest", "-products"] + vs_products + ["-property", "installationPath"],
capture_output=True,
text=True,
check=True,
)
vs_path = result.stdout.strip()
if vs_path:
vs_found = True
break
except subprocess.CalledProcessError:
continue
```
The script no longer fails. I'm not sure that the `winget` installation is enough thugh. I'll follow the error message advices to install visual studio.
Beitragsleitfaden
Rechercherichtung
Beginne damit, das Skript für die automatische Einrichtung und seine bestehende vswhere-Erkennung zu lokalisieren, und prüfe anschließend, wie Visual Studio-Installationen und die Unterstützung für C++-Compiler derzeit überprüft werden. Teste das Verhalten mit einer über winget installierten Visual Studio-Instanz. Als erledigt gilt die Aufgabe, wenn die erwartete Installation zuverlässig erkannt wird oder das Problem zu einer konkreten Anforderung präzisiert ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- build-system
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 35/100