Page MenuHomeFreeBSD

comms/fldigi - fix build with clang
ClosedPublic

Authored by shurd on Jul 4 2014, 10:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 27 2024, 5:27 PM
Unknown Object (File)
Jan 31 2024, 6:27 PM
Unknown Object (File)
Jan 26 2024, 11:50 PM
Unknown Object (File)
Jan 24 2024, 9:52 PM
Unknown Object (File)
Jan 3 2024, 3:55 PM
Unknown Object (File)
Oct 17 2023, 7:15 PM
Unknown Object (File)
Sep 26 2023, 8:02 PM
Unknown Object (File)
Aug 20 2023, 2:12 PM
Subscribers
None

Details

Summary

Fix hashmap macro test to compile with Clang

Commit with: MFH: 2014Q3

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

shurd retitled this revision from to comms/fldigi - fix build with clang.
shurd updated this object.
shurd added reviewers: db, skreuzer.
skreuzer edited edge metadata.

Approved. Thanks

This revision is now accepted and ready to land.Jul 7 2014, 12:33 PM

I'd double check the configure.ac script here before sending fix upstream.
e.g. configure was modified to look for std::hash etc. to make it non specific to GCC or Clang since one of my earlier diffs I sent upstream was too specific.

Looking at configure.ac for example.

    1. std::hash or alternatives
  1. Set ac_cv_std_hash or ac_cv_std_tr1_hash to yes/no
  2. Define HAVE_STD_HASH or HAVE_STD_TR1_HASH in config.h
  3. Substitute BOOST_CPPFLAGS and BOOST_LDFLAGS in Makefile
  4. if falling back to Boost

AC_FLDIGI_HASH

That's my only concern. Approved of course.

shurd updated this revision to Diff 633.

Closed by commit rP361113 (authored by @shurd).