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

Rpm building with qmake #7

Open
wants to merge 32 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@topiasv
Copy link

topiasv commented Sep 29, 2017

Merge @helicalgear and @knokmki612 forks for easy rpm building.

@topiasv topiasv referenced this pull request Sep 29, 2017

Open

RPM build #9

@knokmki612

This comment has been minimized.

Copy link

knokmki612 commented Nov 1, 2018

@topiasv
2 years ago, I worked about setting qmake to build easy this package. But @sleepsounds develops on only SFOS smartphone(this means they develops without pc, mersdk, emulator!). I heard it after I worked. So my change didn't helps @sleepsounds.

On the other hand, they still active to maintain the code with SFOS update.
This is why I started to maintain the package and merge these code.

We(@sleepsounds @helicalgear @knokmki612) know each other in Twitter, and talk sometimes about this package.

I have focusing to maintain pro/spec/patch files in my scope, but welcome about your adding new feature or fixes. if you send me PR to my repo, I'm ready to look into your works.

@topiasv

This comment has been minimized.

Copy link
Author

topiasv commented Nov 1, 2018

I see. If you have @sleepsounds contact details, I would like to ask his formal permission to use his code perhaps under some free license if possible. I think it would be beneficial to release a package in openrepos for everyone to download. I wanted to do that but since I couldn't get in contact with the original author, I didn't think it would have been fair without their consent.

I suggest to @sleepsounds that he'd add some sort of license file into the repository so we and other could keep developing this under clear terms. What kind of license he adds is of course his decision but me and the rest of the community would be very grateful if we could publish this on the openrepos.

If you @knokmki612 @helicalgear @sleepsounds are interested in maintaining this package together, we could set up some group where we can discuss and exchange our knowledge about this keyboard. I have a few improvements in mind that I've already been working on. I found you guys on twitter and added you there now (@kyuu9sha). If you're interested then let's set up a group somewhere like twitter/telegram/line whatever so we can talk about this more.

Cheers!

@knokmki612

This comment has been minimized.

Copy link

knokmki612 commented Nov 2, 2018

I suggest to @sleepsounds that he'd add some sort of license file into the repository

Actually, I already published the package to openrepos which I build without solving license issue...
https://openrepos.net/content/knokmki612/japanese-flick-keyboard-patch-fork

But even now Licensing is important to our development. I agree with your suggestion.

topiasv and others added some commits Nov 3, 2018

Merge pull request #1 from sleepsounds/master
Added licence file (LGPLv2.1)
Merge pull request #2 from knokmki612/remove_var_from_pro
Remove $$original $$patched variables
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.