Page MenuHomeFreeBSD

Fix graphics/drawpile build, missing include.
ClosedPublic

Authored by adridg on Mar 20 2018, 2:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 12 2024, 4:32 AM
Unknown Object (File)
Feb 12 2024, 4:32 AM
Unknown Object (File)
Feb 12 2024, 4:32 AM
Unknown Object (File)
Feb 12 2024, 4:19 AM
Unknown Object (File)
Nov 16 2023, 10:32 PM
Unknown Object (File)
Nov 10 2023, 9:33 AM
Unknown Object (File)
Sep 3 2023, 10:52 PM
Unknown Object (File)
Sep 3 2023, 10:50 PM
Subscribers

Details

Summary

I don't know if this is because of my Qt version,
or clang, or newer CMake, though. Error message is in
the patch, and with the extra include it builds in
poudriere, amd64, 11.1.

Diff Detail

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

Event Timeline

adridg retitled this revision from Fix build, missing include. to Fix graphics/drawpile build, missing include..Mar 22 2018, 9:55 AM

It definitely is the CMake update, but it seems so random to need an extra include with a different CMake version. While this patch makes drawpile build, it deserves some extra investigation to find out why it is falling over.

This revision was not accepted when it landed; it landed in state Needs Review.Apr 2 2018, 8:12 PM
This revision was automatically updated to reflect the committed changes.

It's strange that I've never received any fallout messages.

You also don't need to bump ​PORTREVISION for such patch.

You wouldn't get fallout messages: this is pre-emptive before we reach the point of an exp-run with CMake. I don't think you would get a fallout message from an exp-run, either. I'll try to keep PORTREVISION bumps down in future.