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; \