guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#57608] Upstreaming KDE Plasma and rest of packages


From: phodina
Subject: [bug#57608] Upstreaming KDE Plasma and rest of packages
Date: Tue, 22 Nov 2022 10:20:36 +0000

Hi Marius,

Thanks for the review and push to master!

> Awesome work Petr. :-)
> 
> I went through the branch and applied most of the patches. With a few
> changes:
> 
> * shortened multi-line URLs

Sorry I used the formatter ('guix style') and it set the URLs as such).

> * added (file-name ...) for git sources
> * removed knetworkmanager -- AFAICT it is identical to networkmanager-qt
> * switched to git-fetch for packages that were downloading tarballs from
> the KDE GitLab. This is because such autogenerated tarballs are not
> stable: they may get regenerated with a different hash.

Is this issue somewhere described? I've lately used more tarballs to git 
download on other packages. Is the issue connected with any Gitlab/Forge or 
just the KDE and the release scripts?

> * dropped the !! commits
> * Minor tweaks to synopses and descriptions

My apologies as I noticed there were few packages with wrong license or missing 
description.

> * A few commits had a random edit to a different package: I reverted
> those edits. An example:
> https://github.com/phodina/guix/commit/5eaa9c49a78eed419db7847668a55c079bad5b71

Caused due to rebasing and working on a large patchset. I'll split it next time 
and sent it by smaller parts.

> * Removed use of direct variable references, i.e. #$qtbase. Always use
> (search-input-file ...), (search-input-directory ...) or as a last
> resort #$(this-package-input "foo").

Thanks for the hint. I'll fix all the packages where I use this syntax!
 
> * Skipped commits that would trigger a lot of rebuilds, e.g. gpgme.

I'll submit it in new ticket.

> * Skipped cosmetic commits such as using G-expressions in Qt packages;
> mainly because of rebuilds, but also because they were not indented
> properly. Some also introduced direct #$variable references.

I'll fix the issues and submit it as a new ticket.

> * Dropped the plasma-desktop-service since it was not working for me.

Should we keep this ticket open to package also the plasma-desktop-service?
I'll try it now and see what causes the Plasma not to work.

> For later reference, when adding patches, please add a sentence or two
> at the top of the patch describing what it does. I did not edit the
> patches (except for a long file name), because I did not know what it
> was for. Presumably you did; future you will thank you!
> 
> Pushed to 'master' as fe3be8d5e0..82804298ad !

Sorry I didn't intend to add so many changes and it's been a challenge that 
taught me some new lessons.

I'll definitely add more description for the reviewers and keep it smaller so I 
doesn't take long time to review.

----
Petr





reply via email to

[Prev in Thread] Current Thread [Next in Thread]