-
Notifications
You must be signed in to change notification settings - Fork 16
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
Added cURL script for Windows #59
Conversation
Peut être que ce serait mieux de mettre ça dans la branche network directement non ? |
Oui, ou sinon je merge avec |
Je pense que c'est quand même mieux de tout mettre dans la même branche pour ne pas tout éparpiller et rebaser tout à chaque fois que ça arrive, comme ça à la fin y'a que un rebase en tout. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
juste propre
Et du coup ça fonctionne aussi sur les autres os? |
Pour l'instant faudrait pas merge ça car je vais rebase dessus et ça sera push en même temps que la PR pour le module network en entier. |
En soit le problème se pose que pour Windows où on est obligé de donner le chemin absolu pour la library. Là pour le coup, j'ai juste copier ce qu'on faisait déjà pour SDL2. |
C'est ça pour Linux et macOS faut juste link la dylib de SDL2 quand tu lances le programme. |
Ce sera bon quand cette PR? |
Si cURL est implémenté, alors je pense qu'on peut la merge. |
The library setup process for the cURL library has been bootstrapped and integrated into the project's PlatformIO environment. The changes also include refining the removal of temporary files as well as updating the workspace setup python script to download and setup cURL, extract it, check its existence, and copy the relevant DLL.
This commit introduces a function called 'cURL_copyDLL' to copy libcurl-x64.dll to the build directory on Windows platform. Any build will now also check for the presence of this DLL and copy it if not found, before proceeding.
9294108
to
5957017
Compare
Modified "setup_workspace.py" and "copy_dependencies.py" for Windows to automatically download and link cURL.