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

Added cURL script for Windows #59

Closed
wants to merge 8 commits into from
Closed

Conversation

Charlito33
Copy link
Member

Modified "setup_workspace.py" and "copy_dependencies.py" for Windows to automatically download and link cURL.

@Charlito33 Charlito33 self-assigned this May 11, 2024
@Charlito33 Charlito33 added the enhancement New feature or request label May 11, 2024
@b5i
Copy link
Member

b5i commented May 12, 2024

Peut être que ce serait mieux de mettre ça dans la branche network directement non ?

@Charlito33
Copy link
Member Author

Peut être que ce serait mieux de mettre ça dans la branche network directement non ?

Oui, ou sinon je merge avec main et tu rebase ?

@b5i
Copy link
Member

b5i commented May 12, 2024

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.

Copy link

@Zerbaib Zerbaib left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

juste propre

@paxo-rch
Copy link
Contributor

Et du coup ça fonctionne aussi sur les autres os?

@b5i
Copy link
Member

b5i commented May 19, 2024

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.

@Charlito33
Copy link
Member Author

Et du coup ça fonctionne aussi sur les autres os?

En soit le problème se pose que pour Windows où on est obligé de donner le chemin absolu pour la library.
Sur linux et macos vous avez juste à installer un package ?

Là pour le coup, j'ai juste copier ce qu'on faisait déjà pour SDL2.

@b5i
Copy link
Member

b5i commented May 19, 2024

En soit le problème se pose que pour Windows où on est obligé de donner le chemin absolu pour la library. Sur linux et macos vous avez juste à installer un package ?

C'est ça pour Linux et macOS faut juste link la dylib de SDL2 quand tu lances le programme.

@paxo-rch
Copy link
Contributor

Ce sera bon quand cette PR?

@Charlito33
Copy link
Member Author

Si cURL est implémenté, alors je pense qu'on peut la merge.

b5i and others added 7 commits August 14, 2024 11:18
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.
@b5i b5i closed this Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants