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
F156727562: D38692.id.diff
Fri, May 15, 11:04 PM
Unknown Object (File)
Fri, May 15, 6:53 AM
Unknown Object (File)
Fri, May 15, 5:44 AM
Unknown Object (File)
Thu, May 14, 6:44 PM
Unknown Object (File)
Mon, May 11, 8:32 PM
Unknown Object (File)
Mon, May 11, 1:55 AM
Unknown Object (File)
Mon, May 11, 1:46 AM
Unknown Object (File)
Sun, May 10, 3:52 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.