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
Unknown Object (File)
Sat, Feb 1, 2:07 AM
Unknown Object (File)
Sat, Feb 1, 1:35 AM
Unknown Object (File)
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
Subscribers

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 49917
Build 46809: arc lint + arc unit

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.