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

Add curl #1

Closed
wants to merge 1 commit into from
Closed

Add curl #1

wants to merge 1 commit into from

Conversation

TiCubius
Copy link

@TiCubius TiCubius commented Apr 9, 2016

J'avais besoins du module RTMP pour un de mes projets, j'ai donc adapté ton script.
A toi de voir si tu souhaites faire cette modification, bien que RTMP ne soit utilisé que dans de très rare cas puisque ce n'est pas utile pour le web, mais pour du streaming twitch par exemple.

Ceci étant dit, j'ai également ajouté curl à la liste des apt-get install puisque celui-ci n'était pas installé sur mon système mais était utilisé dans le script.

@angristan
Copy link
Owner

Salut, merci du signalement pour curl.

As-tu testé le fonctionnement du script avec ton patch ?

@TiCubius TiCubius changed the title Added RTMP-Module support Add curl Apr 10, 2016
@TiCubius
Copy link
Author

Chez moi, l'ajout de ce module RTMP fonctionnait parfaitement ^^'

@TiCubius
Copy link
Author

Je vois que tu as toi-même corrigé curl, je ferme donc ce PR :)

@TiCubius TiCubius closed this Apr 10, 2016
@angristan
Copy link
Owner

Chez moi, l'ajout de ce module RTMP fonctionnait parfaitement ^^'

C'est ce que je voulais savoir, si tu avais testé avec le script ^^

Je vois que tu as toi-même corrigé curl, je ferme donc ce PR :)

Je savais pas si tu allais modifier ta PR désolé :3

angristan pushed a commit that referenced this pull request Feb 18, 2017
Cloudflare tls resizing + LibreSSL dev 2.5.1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants