[MacPorts] #44323: gnutls @3.3.5: support c89 (error: comma at end of enumerator list)
#44323: gnutls @3.3.5: support c89 (error: comma at end of enumerator list) --------------------------+---------------------------- Reporter: ryandesign@… | Owner: mschamschula@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Keywords: | Port: gnutls --------------------------+---------------------------- Compiling gloox on Snow Leopard with its gcc-4.2 fails: {{{ /opt/local/include/gnutls/gnutls.h:172: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:186: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:244: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:295: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:394: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:434: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:485: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:599: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:618: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:668: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:688: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:722: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:1598: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:1937: error: comma at end of enumerator list /opt/local/include/gnutls/gnutls.h:2085: error: comma at end of enumerator list }}} It seems that having a comma at the end of an enumerator list is legal in c99 (which is the default for clang) but not with c89 (which is the default for gcc-4.2). So one fix would be to use c99. However, I think it's unreasonable to force all ports that want to use gnutls to switch to c99. Therefore I think gnutls should be fixed to support c89 again, as gnutls 3.1.x did before. I'll work on a patch for this, and an upstream bug report for the developers. -- Ticket URL: <https://trac.macports.org/ticket/44323> MacPorts <http://www.macports.org/> Ports system for OS X
#44323: gnutls @3.3.5: support c89 (error: comma at end of enumerator list) ---------------------------+---------------------------- Reporter: ryandesign@… | Owner: mschamschula@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: | Keywords: Port: gnutls | ---------------------------+---------------------------- Comment (by ryandesign@…): Well, the attached patch is all that we ''should'' need to fix this. However the port fails to build now. Investigating... -- Ticket URL: <https://trac.macports.org/ticket/44323#comment:1> MacPorts <http://www.macports.org/> Ports system for OS X
#44323: gnutls @3.3.5: support c89 (error: comma at end of enumerator list) ---------------------------+---------------------------- Reporter: ryandesign@… | Owner: mschamschula@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: | Keywords: Port: gnutls | ---------------------------+---------------------------- Comment (by ryandesign@…): It seems that patching gnutls.h.in causes the build system to think it should regenerate the documentation. And regenerating the documentation fails on OS X because its /usr/bin/sed is BSD sed. Although the configure script correctly finds /opt/local/bin/gsed, the documentation Makefile does not use the `$SED` variable and instead just uses the program name `sed`. Since we've not made any changes to the file that would warrant regeneration of the documentation, and since regeneration of the documentation makes the build take twice as long, I'll try to suppress regenerating the documentation though fiddling with the timestamps. -- Ticket URL: <https://trac.macports.org/ticket/44323#comment:2> MacPorts <http://www.macports.org/> Ports system for OS X
#44323: gnutls @3.3.5: support c89 (error: comma at end of enumerator list) ---------------------------+---------------------------- Reporter: ryandesign@… | Owner: mschamschula@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: | Keywords: Port: gnutls | ---------------------------+---------------------------- Comment (by ryandesign@…): Replying to [comment:2 ryandesign@…]:
And regenerating the documentation fails on OS X because its /usr/bin/sed is BSD sed.
(For the record, it fails with BSD sed because they are using GNU-specific sed extensions, specifically that the argument to `-i` is optional, which it isn't under BSD sed.) -- Ticket URL: <https://trac.macports.org/ticket/44323#comment:3> MacPorts <http://www.macports.org/> Ports system for OS X
#44323: gnutls @3.3.5: support c89 (error: comma at end of enumerator list) ---------------------------+---------------------------- Reporter: ryandesign@… | Owner: mschamschula@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: | Keywords: Port: gnutls | ---------------------------+---------------------------- Comment (by ryandesign@…): Replying to [comment:2 ryandesign@…]:
I'll try to suppress regenerating the documentation though fiddling with the timestamps.
That works, but it turns out additional headers have the same issue. I'll try to find them all. -- Ticket URL: <https://trac.macports.org/ticket/44323#comment:4> MacPorts <http://www.macports.org/> Ports system for OS X
#44323: gnutls @3.3.5: support c89 (error: comma at end of enumerator list) ---------------------------+---------------------------- Reporter: ryandesign@… | Owner: mschamschula@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: | Keywords: haspatch Port: gnutls | ---------------------------+---------------------------- Changes (by ryandesign@…): * keywords: => haspatch Comment: This should now be all the needed patches. I still can't build gloox with apple-gcc-4.2, but the reason is now different, so I feel we can commit these gnutls fixes now. -- Ticket URL: <https://trac.macports.org/ticket/44323#comment:6> MacPorts <http://www.macports.org/> Ports system for OS X
#44323: gnutls @3.3.5: support c89 (error: comma at end of enumerator list) ---------------------------+---------------------------- Reporter: ryandesign@… | Owner: mschamschula@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: | Keywords: haspatch Port: gnutls | ---------------------------+---------------------------- Comment (by ryandesign@…): Replying to [ticket:44323 ryandesign@…]:
I'll work on a patch for this, and an upstream bug report for the developers.
Upstream bug report filed: https://savannah.gnu.org/support/index.php?108614 -- Ticket URL: <https://trac.macports.org/ticket/44323#comment:7> MacPorts <http://www.macports.org/> Ports system for OS X
#44323: gnutls @3.3.5: support c89 (error: comma at end of enumerator list) ---------------------------+---------------------------- Reporter: ryandesign@… | Owner: mschamschula@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: | Keywords: haspatch Port: gnutls | ---------------------------+---------------------------- Comment (by ryandesign@…): Replying to [comment:3 ryandesign@…]:
(For the record, it fails with BSD sed because they are using GNU- specific sed extensions, specifically that the argument to `-i` is optional, which it isn't under BSD sed.)
Upstream bug report filed: https://savannah.gnu.org/support/index.php?108615 -- Ticket URL: <https://trac.macports.org/ticket/44323#comment:8> MacPorts <http://www.macports.org/> Ports system for OS X
#44323: gnutls @3.3.5: support c89 (error: comma at end of enumerator list) ---------------------------+---------------------------- Reporter: ryandesign@… | Owner: mschamschula@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.3.1 Resolution: fixed | Keywords: haspatch Port: gnutls | ---------------------------+---------------------------- Changes (by ryandesign@…): * status: new => closed * resolution: => fixed Comment: Replying to [comment:6 ryandesign@…]:
This should now be all the needed patches. I still can't build gloox with apple-gcc-4.2, but the reason is now different, so I feel we can commit these gnutls fixes now.
r122082 -- Ticket URL: <https://trac.macports.org/ticket/44323#comment:9> MacPorts <http://www.macports.org/> Ports system for OS X
participants (1)
-
MacPorts