ports/182906: audio/taglib .h files not c++ pedantic clean, breaks audio/soundkonverter (mp4coverart.h:49: error: comma at end of enumerator list)

[Mark Martinec]

>Number:         182906
>Category:       ports
>Synopsis:       audio/taglib .h files not c++ pedantic clean, breaks audio/soundkonverter (mp4coverart.h:49: error: comma at end of enumerator list)
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Fri Oct 11 15:30:00 UTC 2013
>Closed-Date:
>Last-Modified:
>Originator:     Mark Martinec
>Release:        9.2-STABLE
>Organization:
J. Stefan Institute
>Environment:
FreeBSD neli.ijs.si 9.2-STABLE FreeBSD 9.2-STABLE #0 r255973: Tue Oct  1 19:43:49 CEST 2013     mark at neli.ijs.si:/usr/obj/usr/src/sys/NELI  amd64
>Description:
Building of port audio/soundkonverter fails because it tries to include a
file /usr/local/include/taglib/mp4coverart.h (installed by port audio/taglib)
using a c++ compiler with a -pedantic option enabled:

In file included from /usr/local/include/taglib/mp4item.h:31,
                 from /usr/local/include/taglib/mp4tag.h:37,
                 from /wrkdirs/usr/ports/audio/soundkonverter/work/soundkonverter-2.0.4/metadata/tagengine.cpp:32:
/usr/local/include/taglib/mp4coverart.h:49: error: comma at end of enumerator list
*** [CMakeFiles/soundkonverter.dir/metadata/tagengine.o] Error code 1


Sure enough, there is a trailing comma there
in /usr/local/include/taglib/mp4coverart.h near line 49:

      enum Format {
        JPEG    = TypeJPEG,
        PNG     = TypePNG,
        BMP     = TypeBMP,
        GIF     = TypeGIF,
        Unknown = TypeImplicit,
      };


Using system default compiler: c++ (GCC) 4.2.1 20070831 patched [FreeBSD].
Fresh versions from ports: audio/soundkonverter 2.0.4, audio/taglib 1.9.1.
Full build log at: http://www.ijs.si/usr/mark/tmp/soundkonverter-2.0.4.log

The 'comma at end of enumerator' is not valid in C++ (nor in C89),
although it is valid in C99. Apparently it becomes valid in C++11,
but we are not there yet.
>How-To-Repeat:
Build audio/soundkonverter from ports.
>Fix:
Depending on a point of view, the error is either in the exported .h file(s)
by a port audio/taglib, or in using a -pedantic option in audio/soundkonverter
which tries to include plain C include files from a C++ code.

Perhaps the least effort solution would be to drop -pedantic from
C++ options in port audio/soundkonverter .

A cleaner solution would be to drop trailing commas in .h files
as installed by port audio/taglib, in particular the mp4coverart.h file.

>Release-Note:
>Audit-Trail:
>Unformatted: