Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#44670 closed defect (fixed)

inkscape @0.48.5_1 fails to build

Reported by: hgonzale (Humberto Gonzalez) Owned by: dbevans (David B. Evans)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: captainproton1971 (Captain Proton), jxy (Xiao-Yong), jjstickel (Jonathan Stickel), gorticus (Jason Mitchell)
Port: inkscape

Description

:info:build object-snapper.cpp:564:38: error: no matching constructor for initialization of 'Inkscape::SnappedPoint'
:info:build                         SnappedPoint s(_snapmanager->getDesktop()->doc2dt(p_inters), p.getSourceType(), p.getSourceNum(), k->target_type, dist, getSnapperTolerance(), getSnapperAlwaysSnap(), true, k->target_bbox);
:info:build                                      ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
:info:build ./snapped-point.h:32:5: note: candidate constructor not viable: no known conversion from 'const Geom::OptRect' to 'const bool' for 9th argument
:info:build     SnappedPoint(Geom::Point const &p, SnapSourceType const &source, long source_num, SnapTargetType const &target, Geom::Coord const &d, Geom::Coord const &t, bool const &a, bool const &constrained_snap, bool const &fully_constrained, Geom::OptRect target_bbox = Geom::OptRect());
:info:build     ^
:info:build ./snapped-point.h:29:5: note: candidate constructor not viable: requires 0 arguments, but 9 were provided
:info:build     SnappedPoint();
:info:build     ^
:info:build ./snapped-point.h:30:5: note: candidate constructor not viable: requires single argument 'p', but 9 arguments were provided
:info:build     SnappedPoint(Geom::Point const &p);
:info:build     ^
:info:build ./snapped-point.h:31:5: note: candidate constructor not viable: requires 13 arguments, but 9 were provided
:info:build     SnappedPoint(Geom::Point const &p, SnapSourceType const &source, long source_num, SnapTargetType const &target, Geom::Coord const &d, Geom::Coord const &t, bool const &a, bool const &at_intersection, bool const &constrained_snap, bool const &fully_constrained, Geom::Coord const &d2, Geom::Coord const &t2, bool const &a2);
:info:build     ^
:info:build ./snapped-point.h:33:5: note: candidate constructor not viable: requires 7 arguments, but 9 were provided
:info:build     SnappedPoint(SnapCandidatePoint const &p, SnapTargetType const &target, Geom::Coord const &d, Geom::Coord const &t, bool const &a, bool const &constrained_snap, bool const &fully_constrained);
:info:build     ^
:info:build ./snapped-point.h:25:7: note: candidate constructor (the implicit copy constructor) not viable: requires 1 argument, but 9 were provided
:info:build class SnappedPoint
:info:build       ^

Attachments (1)

inkscape_build.log (1.3 MB) - added by hgonzale (Humberto Gonzalez) 10 years ago.

Download all attachments as: .zip

Change History (13)

Changed 10 years ago by hgonzale (Humberto Gonzalez)

Attachment: inkscape_build.log added

comment:1 Changed 10 years ago by larryv (Lawrence Velázquez)

Cc: devans@… removed
Owner: changed from macports-tickets@… to devans@…

comment:2 Changed 10 years ago by captainproton1971 (Captain Proton)

Cc: captainproton1971@… added

Cc Me!

comment:3 Changed 10 years ago by jxy (Xiao-Yong)

Cc: jinxiaoyong@… added

Cc Me!

comment:4 Changed 10 years ago by jjstickel (Jonathan Stickel)

Cc: jjstickel@… added

Cc Me!

comment:5 Changed 10 years ago by jjstickel (Jonathan Stickel)

Upstream bug reprot: https://bugs.launchpad.net/inkscape/+bug/1357411

The patchfile "patch-src-object-snapper.cpp-v3.diff" provided there in comment #7 worked for me.

comment:6 Changed 10 years ago by dbevans (David B. Evans)

Status: newassigned

Testing now. Thanks.

comment:7 Changed 10 years ago by larryv (Lawrence Velázquez)

Cc: jason-macports@… added

Has duplicate #44711 against @0.48.5_2. Reporter notes that avoiding Clang works.

comment:8 Changed 10 years ago by dbevans (David B. Evans)

Recommended upstream patch applied in r124143. This works for me on Mavericks using current Clang build. Waiting for buildbots to confirm build on other platforms.

comment:9 in reply to:  8 Changed 10 years ago by larryv (Lawrence Velázquez)

Jason’s log suggests he’s on Mountain Lion.

comment:10 Changed 10 years ago by dbevans (David B. Evans)

Yes, buildbots have built successfully on Mavericks, ML and SL so far. Lion is backed up but the upstream patch provider has reported building successfully on that platform so I don't expect any problems.

comment:11 Changed 10 years ago by dbevans (David B. Evans)

Resolution: fixed
Status: assignedclosed
Version: 2.3.1

Builds successfully on all four buildbots.

comment:12 Changed 10 years ago by gorticus (Jason Mitchell)

Builds/runs for me on 10.8 and 10.9.

Note: See TracTickets for help on using tickets.