#58167 closed defect (fixed)
bfs @1.3.3 - not UsingTheRightCompiler
Reported by: | kencu (Ken) | Owned by: | ylluminarious (George Plymale II) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | ||
Port: | bfs |
Description
bfs is a no-configure port, that for most builds just calls cc, whatever cc is.
Needs to set up a proper build environment, something like legacy-support or similar.
Some fixes have been partially done that are applying only to universal builds.
Change History (7)
comment:1 Changed 6 years ago by ylluminarious (George Plymale II)
comment:2 Changed 6 years ago by kencu (Ken)
Executing: cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_sysutils_bfs/bfs/work/bfs-1.3.3" && /usr/bin/make -j4 -w release make: Entering directory `/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_sysutils_bfs/bfs/work/bfs-1.3.3' cc -D__EXTENSIONS__ -D_ATFILE_SOURCE -D_BSD_SOURCE -D_DARWIN_C_SOURCE -D_DEFAULT_SOURCE -D_FILE_OFFSET_BITS=64 -D_GNU_SOURCE -DBFS_VERSION=\"1.3.3\" -std=c99 -g -Wall -Wmissing-declarations -Wstrict-prototypes -O3 -flto -DNDEBUG -MD -MP -MF bftw.d -c bftw.c -o bftw.o
it needs something like this added to the Portfile:
build.env CXX="${configure.cxx}" \ CXXFLAGS="${configure.cxxflags} [get_canonical_archflags cxx]" \ CC="${configure.cc}" \ CFLAGS="${configure.cflags} [get_canonical_archflags cc]" \ LDFLAGS="${configure.ldflags} [get_canonical_archflags ld]" \ PREFIX=${prefix}
but then also have to sort out the extras for +universal.
comment:3 Changed 6 years ago by kencu (Ken)
I should say -- IMHO this should all happen in base, but it doesn't presently.
comment:4 follow-up: 5 Changed 6 years ago by kencu (Ken)
comment:5 Changed 6 years ago by ylluminarious (George Plymale II)
Replying to kencu:
See also <https://trac.macports.org/wiki/UsingTheRightCompiler>
Hmm, I thought this was already taken care of in the original PR for this port. You can check it out for details. Is there something different about this situation?
comment:6 Changed 5 years ago by kencu (Ken)
Resolution: | → worksforme |
---|---|
Status: | assigned → closed |
I believe we have decided to leave this one as it is.
comment:7 Changed 5 years ago by jmroot (Joshua Root)
Resolution: | worksforme → fixed |
---|
Note: See
TracTickets for help on using
tickets.
Replying to kencu:
Umm, I'm not exactly sure what the issue is here. Could you give a specific example of the code that you want changed?