[MacPorts] #34331: libgda3 build fails with clang
#34331: libgda3 build fails with clang -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: jwa@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.0.4 Keywords: clang | Port: libgda3 -------------------------------------+-------------------------------------- http://build.macports.org/builders/buildports-lion- x86_64/builds/548/steps/compile/logs/stdio {{{ gda-entity-field.c:112:2: error: void function 'gda_entity_field_set_dict_type' should not return a value [-Wreturn-type] }}} The software should be patched to work with clang... -- Ticket URL: <https://trac.macports.org/ticket/34331> MacPorts <http://www.macports.org/> Ports system for Mac OS
#34331: libgda3 build fails with clang ---------------------------+------------------- Reporter: ryandesign@… | Owner: jwa@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.0.4 Resolution: | Keywords: clang Port: libgda3 | ---------------------------+------------------- Comment (by devans@…): fix committed in r105103 -- Ticket URL: <https://trac.macports.org/ticket/34331#comment:1> MacPorts <http://www.macports.org/> Ports system for OS X
#34331: libgda3 build fails with clang ---------------------------+-------------------- Reporter: ryandesign@… | Owner: jwa@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.0.4 Resolution: fixed | Keywords: clang Port: libgda3 | ---------------------------+-------------------- Changes (by devans@…): * status: new => closed * resolution: => fixed -- Ticket URL: <https://trac.macports.org/ticket/34331#comment:2> MacPorts <http://www.macports.org/> Ports system for OS X
#34331: libgda3 build fails with clang ---------------------------+---------------------- Reporter: ryandesign@… | Owner: jwa@… Type: defect | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.0.4 Resolution: | Keywords: clang Port: libgda3 | ---------------------------+---------------------- Changes (by jmr@…): * status: closed => reopened * resolution: fixed => Comment: Telling the compiler to ignore the problem is not a real fix. The code needs to be fixed so the function prototype and the actual behaviour match. -- Ticket URL: <https://trac.macports.org/ticket/34331#comment:3> MacPorts <http://www.macports.org/> Ports system for OS X
#34331: libgda3 build fails with clang ---------------------------+---------------------- Reporter: ryandesign@… | Owner: jwa@… Type: defect | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.0.4 Resolution: | Keywords: clang Port: libgda3 | ---------------------------+---------------------- Comment (by egall@…): Replying to [comment:3 jmr@…]:
The code needs to be fixed so the function prototype and the actual behaviour match.
That's something upstream should do -- Ticket URL: <https://trac.macports.org/ticket/34331#comment:4> MacPorts <http://www.macports.org/> Ports system for OS X
#34331: libgda3 build fails with clang ---------------------------+---------------------- Reporter: ryandesign@… | Owner: jwa@… Type: defect | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.0.4 Resolution: | Keywords: clang Port: libgda3 | ---------------------------+---------------------- Comment (by ryandesign@…): I agree that would be ideal, however libgda 3.0.4 was released five years ago, and has been surpassed by versions 4 and 5, so I doubt upstream will be making any further changes to version 3. -- Ticket URL: <https://trac.macports.org/ticket/34331#comment:5> MacPorts <http://www.macports.org/> Ports system for OS X
#34331: libgda3 build fails with clang ---------------------------+---------------------- Reporter: ryandesign@… | Owner: jwa@… Type: defect | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.0.4 Resolution: | Keywords: clang Port: libgda3 | ---------------------------+---------------------- Comment (by jmr@…): Regardless of whose job it is to fix it, letting invalid code through is absolutely not something you want to do. What even happens when a void function returns a value? It's completely undefined. The top of the stack could be clobbered by the unexpected value, the program could crash, or anything at all could happen really. -- Ticket URL: <https://trac.macports.org/ticket/34331#comment:6> MacPorts <http://www.macports.org/> Ports system for OS X
#34331: libgda3 build fails with clang ---------------------------+---------------------- Reporter: ryandesign@… | Owner: jwa@… Type: defect | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.0.4 Resolution: | Keywords: clang Port: libgda3 | ---------------------------+---------------------- Changes (by ryandesign@…): * cc: devans@… (added) Comment: Well, presumably nothing catastrophically bad happens, since all users of this program on all platforms have been using this bad code. And I guess gcc merely issued a warning for this and allowed compilation to proceed. So changing the flags to let clang do the same might not be so terrible. Sure, if someone can figure out what the developers meant to do, and fix the code to actually do that, then that would be better. -- Ticket URL: <https://trac.macports.org/ticket/34331#comment:7> MacPorts <http://www.macports.org/> Ports system for OS X
#34331: libgda3 build fails with clang ---------------------------+---------------------- Reporter: ryandesign@… | Owner: jwa@… Type: defect | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.0.4 Resolution: | Keywords: clang Port: libgda3 | ---------------------------+---------------------- Comment (by jmr@…): Maybe nothing bad happened with gcc (or did it? How do you know the data being processed didn't come out subtly wrong?), but given that the compiler is within its rights to do literally anything when it encounters code with undefined behaviour, the same is not guaranteed to be true with clang. -- Ticket URL: <https://trac.macports.org/ticket/34331#comment:8> MacPorts <http://www.macports.org/> Ports system for OS X
#34331: libgda3 build fails with clang ---------------------------+---------------------- Reporter: ryandesign@… | Owner: jwa@… Type: defect | Status: reopened Priority: Normal | Milestone: Component: ports | Version: 2.0.4 Resolution: | Keywords: clang Port: libgda3 | ---------------------------+---------------------- Comment (by devans@…): Well putting in my two cents, I don't think its worth spending much time on this port. This version of libgda is way out of date and there are no dependents. Newer versions are in libgda4 (gtk2 compatible) and libgda5 (gtk3 compatible). I vote to nuke the port rather than mess with it. -- Ticket URL: <https://trac.macports.org/ticket/34331#comment:9> MacPorts <http://www.macports.org/> Ports system for OS X
#34331: libgda3 build fails with clang ---------------------------+-------------------- Reporter: ryandesign@… | Owner: jwa@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.0.4 Resolution: fixed | Keywords: clang Port: libgda3 | ---------------------------+-------------------- Changes (by jeremyhu@…): * status: reopened => closed * resolution: => fixed Comment: Done. r109448 -- Ticket URL: <https://trac.macports.org/ticket/34331#comment:10> MacPorts <http://www.macports.org/> Ports system for OS X
participants (1)
-
MacPorts