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

Update build-kdenlive.sh #11

Closed
wants to merge 1 commit into from

Conversation

carlosnewmusic
Copy link

Updating programs https://bitbucket.org/eigen/eigen git://code.dyne.org/frei0r.git and lame

Updating programs https://bitbucket.org/eigen/eigen git://code.dyne.org/frei0r.git and lame
@bmatherly
Copy link
Member

General comments:
We moved away from code.dyne.org because it was unreliable. Has the reliability improved?

Also, I don't care for leaving the old entries commented out. That's the whole reason for Git.

I'll let Dan chime in on the specific versions - I don't have any problem with them myself.

@ddennedy
Copy link
Member

ddennedy commented May 16, 2016

Someone from Kdenlive should approve changes to this script from now on, but if I do not hear from them in a while, I will approve this minor change.
I completely agree to remove the commented lines.
Regarding frei0r repo, the one to use now is https://github.com/dyne/frei0r
We should do that for all the scripts. My repo was a stop-gap for a good while, but I have recently deprecated it. Dyne is authoritative, and GitHub provides better reliability - win/win.

@vpinon
Copy link
Contributor

vpinon commented May 16, 2016

Hello,
Of course no problem for me...
Thanks a lot!

@carlosnewmusic
Copy link
Author

already put in all

@ddennedy
Copy link
Member

ddennedy commented Jun 1, 2016

I applied your changes manually.

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

Successfully merging this pull request may close these issues.

None yet

4 participants