Alexander, The rational for the request to add these new environmentals was to handle those packages that currently need the X11 includes passed on CFLAGS rather than CPPFLAGS.
% grep -R X11 * | grep CFLAGS | grep info gnome/gedit.info:ConfigureParams: --disable-static --disable-scrollkeeper --disable-schemas-install --mandir=%p/share/man --libexecdir=%p/lib --enable-dependency-tracking PKG_CONFIG_PATH="%p/lib/glib-2.0/pkgconfig-strict:%p/lib/fontconfig2/lib/pkgconfig:$PKG_CONFIG_PATH" PYTHON=%p/bin/python2.7 F77=no --enable-gtk-doc EGG_SMCLIENT_CFLAGS=" " EGG_SMCLIENT_LIBS="-L/usr/X11R6/lib -lSM -lICE -lX11" --disable-silent-rules X11_CFLAGS=" " X11_LIBS=" " gnome/gnome-games.info: SMCLIENT_CFLAGS=" " SMCLIENT_LIBS="-lSM -lICE -L/usr/X11/lib -L/usr/X11R6/lib" \ gnome/gnome-session.info:ConfigureParams: --mandir=%p/share/man --disable-docbook-docs --disable-schemas-install --enable-dependency-tracking --disable-static PKG_CONFIG_PATH="%p/lib/glib-2.0/pkgconfig-strict:%p/lib/fontconfig2/lib/pkgconfig:$PKG_CONFIG_PATH" --x-i=/usr/X11R6/include --x-l=/usr/X11R6/lib SM_CFLAGS=" " SM_LIBS="-lSM -L/usr/X11/lib -L/usr/X11R6/lib" ICE_CFLAGS=" " ICE_LIBS=-lICE XTEST_CFLAGS=" " XTEST_LIBS=-lXtst XRENDER_CFLAGS=" " XRENDER_LIBS=-lXrender SPLASH_CFLAGS=" " SPLASH_LIBS=" " --disable-silent-rules gnome/gnome-terminal.info:ConfigureParams: --disable-scrollkeeper --disable-schemas-install --enable-dependency-tracking --disable-static PKG_CONFIG_PATH="%p/lib/glib-2.0/pkgconfig-strict:%p/lib/fontconfig2/lib/pkgconfig:$PKG_CONFIG_PATH" F77=no SMCLIENT_CFLAGS="-I/usr/X11/include -I/usr/X11R6/include" SMCLIENT_LIBS="-lSM -lICE -lX11 -L/usr/X11/lib -L/usr/X11R6/lib" gnome/gnome-utils.info: XSHAPE_CFLAGS=" " XSHAPE_LIBS="-lXext -lX11" gnome/libgnomekbd2.info:ConfigureParams: --enable-schemas-install --libexecdir=%p/lib/%n --mandir=%p/share/man --enable-dependency-tracking PKG_CONFIG_PATH="%p/lib/glib-2.0/pkgconfig-strict:%p/lib/fontconfig2/lib/pkgconfig:$PKG_CONFIG_PATH" --disable-static --x-libraries=/usr/X11R6/lib --x-includes=/usr/X11R6/include DBUS_CFLAGS=" " DBUS_LIBS=" " gnome/yelp.info: SMCLIENT_CFLAGS=" " SMCLIENT_LIBS="-lSM -L/usr/X11/lib -L/usr/X11R6/lib" \ graphics/ffmpeg.info:SetCFLAGS: -I%p/lib/libspeex1/include -I%p/include -I/usr/X11R6/include -MD graphics/libavcodec53-shlibs.info:SetCFLAGS: -I%p/lib/libspeex1/include -I%p/include -I/usr/X11R6/include -MD graphics/libavcodec54-0.11-shlibs.info:SetCFLAGS: -I%p/lib/libspeex1/include -I%p/include -I/usr/X11R6/include -MD graphics/libavcodec54-1.0-shlibs.info:SetCFLAGS: -I%p/lib/libspeex1/include -I%p/include -I/usr/X11R6/include -MD graphics/libavcodec54-1.2-shlibs.info:SetCFLAGS: -I%p/lib/libspeex1/include -I%p/include -I/usr/X11R6/include -MD graphics/libavcodec55-2.0-shlibs.info:SetCFLAGS: -I%p/lib/libspeex1/include -I%p/include -I/usr/X11R6/include -MD graphics/libavcodec56-2.4-shlibs.info:SetCFLAGS: -I%p/lib/libspeex1/include -I%p/include -I/usr/X11R6/include -MD sci/xcircuit-testing.info:SetCFLAGS: -I/usr/X11/include -fno-common sci/xcircuit-testing.info:Added -I/usr/X11/include to CFLAGS because it wouldn't work with sci/xcircuit.info:SetCFLAGS: -I/usr/X11/include -fno-common sci/xcircuit.info:Added -I/usr/X11/include to CFLAGS because it wouldn't work with utils/expect-10.10.info:SetCFLAGS: -I%p/include -I/usr/X11R6/include utils/expect-10.9.info:SetCFLAGS: -I%p/include -I/usr/X11R6/include utils/expect.info:SetCFLAGS: -I%p/include -I/usr/X11R6/include x11/freeglut.info:SetCFLAGS: -O3 -DTARGET_HOST_POSIX_X11 -I%p/include -I/usr/X11R6/include x11/openmotif3.info:SetCFLAGS: -I%p/include -I/usr/X11R6/include -DXNO_MTSAFE_STRINGAPI -DXNO_MTSAFE_PWDAPI -DXNO_MTSAFE_TIMEAPI -O2 -g x11/openmotif3.info: ./configure %c X_CFLAGS="-I%p/include -I/usr/X11R6/include" X_LIBS="-L%p/lib -L/usr/X11R6/lib" x11/openmotif4.info:SetCFLAGS: -I%p/include -I/usr/X11R6/include -DXNO_MTSAFE_STRINGAPI -DXNO_MTSAFE_PWDAPI -DXNO_MTSAFE_TIMEAPI Likewise, some of the 107 usages of --x-includes=/usr/X11R6/include in the current package set may still be required if those are due to their configure scripts testing internally for X11 header support using CFLAGS rather than CPPFLAGS. Jack On Tue, Feb 10, 2015 at 10:14 PM, Alexander Hansen <alexanderk.han...@gmail.com> wrote: > > On Feb 10, 2015, at 6:44 PM, Jack Howarth <howarth.at.f...@gmail.com> wrote: > > Hanspeter, > Now that a working implementation for the automatic appending of > the appropriate X11 include and linker flags to CPPFLAGS and LDFLAGS > has been checked in to fink git... > > http://sourceforge.net/p/fink/patches/158/ > > > It’s in a branch covered by a pull request. > > And before we call it “working”, I’d like to verify that it works _in all > aspects_. For example, does your automatic implementation give the right > results from “fink dumpinfo -eLDFLAGS <packagename>” and “fink dumpinfo > -eCPPFLAGS <packagename>” for packages that BuildDepend on x11-dev? > > > we need the to implement the other half required for fully migrating > off of depending on the X11 convenience symlinks. If you can enhance > the existing fink-buildenv-modules.info to provide build time > variables for XORG_INCL_PATH and XORG_LINK_PATH which would be > conditionally set to > > XORG_INCL_PATH=/usr/X11/include > XORG_LIB_PATH=/usr/X11/lib > > when the running OS is darwin11 or earlier and > > XORG_INCL_PATH=/opt/X11/include > XORG_LIB_PATH=/opt/X11/lib > > otherwise, this would allow package maintainers to stop hard-coding in > the fully deprecated /usr/X11R based paths and the partially > deprecated /usr/X11 paths. > Jack > > > Please elaborate upon why just allowing individual .info to set CPPFLAGS and > LDFLAGS based on XORG_INCL_PATH and XORG_LIB_PATH from fink-buildenv-modules > isn’t sufficient. Having the environment get set by default by fink is > less transparent to maintainers than doing it in individual .info files. > > -- > Alexander Hansen, Ph.D. > Fink User Liaison > ------------------------------------------------------------------------------ Dive into the World of Parallel Programming. The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net/ _______________________________________________ Fink-devel mailing list Fink-devel@lists.sourceforge.net List archive: http://news.gmane.org/gmane.os.apple.fink.devel Subscription management: https://lists.sourceforge.net/lists/listinfo/fink-devel