Include Ubuntu Mono as a shipped font #1325 #1328 #1329 and Add Licencing info. #1332
Include Ubuntu Mono as a shipped font #1325 #1328 #1329 and Add Licencing info. #1332satnamiaman wants to merge 2 commits intoMudlet:developmentfrom satnamiaman:development
Conversation
|
|
|
#1334 is a massive PR and will take a really long while to get in - I'd rather have it not create a traffic jam of everything else. I'm not sold on it's proposal of hand-editing Qt's HTML in a raw text editor, that's not tenable going forward :( I think we should get this in and adjust #1334 as necessary. |
|
@SlySven thoughts? |
|
I am not seeing the change to add these files to the Indeed, it might be worth making the following change to to: however the existing code will handle this as it is if the calls to things like: are extended to include: where ... represents the other files to also copy out of the resource file... |
|
Good point, @zzbazinga do you think you'll be able to do these changes, or would you like some help and explanation? |
|
@vadi2 It will be very helpful if you provide some explanation. |
|
For sure. So we added the font files to the source code, but now we also need to do the following:
That should be it - if you get stuck, let me know! |
Given that you have already put the files in a suitable sub-directory you need to "Add existing files..." to the 💡 Hint: the files do not get added to the (binary) resources until the next time you compile (when Qt updates the internally managed |
|
@zzbazinga hey, how're you going with this? Can I help with an explanation? |
|
Oh, I was taking a look at this and I can see that the OP has promulgated this PR as his own development branch which means that he cannot easily update his repository to bring in any changes since he started the PR - IMHO he needs to create another branch in his local repository at the same point in the history - and work on that, closing this PR and then putting up a new PR based upon that new branch which only has to persist until it is merged into the development branch here. At the same time he can do a We will have to make inclusion of this font in the binary as a resource an optional feature because the licence terms are just outside of what Debian at least can consider as "Free" (see here on Wikipedia) - and they will not want to distribute it inside their packages. As it happens they already strip out the Bitstream Vera fonts - but that is because those are available within their packages and there is no need to carry a local copy of them but I will look to include a build option to control the inclusion - which @csmall will hopefully appreciate! I am putting this "on hold" as a placeholder and will submit a replacement shortly... |
|
Superseded by #1446. Thanks for your start on this, @zzbazinga! |
-Ubuntu Mono Font Added.
-Ubuntu Licence is added to ui/about_dialog.ui