ansys / ansys/python-usd-viewer
Should the auto setup check for products ?
- Ngôn ngữ chính
- Python
- Star
- 0
- Fork
- 0
- Merge trung bình
- 1 ngày 16 giờ
- Pull request đã merge (30 ngày)
- 7
Mô tả
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.
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu bằng cách xác định script thiết lập tự động và cơ chế phát hiện vswhere hiện có, sau đó xem xét cách các bản cài đặt Visual Studio và khả năng hỗ trợ trình biên dịch C++ hiện được kiểm tra. Kiểm thử hành vi với một phiên bản Visual Studio được cài đặt bằng winget. Được xem là hoàn tất khi bản cài đặt dự kiến được phát hiện một cách đáng tin cậy, hoặc vấn đề được làm rõ thành một yêu cầu cụ thể.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- build-system
- Loại issue
- Tính năng
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 35/100