Page MenuHomeFreeBSD

sysutils/rsyslog: Fix libinotify build conflict
ClosedPublic

Authored by koobs on Jul 1 2019, 5:49 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 8, 5:24 PM
Unknown Object (File)
Thu, Nov 7, 8:06 PM
Unknown Object (File)
Sat, Nov 2, 9:13 AM
Unknown Object (File)
Oct 5 2024, 12:01 AM
Unknown Object (File)
Sep 19 2024, 4:29 AM
Unknown Object (File)
Sep 8 2024, 3:25 PM
Unknown Object (File)
Sep 7 2024, 5:49 PM
Unknown Object (File)
Sep 7 2024, 4:53 PM
Subscribers

Details

Summary
sysutils/rsyslog: Fix libinotify build conflict

ports r444784 added CONFLICTS_BUILD on devel/libinotify as the
build breaks when it is installed/available:

  imfile.c:42:10: fatal error: 'linux/types.h' file not found
  #include <linux/types.h>
           ^~~~~~~~~~~~~~~
  1 error generated.       

The inability to build a port based on the presence of an arbitrary port
is not a good user experience as anything can be installed in the user
environment.

In almost all cases, it is preferable to resolve the conflict explicitly, 
either by patching the parts of the build system that check for or define
functionality based on the presence of a library, or any methods available
in the build system to explicitly disable that particular functionality.

rsyslog does not contain any explicitly user-toggles for disabling inotify
support (optional, for the imfile module), but GNU configure provides
variables for each check that can be overriden.

This method is used in /usr/ports/Templates/config.site to provide cached
values to GNU configure scripts, improving performance/build times by
avoiding actually running the tests/checks.

This change sets two configure variables involved in inotify
function/header detection, which disables the functionality, allowing the
build to proceed even in libinotify's presence.

While I'm here, pet portlint:

  * FATAL: Makefile: [73]: use a tab (not space) after a variable name
  * WARN: Makefile: "USES" has to appear earlier.

Reported by: atari83 (via IRC)
Reviewed_by: matthew (maintainer)
Approved by: matthew (maintainer)
Differential_Revision: D20815
Test Plan
  • portlint: OK (4 x spurious pkg-message: possible use of absolute pathname)
  • testport: OK (poudriere: 12amd64)

Diff Detail

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

Event Timeline

Fix portlint issues:

  • FATAL: Makefile: [73]: use a tab (not space) after a variable name
  • WARN: Makefile: "USES" has to appear earlier.
This revision is now accepted and ready to land.Jul 7 2019, 10:54 AM

Oh, you didn't use my commit log message verbatim ;(~ *tear*

Thanks for committing, i would have been happy to take care of it :D