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

Remove FLP import, revert to GPLv2+ only #2904

Merged
merged 1 commit into from
Jul 12, 2016
Merged

Remove FLP import, revert to GPLv2+ only #2904

merged 1 commit into from
Jul 12, 2016

Conversation

grejppi
Copy link
Contributor

@grejppi grejppi commented Jul 10, 2016

The fact that searching for flp gives only three results somehow tells me it won't be missed. @LMMS/developers, Y/n?

This should also close #2752 and resolve the GPLv3 issues once and for all.

May still contain traces in the translation files. @liushuyu, should they be edited on Transifex, your lmms-translations repository, or here?

@Umcaruje
Copy link
Member

Yes from me, it didn't import properly anyways, and if it clashes with our license that's another reason to remove it.

@liushuyu
Copy link
Member

May still contain traces in the translation files.@liushuyu, should they be edited on Transifex, your lmms-translations repository, or here?

My lmms-translations repository is just a mirror of Transifex stash.
And if you don't want to refresh the translation file yourself, I will do the job. Just simply run lupdate -Iinclude/ src/ plugins/ -ts data/locale/en.ts on working copy of LMMS source directory.

And for the "traces" you have mentioned, Qt will mark these translations as deprecated by default instead of wiping them out, so do you want to just make them hidden or totally delete them?

@DeRobyJ
Copy link
Contributor

DeRobyJ commented Jul 11, 2016

I agree on this too: it's better to export the FLStudio project (or any other daw) to MIDI and import that in LMMS, so we should concentrate on MIDI rather than FLP files.

@grejppi
Copy link
Contributor Author

grejppi commented Jul 11, 2016

Alright, this has got quite a few 👍s already, so I guess I'll merge this in 24 hours unless someone else does it first unless there are no objections.

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.

4 participants