plugins icon indicating copy to clipboard operation
plugins copied to clipboard

Add lightning-qt in tree from darosior/lightning-qt

Open darosior opened this issue 4 years ago • 8 comments
trafficstars

It deserves more love than i can afford to give it at the moment, hopefully it can be better maintained here as it appears that people are actually using it.

However it's ancient: last time i contributed to it was more than 2 years ago, since then i've been only merging bugfixes with months of delay... So i still need to clean it up and test it. I also need to change the way it's installed to get rid of committing big blobs to the repo.

darosior avatar Aug 20 '21 13:08 darosior

Awesome work @darosior, happy to move this over here and get some more eyes on it.

Just some due diligence questions:

  • What license are the assets under?
  • I assume the code is ok with a permissive BSD-MIT license?
  • Any pending issues that should be replicated here?

cdecker avatar Aug 25 '21 16:08 cdecker

What license are the assets under?

I had a README for the icons (which i should remove in this PR btw) indicating their origin. Most were taken from the Bitcoin Core project and then modified by myself (to have them in Yellow, much Lightning designing) a while back. The Lightning one was taken from https://paomedia.github.io/small-n-flat/ and licensed under CC0. The black ones recently contributed by @grazhopr are available on https://www.freepng.fr . For instance the thrash icon is and they claim to provide these icons free for use in personal projects not intended to be used as commercial products.

I assume the code is ok with a permissive BSD-MIT license?

Yes. The repository was previously under a BSD 3-clause license. Pinging the contributors for full disclosure: @vincenzopalazzo @grazhopr @rsbondi @jb55 @bruuuuuuhhhhhhhh @vadorovsky FYI i'm upstreaming lightning-qt to which you contributed under a BSD 3-clause licence to the lightningd/plugins repo where the code could be given a BSD or MIT license.

Any pending issues that should be replicated here?

A few that my update should fix, another one that i would probably open here, and a lot of "good first issues" i can think of to open here :)

darosior avatar Aug 30 '21 10:08 darosior

Ok, so i did another round of due diligence for the icons:

  • Removed the icons i could not find the provenance of (have been replaced)
  • Added the provenance of each and every icon, added a note to the Contributing section

I also did another round of updates and made some basic testing. The code is not super clean, but the software is functional.

darosior avatar Sep 15 '21 11:09 darosior

Hi, really sorry taken a long long time to get back on this. For the icons then they are all taken from Bitcoin Core and coloured the same yellow. The only icon that wasnt was the payment_expired which I'd planned to create a new one from combining the clock.png and warning.png (but I didnt :) I just forgot and temporarily found one that looked similar. I'll create i anyway if of use. apologies again for late response and for not being clearer on teh icons...

grazhopr avatar Sep 15 '21 11:09 grazhopr

Oh just after i replaced them all :sob:

apologies again for late response and for not being clearer on teh icons...

Sure, thanks for the head up!

darosior avatar Sep 15 '21 11:09 darosior

Oh just after i replaced them all

apologies again for late response and for not being clearer on teh icons...

Sure, thanks for the head up!

Damn, sorry !

grazhopr avatar Sep 15 '21 11:09 grazhopr

I had forgotten about this PR.. I'll get back to rebase it. If anyone cares and gets to it before i do please feel free to consider this as up for grab. (And maybe take @icota's fix from https://github.com/darosior/lightning-qt/pull/20.)

darosior avatar Oct 24 '22 12:10 darosior

Needs rebase, CI should be working now

chrisguida avatar Feb 07 '24 16:02 chrisguida