While this is applied for convenience we shouldn't silently hide fixups
Details
Details
Diff Detail
Diff Detail
- Repository
- R11 FreeBSD ports repository
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Mk/bsd.port.mk | ||
---|---|---|
3542 | I wonder whether it would make more sense to add it to DEV_WARNINGS and not to echo it manually. |
Mk/bsd.port.mk | ||
---|---|---|
3542 | DEV_WARNINGS and WARNINGS are like, static things, that run before everything, they can't really know what will happen during staging. |
Mk/bsd.port.mk | ||
---|---|---|
3542 | hm, then maybe manually check for the DEVELOPER mode? |
Mk/bsd.port.mk | ||
---|---|---|
3542 | I think it can be useful in general as this "silent fix" causes a different end result than what build log shows. |
Comment Actions
I can't seem to edit and propose a change for some reason, but, change it to something like this:
if [ -n "${DEVELOPER:Dyes}" ]; then \ ${ECHO_MSG} "===> File(s) found in lib/pkgconfig while correct path is libdata/pkgconfig"; \ ${ECHO_MSG} " Applying fix but consider using USES= pathfix or adjust install path"; \ fi; \