Ports with USES=localbase and lang/gcc* look under /usr/local/include by default. Unexpected consumers may auto-detect sysinfo() based on sys/sysinfo.h availability but assume the symbol is in libc, not a separate library. In shared libraries underlinking may go unnoticed. Steps to reproduce: 1. pkg install gcc9 libsysinfo xorg-minimal 2. CC=gcc9 make install clean -C/usr/ports/x11-drivers/xf86-video-intel 3. Adjust xorg.conf to use "intel" driver 4. startx 5. Livelock from VT switch to X server that failed to start
3.1. Add xorg.conf to use Option "AccelMethod" "SNA"
I've seen this in the past, and was unsure what the actual ask was, so without a patch I've been ignoring this since it was unclear what the desired change is. The word 'prefix' at the end is confusing. Is it really you want to install in ${PREFIX}/include/libsysinfo? And will all the programs that use it really do the right thing?
Created attachment 227098 [details] v0 (only few consumers adapted) (In reply to Warner Losh from comment #2) > without a patch I've been ignoring this since it was unclear what the desired change is After the pushback in bug 235240 I didn't want to waste time. Also, upstream Makefile is a bit crufty which makes patching ugly but not impossible. > Is it really you want to install in ${PREFIX}/include/libsysinfo? Yep. > And will all the programs that use it really do the right thing? Consumers that don't rely on "pkg-config --cflags" or "pkg-config --variable=includedir" may need to be patched. According to "git grep -Fl :devel/libsysinfo | wc -l" there are only 18 of those. In v0 I've patched only a few lightweight ports. Fixing this port is not my responsibility, so other work takes priority over rebuilding the affected ports and some of their heavy dependencies.
thanks Jan. I'll work with the other maintainers as needed to get this updated.
After due consideration, what's there today works. this is an request to rework the port in arbitrary ways, which I'm not ever going to do. There's too many ports that need to change to implement this, and the benefit is not worth it. Since nobody is willing to do the work to make this happen, I'm closing this bug to reduce the noise in my bug list.