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)
Sun, Feb 22, 9:46 AM
Unknown Object (File)
Sun, Feb 22, 9:33 AM
Unknown Object (File)
Sat, Feb 21, 2:45 PM
Unknown Object (File)
Sat, Feb 21, 2:56 AM
Unknown Object (File)
Fri, Feb 20, 4:42 PM
Unknown Object (File)
Sun, Feb 8, 2:22 PM
Unknown Object (File)
Sat, Feb 7, 1:07 PM
Unknown Object (File)
Sat, Jan 31, 12:40 AM
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.