build failing with react native@0.81.5
Offen
Anfängerfreundlich
Dieses Issue hat noch niemand übernommen.
Needs: Author Feedback
Needs: Repro
- Vorherrschende Sprache
- C++
- Sterne
- 127k
- Forks
- 25.3k
- Ø Merge
- 1 T. 23 Std.
- Gemergte PRs (30 T.)
- 4
Beschreibung
Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch react-native@0.81.5 for the project I'm working on.
Here is the diff that solved my problem:
diff --git a/node_modules/react-native/ReactCommon/react/renderer/core/graphicsConversions.h b/node_modules/react-native/ReactCommon/react/renderer/core/graphicsConversions.h
index bb59351..bcff1d7 100644
--- a/node_modules/react-native/ReactCommon/react/renderer/core/graphicsConversions.h
+++ b/node_modules/react-native/ReactCommon/react/renderer/core/graphicsConversions.h
@@ -76,8 +76,11 @@ inline folly::dynamic toDynamic(const YGValue& dimension) {
return "stretch";
case YGUnitPoint:
return dimension.value;
- case YGUnitPercent:
- return std::format("{}%", dimension.value);
+ case YGUnitPercent: {
+ char buffer[32];
+ std::snprintf(buffer, sizeof(buffer), "%g%%", dimension.value);
+ return std::string(buffer);
+ }
}
return nullptr;
This issue body was partially generated by patch-package.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit ReactCommon/react/renderer/core/graphicsConversions.h und reproduziere den Build-Fehler von React Native 0.81.5. Vergleiche die im Issue gezeigte Umwandlung des Prozentwerts mit dem umgebenden Code und überprüfe anschließend, dass das Projekt ohne einen patch-package-Workaround baut.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp, react-native
- Bereich
- build-system, mobile
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Aktiv
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 85/100