Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows Release builds failing due to missing deps (Patch File Fix) #2216

Closed
mat2718 opened this issue Feb 1, 2024 · 2 comments
Closed

Windows Release builds failing due to missing deps (Patch File Fix) #2216

mat2718 opened this issue Feb 1, 2024 · 2 comments

Comments

@mat2718
Copy link

mat2718 commented Feb 1, 2024

Hi! 👋

Firstly, thanks for your work on this project! 🙂

Today I used patch-package to patch [email protected] for the project I'm working on.

when building for release on react native windows this library fails due to missing dependencies that are listed for Debug but not in Release. this patch just adds the dependencies for release as well enabling a successful build. 😄 hope this helps!!

Here is the diff that solved my problem:

diff --git a/node_modules/react-native-svg/windows/RNSVG/RNSVG.vcxproj b/node_modules/react-native-svg/windows/RNSVG/RNSVG.vcxproj
index 11ad6f5..bd96537 100644
--- a/node_modules/react-native-svg/windows/RNSVG/RNSVG.vcxproj
+++ b/node_modules/react-native-svg/windows/RNSVG/RNSVG.vcxproj
@@ -120,6 +120,9 @@
     <ClCompile>
       <PreprocessorDefinitions>NDEBUG;%(PreprocessorDefinitions)</PreprocessorDefinitions>
     </ClCompile>
+    <Link>
+      <AdditionalDependencies>dxguid.lib;WindowsApp.lib;%(AdditionalDependencies)</AdditionalDependencies>
+    </Link>
   </ItemDefinitionGroup>
   <ItemGroup>
     <ClInclude Include="BrushView.h" />

This issue body was partially generated by patch-package.

@afonso-tsx
Copy link

bump, this works and is needed for windows support

@bohdanprog
Copy link
Member

Hello @mat2718,
As I see it was already implemented.
Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants