P.S. $wl-no_fixup_chains: I tested this patch locally in configure, on mac OS Monterey. The warnings disappeared, and make check passed everything. I think it is working.
On Tue, May 14, 2024 at 10:58 PM Dave Allured - NOAA Affiliate < [email protected]> wrote: > GSL 2.7.101 with Apple clang 14 on Mac OS gives two of these warnings: > >> ld: warning: -undefined dynamic_lookup may not work with chained fixups > > > Web searching finds more than one solution. I like the following for > simplicity. In aclocal.m4, change this line as follows. It is guarded for > certain Mac software versions, so it should not impact other platforms. > Patch, then autoconf of course, to generate a new configure. This should > avoid chained fixups by expanding the use of lazy binding. > > < _lt_dar_allow_undefined='$wl-undefined ${wl}dynamic_lookup' ;; >> > _lt_dar_allow_undefined='$wl-undefined ${wl}dynamic_lookup >> $wl-no_fixup_chains' ;; > > > This patch was inspired by the same fix a year ago, in another package: > https://github.com/lcm-proj/lcm/pull/416 > "Compared to using chained fix-ups, the only potential downside for us is > a slightly larger dylib" > > You might think that something like this should already be in GNU autoconf > and aclocal. My guess is that GNU is simply not caught up with recent Mac > issues, for some reason. > > > On Tue, May 14, 2024 at 5:50 PM Dave Allured - NOAA Affiliate < > [email protected]> wrote: > >> Patrick, thank you for your efforts so far to try to make Apple clang >> happy. It is good to see all those filter/movstat/enum problems cleaned up. >> >> I find 234 notes or warnings remaining. Almost all of them are fabs >> related, so probably easy fixes. See attached files for make and make >> check. If I could save you a release cycle by running a few short tests on >> my Mac, let me know. >> >> On the two dynamic_lookup warnings, I have a proposed simple fix. That >> is a messy, platform dependent problem. Hold off on that until I can send >> my proposal and explanation. >> >> >> On Tue, May 14, 2024 at 2:49 PM Patrick Alken <[email protected]> wrote: >> >>> Thank you all again for your reports. I have (I think) fixed all the >>> compiler warnings related to filter/movstat, as well as the fabs issues. >>> I would greatly appreciate it if you could do another round of testing >>> on this file: >>> >>> ftp://alpha.gnu.org/gnu/gsl/gsl-2.7.101.tar.gz >>> >>> Thanks, >>> >>> Patrick >>> >>> On 5/12/24 13:34, Patrick Alken wrote: >>> > [External email - use caution] >>> > >>> > >>> > All, thank you for your testing reports. I believe I have addressed all >>> > the compiler warnings and other issues raised. I have uploaded a new >>> > test release here: >>> > >>> > >>> > ftp://alpha.gnu.org/gnu/gsl/gsl-2.7.100.tar.gz >>> > >>> > >>> > I would greatly appreciate if everyone can run the tests again: >>> > >>> > ./configure && make && make check >>> > >>> > Thanks, >>> > >>> > Patrick >>> > >>> > >>> > On 5/10/24 16:04, Patrick Alken wrote: >>> >> [External email - use caution] >>> >> >>> >> >>> >> Dear all, >>> >> >>> >> It is time to make a new GSL release. I have uploaded a test release >>> >> to: >>> >> >>> >> ftp://alpha.gnu.org/gnu/gsl/gsl-2.7.99.tar.gz >>> >> >>> >> >>> >> ftp://alpha.gnu.org/gnu/gsl/gsl-2.7.99.tar.gz.sig >>> >> >>> >> >>> >> >>> >> All reports are welcome - anyone who can test on various platforms >>> would >>> >> be appreciated (Linux, BSD, Mac OS, Windows). >>> >> >>> >> Please try testing the build: >>> >> >>> >> ./configure && make && make check >>> >> >>> >> If you wish you can test building the documentation, but you will need >>> >> to install the python-based sphinx software first: >>> >> >>> >> 1. pip install -U --user Sphinx >>> >> >>> >> 2. pip install -U --user sphinx_rtd_theme >>> >> >>> >> (the --user flag will install in your home directory rather than >>> >> system-wide) >>> >> >>> >> 3. cd doc ; make html >>> >> >>> >> Please report any successes/failures. >>> >> >>> >> Thanks, >>> >> Patrick >>> >>
