Page MenuHomeFreeBSD

sysutils/lxqt-config: Fix build after new plasma
ClosedPublic

Authored by zirias on Feb 20 2023, 1:26 PM.
Tags
None
Referenced Files
F109042053: D38692.id.diff
Fri, Jan 31, 12:56 AM
Unknown Object (File)
Mon, Jan 27, 8:16 PM
Unknown Object (File)
Sun, Jan 26, 7:41 PM
Unknown Object (File)
Thu, Jan 23, 10:34 PM
Unknown Object (File)
Thu, Jan 23, 6:55 AM
Unknown Object (File)
Sun, Jan 19, 10:02 PM
Unknown Object (File)
Sun, Jan 19, 7:36 PM
Unknown Object (File)
Dec 30 2024, 12:57 PM
Subscribers

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Thanks for looking into this! Looks good but you could perhaps use upstream with PATCH_SITES and PATCHFILE since they (also ) did the changes https://github.com/lxqt/lxqt-config/commit/6add4e4f0040693e7c4242fbae48c9d3200768.patch ? Just an opinion, I use to do that with non released upstream changes, it makes it easier for me to know which patches are belonging to upstream, and expected in next release..

Thanks I already used that myself. Unfortunately, there's no common agreement this is a good idea. But if you ask me, I prefer adding upstream patches that way!
The only reason this isn't committed yet is my test-build (for smoke-testing the change) is still running :-)

Updated to fetch necessary patch directly from upstream, test builds fine.

This revision was not accepted when it landed; it landed in state Needs Review.Feb 20 2023, 10:18 PM
This revision was automatically updated to reflect the committed changes.