Opened 20 months ago

Closed 19 months ago

Last modified 19 months ago

#65585 closed defect (fixed)

libgit2 @1.5.0: builds failing for 10.7: parameter '...' not found in function declaration

Reported by: mascguy (Christopher Nielsen) Owned by: mascguy (Christopher Nielsen)
Priority: Normal Milestone:
Component: ports Version: 2.7.2
Keywords: lion pending Cc:
Port: libgit2

Description (last modified by mascguy (Christopher Nielsen))

The following is seen both on our 10.7 buildbot, as well as locally:

In file included from libgit2/work/libgit2-1.5.0/tests/headertest/headertest.c:8:
In file included from libgit2/work/libgit2-1.5.0/include/git2.h:11:
In file included from libgit2/work/libgit2-1.5.0/include/git2/annotated_commit.h:10:
libgit2/work/libgit2-1.5.0/include/git2/common.h:477:11: error: parameter '...' not found in the function declaration [-Werror,-Wdocumentation]

Blacklisting older Clangs takes care of it. But perhaps it would be preferable to make this a warning, rather than an error...?

Attachments (1)

libgit2-build-10.7-error.log (503.3 KB) - added by mascguy (Christopher Nielsen) 20 months ago.

Download all attachments as: .zip

Change History (9)

Changed 20 months ago by mascguy (Christopher Nielsen)

comment:1 Changed 20 months ago by mascguy (Christopher Nielsen)

Description: modified (diff)

comment:2 Changed 20 months ago by jmroot (Joshua Root)

Yes, turn off use of -Werror. Changing compilers over a -Wdocumentation warning would be silly.

comment:3 Changed 19 months ago by mascguy (Christopher Nielsen)

Owner: changed from dbevans to mascguy

comment:4 Changed 19 months ago by mascguy (Christopher Nielsen)

Among other ports, this is blocking webkit2-gtk. So I'll fix.

comment:5 Changed 19 months ago by mascguy (Christopher Nielsen)

Keywords: pending added

comment:6 Changed 19 months ago by Christopher Nielsen <mascguy@…>

Resolution: fixed
Status: assignedclosed

In b68f65f8e59f43768a40d05e6464a18048b66700/macports-ports (master):

libgit2: fix compilation errors for Wdocumentation
Fixes: #65585

comment:7 Changed 19 months ago by Christopher Nielsen <mascguy@…>

comment:8 Changed 19 months ago by mascguy (Christopher Nielsen)

Rebuilds have been queued for 10.7, covering the vast majority of ports blocked by this over the past four weeks. That includes both direct dependents, as well as indirect dependents based on rust/cargo.

Note: See TracTickets for help on using tickets.