AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

ocio_deps.bat script can provide wrong instructions

Offen
#1,740 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Build Issue good first issue help wanted
Vorherrschende Sprache
C++
Sterne
2.1k
Forks
503
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Hello, fairly minor issue but:
(https://github.com/AcademySoftwareFoundation/OpenColorIO/blob/main/share/dev/windows/ocio_deps.bat#L116)
On line 116 or ocio_depts.bat script, the script instructs to use `ocio_deps --vs `, however, the script does not accept `--vs` as an argument.
I would have attempted a PR myself, however, I'm unsure what the intended use is, and it seems the `MSVS` variable is never used again, so maybe it only serves as a warning in case Visual Studio is not found?

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne mit share/dev/windows/ocio_deps.bat um Zeile 116 und prüfe, wie das Skript Argumente parst und die Variable MSVS verwendet. Bestätige den unterstützten Aufruf und ob die Meldung eine Warnung oder eine Anweisung ist. Erledigt ist es, wenn der angezeigte Befehl mit den vom Skript akzeptierten Argumenten und seinem vorgesehenen Visual-Studio-Verhalten übereinstimmt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Bereich
build-system
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.