devel/ddd is a bit old and assumes Texinfo pre-5 when generating the info files. Attached are three patches that makes print/texinfo 5.2.20140707 happy. devel/ddd doesn't work with clang on stable/9, stable/10, nor head, nor with USE_GCC=4.7. Using USE_GCC=any works when world was compiled and installed with WITH_GCC=yes (and possibly with WITH_GNUCXX=yes, as I have set both) in /etc/src.conf. If you're stupid enough, like me, to set CPUTYPE?=corei7 or to a suitable value in /etc/make.conf, you'll likely run into trouble when using /usr/bin/gcc on stable/9, stable/10, and head. Maybe the USE_GCC= logic should consider the CPUTYPE variable and either simply undefine it when resorting to use /usr/bin/gcc, or translate the variable into a value acceptable to /usr/bin/gcc. I added these lines to my devel/ddd/Makefile.local: USE_GCC= any .ifdef CPUTYPE .if ! "${CPUTYPE}" == "core2" CPUTYPE=core2 .endif .endif
Created attachment 145072 [details] Patch for ddd/ddd.texi
Created attachment 145073 [details] Patch for ddd/macros.texi
Created attachment 145074 [details] Patch for ddd/ddd-themes.texi
notify maintainer for approval
This PR timed out, maintain approval no longer required. Anybody can commit this now.
(In reply to Trond.Endrestol from comment #0) > devel/ddd is a bit old and assumes Texinfo pre-5 when generating the info > files. Attached are three patches that makes print/texinfo 5.2.20140707 > happy. Please, in the future, don't provide individual patches to the software. Provide a single diff to the port itself, a diff that creates the patches and puts them in file. Otherwise the committer has to rename these files and move them him or herself. > > devel/ddd doesn't work with clang on stable/9, stable/10, nor head, nor with > USE_GCC=4.7. Using USE_GCC=any works when world was compiled and installed > with WITH_GCC=yes (and possibly with WITH_GNUCXX=yes, as I have set both) in > /etc/src.conf. > > If you're stupid enough, like me, to set CPUTYPE?=corei7 or to a suitable > value in /etc/make.conf, you'll likely run into trouble when using > /usr/bin/gcc on stable/9, stable/10, and head. > > Maybe the USE_GCC= logic should consider the CPUTYPE variable and either > simply undefine it when resorting to use /usr/bin/gcc, or translate the > variable into a value acceptable to /usr/bin/gcc. I added these lines to my > devel/ddd/Makefile.local: > > USE_GCC= any > > .ifdef CPUTYPE > .if ! "${CPUTYPE}" == "core2" > CPUTYPE=core2 > .endif > .endif I don't know what to do with this information. Are you saying you want to change the Makefile too? Where's the patch for that?
(In reply to John Marino from comment #6) > (In reply to Trond.Endrestol from comment #0) > > devel/ddd is a bit old and assumes Texinfo pre-5 when generating the info > > files. Attached are three patches that makes print/texinfo 5.2.20140707 > > happy. > > Please, in the future, don't provide individual patches to the software. > Provide a single diff to the port itself, a diff that creates the patches > and puts them in file. Otherwise the committer has to rename these files > and move them him or herself. Wilco. > > devel/ddd doesn't work with clang on stable/9, stable/10, nor head, nor with > > USE_GCC=4.7. Using USE_GCC=any works when world was compiled and installed > > with WITH_GCC=yes (and possibly with WITH_GNUCXX=yes, as I have set both) in > > /etc/src.conf. > > > > If you're stupid enough, like me, to set CPUTYPE?=corei7 or to a suitable > > value in /etc/make.conf, you'll likely run into trouble when using > > /usr/bin/gcc on stable/9, stable/10, and head. > > > > Maybe the USE_GCC= logic should consider the CPUTYPE variable and either > > simply undefine it when resorting to use /usr/bin/gcc, or translate the > > variable into a value acceptable to /usr/bin/gcc. I added these lines to my > > devel/ddd/Makefile.local: > > > > USE_GCC= any > > > > .ifdef CPUTYPE > > .if ! "${CPUTYPE}" == "core2" > > CPUTYPE=core2 > > .endif > > .endif > > > I don't know what to do with this information. Are you saying you want to > change the Makefile too? Where's the patch for that? There isn't any. The problem is that while corei7 is accepted by clang, the same value isn't accepted by gcc 4.2.1, nor is it translated to something accepted by gcc 4.2.1. I gather there isn't anything to do. In the long run, gcc will eventually be replaced by clang, and software such as devel/ddd will either need a rewrite or will be removed from the ports collection. I simply wanted to leave a trace of what I had done and how to work around it, just in case someone else trips over the same issue. Maybe that counts as a misuse of the bugzilla system. And maybe the morale of this story is simply to avoid the use of CPUTYPE in /etc/make.conf, it sure creates a lot of hassle in my encounters.
(In reply to Trond.Endrestol from comment #7) > And maybe the morale of this story is simply to avoid the use of CPUTYPE in > /etc/make.conf, it sure creates a lot of hassle in my encounters. That's good advice, especially for building the OS itself, much less the problems it causes in ports.
Another patch has been committed in r364155, which fixes build with clang and new texinfo