#33821: base: add a default +debug and +no_opt variant -------------------------------------------+-------------------------------- Reporter: sean.michael.farley@… | Owner: macports-tickets@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: base | Version: 2.0.4 Keywords: | Port: -------------------------------------------+-------------------------------- Comment(by sean.michael.farley@…): Replying to [comment:1 ryandesign@…]:

I'm not certain if we want this.

Certainly, we [wiki:PortfileRecipes#default_variants don't want variants whose names begin with "no_" anymore]. But on the other hand we don't want to add a default "opt" variant to all ports either; we'd have to rebuild all our binary packages. Could disabling optimizations simply be part of the debug variant?

Why would a normal user want a debug variant? Granted we have it in the cmake portgroup but I'm not sure why we have that either.

First, having a debug variant is vitally important for macport users, particularly those that build code on top of certain libraries. Take, for instance, PETSc. PETSc can be built on top of gcc4{4,5,6,7}, metis, parmetis, suitesparse, hypre, superlu, superlu_dist, mpich2 (or openmpi), triangle, tetgen, fftw-3, ml, sundials, mumps (which implies scalapack and blacs), hdf5-18, netcdf, libyaml, and valgrind. Adding a +debug variant to all these ports is ... tedious. But without debug symbols being built (and then stashed into a .dSYM) it would be impossible to use gdb my own code that uses PETSc. More over, I found the +debug particularly useful in building my own gfortran port so that gdb worked as expected. I agree about the +no_opt variant. I did originally have -O0 logic in the +debug variant but that ran into problems with building mpfr. The issue is multi-faceted, unfortunately, with zero optimization. Depending on the compiler, there are different settings for which optimization to use (if nothing is specified most compilers use -O2). Using anything more than -O0 causes problems with debugging because a lot of variables are optimized out, e.g. using 'print var' in gdb causes a hit-or-miss of "variable is optimized out". So, clearly, not every port can work with -O0 but then again it is *extremely* useful to have a +debug variant to keep the .dSYM files around. I don't know how to proceed. I'm ok with dropping the +no_opt and making each portfile write its own logic for -O0. Probably something like, {{{ pre-configure { if {[variant_isset debug]} { configure.cflags-append -O0 ... } } }}} As an aside for the naming convention, would "+optimization_debug" (or +debug_opt?) have worked? Also, since this is on the subject of variants, a more dangerous idea I had was based off a suggestion on the list: wanting to compile different ports with the same compiler. For example, having a default +gcc44 ... +gcc47, +clang, +system_gcc, +llvm_gcc, etc. Thoughts? (There would be similar problems as having -O0, i.e. some ports wouldn't be able to handle it or would have a cyclic dependency) -- Ticket URL: <https://trac.macports.org/ticket/33821#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS

#33821: base: add a default +debug and +no_opt variant -------------------------------------------+-------------------------------- Reporter: sean.michael.farley@… | Owner: macports-tickets@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: base | Version: 2.0.4 Keywords: | Port: -------------------------------------------+-------------------------------- Comment(by sean.michael.farley@…): Replying to [comment:1 ryandesign@…]:

I'm not certain if we want this.

Certainly, we [wiki:PortfileRecipes#default_variants don't want variants whose names begin with "no_" anymore]. But on the other hand we don't want to add a default "opt" variant to all ports either; we'd have to rebuild all our binary packages. Could disabling optimizations simply be part of the debug variant?

Why would a normal user want a debug variant? Granted we have it in the cmake portgroup but I'm not sure why we have that either.

Are there any more comments on this patch? -- Ticket URL: <https://trac.macports.org/ticket/33821#comment:4> MacPorts <http://www.macports.org/> Ports system for Mac OS

#33821: base: add a default +debug and +no_opt variant -------------------------------------------+-------------------------------- Reporter: sean.michael.farley@… | Owner: macports-tickets@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: base | Version: 2.0.4 Keywords: | Port: -------------------------------------------+-------------------------------- Comment(by sean.michael.farley@…): Replying to [comment:5 snc@…]:

Having a debug variant available would be handy for people developing directly against said library. Unfortunately, this isn't the typical user and certainly not a majority—the ability to edit the Portfile to include "-g -O0" in the configure.args should be sufficient, or at least has been for me.

It's more than just adding "-g -O0", it's also generating the .dSYM libraries (at the very least). It is much easier to tell a user "just copy and paste 'sudo port -v install metis +debug' into your terminal" than to tell them to edit a Portfile. Forget about explaining tcl syntax as well. Editing Portfiles becomes a nightmare of maintenance, especially if you're trying to share them with your co-workers. With the newest version of this patch, I've run into no problems (this includes building gcc46, gcc47 as well as other common ports that usually give me trouble). -- Ticket URL: <https://trac.macports.org/ticket/33821#comment:6> MacPorts <http://www.macports.org/> Ports system for Mac OS

#33821: base: add a default +debug and +no_opt variant -------------------------------------------+-------------------------------- Reporter: sean.michael.farley@… | Owner: macports-tickets@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: base | Version: 2.0.4 Keywords: | Port: -------------------------------------------+-------------------------------- Comment(by sean.michael.farley@…): Replying to [comment:7 snc@…]:

I don't have the experience of a situation where I'd need the user (a non-developer) to actually debug a library from MacPorts on my behalf, so I'm wondering if that's worth the burden that Ryan pointed out in the mailing list: having more variants is going to be relatively messy for MacPorts to support.

There are a few instances where this has come up very frequently for me. For one, using a gcc4X compiled binary in gdb leads to extremely messy debugging because the compiler itself needs to keep around its own .dSYM files. Further, it seems a case that is being very overlooked here is writing software dependent on libraries that a person doesn't maintain. PETSc is a good example of this (as I outlined in my previous comment). I've been using +debug for almost two years now and it has been a much more sane environment than before.

I definitely like your patches for what they are, though.

Thanks :-) thought at this point, I'd just be happy with getting the +debug variant accepted. -- Ticket URL: <https://trac.macports.org/ticket/33821#comment:8> MacPorts <http://www.macports.org/> Ports system for Mac OS

#33821: base: add a default +debug and +no_opt variant ------------------------------------+-------------------------------- Reporter: sean.michael.farley@… | Owner: macports-tickets@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: base | Version: 2.0.4 Resolution: | Keywords: Port: | ------------------------------------+-------------------------------- Comment (by cnehren+macports@…): What's holding this up? -- Ticket URL: <https://trac.macports.org/ticket/33821#comment:16> MacPorts <http://www.macports.org/> Ports system for OS X

#33821: base: add a default +debug and +no_opt variant ------------------------------------+-------------------------------- Reporter: sean.michael.farley@… | Owner: macports-tickets@… Type: enhancement | Status: closed Priority: Normal | Milestone: Component: base | Version: 2.0.4 Resolution: wontfix | Keywords: Port: | ------------------------------------+-------------------------------- Changes (by sean@…): * status: new => closed * resolution: => wontfix Comment: I'm going to close this ticket now and suggest that the macports-dev mailing list be used to talk about how this port group should be improved. -- Ticket URL: <https://trac.macports.org/ticket/33821#comment:19> MacPorts <http://www.macports.org/> Ports system for OS X