[MacPorts] #50476: Portfile for GNOME Internet Radio Locator (GIRL) version 8.0.1

[MacPorts]

#50476: Portfile for GNOME Internet Radio Locator (GIRL) version 8.0.1
-------------------------+------------------------
  Reporter:  ole@…       |      Owner:  devans@…
      Type:  submission  |     Status:  assigned
  Priority:  Normal      |  Milestone:
 Component:  ports       |    Version:
Resolution:              |   Keywords:  maintainer
      Port:  girl        |
-------------------------+------------------------

Comment (by devans@…):

 Given the problem with recording, I suggest either disabling it entirely
 until it can be fixed or disable it and add a variant to enable it for
 testing.

 Have updated my proposed Portfile patch to demonstrate the second option
 by adding a variant +record.  In addition, have added a +debug variant
 that enables building with debug symbols (for gdb) and your debug
 messages.

 Have done that, running girl +debug resulted in a segfault after a few
 initial messages.

 Here is the output of a run under gdb:

 {{{
 $ gdb girl
 (gdb) run

 Reading symbols for shared libraries . done
 ** Message: girl->selected_listener_uri:

 ** Message: girl->selected_listener_name:

 ** Message: girl->selected_listener_location:

 ** Message: girl->selected_listener_band:

 ** Message: girl->selected_listener_description:

 ** Message: girl->selected_station_uri:

 ** Message: girl->selected_station_name:

 ** Message: girl->selected_station_location:

 ** Message: girl->selected_station_band:

 ** Message: girl->selected_station_description:

 ** Message: girl->selected_streams_uri:

 ** Message: girl->selected_streams_mime:

 ** Message: girl->selected_streams_codec:

 ** Message: girl->selected_streams_bitrate:

 ** Message: girl->selected_streams_samplerate:


 Program received signal EXC_BAD_ACCESS, Could not access memory.
 Reason: KERN_INVALID_ADDRESS at address: 0x000000000183be30
 0x00007fff83e6ef92 in strlen ()
 (gdb) bt
 #0  0x00007fff83e6ef92 in strlen ()
 #1  0x00007fff83eb3d5a in __vfprintf ()
 #2  0x00007fff83edc807 in __v2printf ()
 #3  0x00007fff83eb258e in _vasprintf ()
 #4  0x0000000100fac92b in g_vasprintf ()
 #5  0x0000000100f8b640 in g_strdup_vprintf ()
 #6  0x0000000100f78178 in g_logv ()
 #7  0x0000000100f78122 in g_log ()
 #8  0x000000010000ec2f in create_girl_app () at girl-gui.c:1064
 #9  0x0000000100013299 in main (argc=1, argv=0x7fff5fbffa80) at girl.c:79
 (gdb) list girl-gui.c:1064
 1059                   girl->selected_streams_codec);
 1060            GIRL_DEBUG_MSG("girl->selected_streams_bitrate: %s\n",
 1061                   girl->selected_streams_bitrate);
 1062            GIRL_DEBUG_MSG("girl->selected_streams_samplerate: %s\n",
 1063                   girl->selected_streams_samplerate);
 1064            GIRL_DEBUG_MSG("girl->selected_channels: %s\n",
 1065                   girl->selected_streams_channels);
 1066
 1067            gnome_config_pop_prefix();
 1068    #endif
 (gdb) print girl->selected_streams_channels
 $1 = 25411120
 }}}

 Problem is that girl->selected_streams_channels is not a string but an
 enum (which appears not to be initialized at this first pass). Changing %s
 to %0x in the associated format string fixes the issue.  Have included a
 patch to fix this as part of the updated Portfile patch.

-- 
Ticket URL: <https://trac.macports.org/ticket/50476#comment:7>
MacPorts <https://www.macports.org/>
Ports system for OS X