Skip to content

Add $$QMAKE_LIBS_EXECINFO#1623

Closed
lbartoletti wants to merge 1 commit into
nullptrlabs:developfrom
lbartoletti:patch-1
Closed

Add $$QMAKE_LIBS_EXECINFO#1623
lbartoletti wants to merge 1 commit into
nullptrlabs:developfrom
lbartoletti:patch-1

Conversation

@lbartoletti
Copy link
Copy Markdown

Unbreak build on FreeBSD (and other BSD I think so).

$$QMAKE_LIBS_EXECINFO will add -lexecinfo if required

Unbreak build on FreeBSD (and other BSD I think so).

$$QMAKE_LIBS_EXECINFO will add -lexecinfo if required
rkhaotix added a commit that referenced this pull request Jan 3, 2022
apps/apps.pri

Added the instruction $$QMAKE_LIBS_EXECINFO (PR #1623) to apps/
pgmodeler/pgmodeler.pro
@rkhaotix
Copy link
Copy Markdown
Member

rkhaotix commented Jan 3, 2022

@lbartoletti Thanks for pointing this. While testing locally your solution just to confirm that the build doesn't break in other OSes I found the opportunity to make a small refactor on executables subprojects. I also added the $$QMAKE_LIBS_EXECINFO apps/pgmodeler/pgmodeler.pro just like your PR. Could you please test the latest commit on develop branch and see if it builds correctly on BSD? Thanks.

@lbartoletti
Copy link
Copy Markdown
Author

@lbartoletti Thanks for pointing this. While testing locally your solution just to confirm that the build doesn't break in other OSes I found the opportunity to make a small refactor on executables subprojects. I also added the $$QMAKE_LIBS_EXECINFO apps/pgmodeler/pgmodeler.pro just like your PR. Could you please test the latest commit on develop branch and see if it builds correctly on BSD? Thanks.

@rkhaotix Thanks! I can confirm it's ok on FreeBSD.

@lbartoletti lbartoletti closed this Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants