Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#28247 closed defect (fixed)

inkscape @0.48.0, Revision 3 +enable_debugging - fails to build

Reported by: a.y.harano@… Owned by: dbevans (David B. Evans)
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: Cc: SlaunchaMan (Jeff Kelley), mklein-de (Michael Klein), captainproton1971 (Captain Proton), jeremyhu (Jeremy Huddleston Sequoia), mail@…, turkowski, carsomyr@…, ryandesign (Ryan Carsten Schmidt), markus.falb@…
Port: inkscape

Description

:info:build extension/internal/pdfinput/pdf-parser.cpp: In member function 'void PdfParser::fillPatch(GfxPatch*, int, int)':
:info:build extension/internal/pdfinput/pdf-parser.cpp:1895: error: no matching function for call to 'GfxState::setFillColor(GfxPatch::ColorValue*)'
:info:build /opt/local/include/poppler/GfxState.h:1334: note: candidates are: void GfxState::setFillColor(GfxColor*)
:info:build make[2]: *** [extension/internal/pdfinput/pdf-parser.o] Error 1

...

:info:build Warning: the following items did not execute (for inkscape): org.macports.activate org.macports.build org.macports.destroot org.macports.install
:notice:build Log for inkscape is at: /opt/local/var/macports/logs/_opt_local_var_macports_sources_rsync.macports.org_release_ports_graphics_inkscape/main.log

Attachments (2)

patch.tgz (1004 bytes) - added by carsomyr@… 13 years ago.
The port patch archive.
gen_macports_patches.sh (3.6 KB) - added by carsomyr@… 13 years ago.
The patch archive generation script.

Download all attachments as: .zip

Change History (19)

comment:1 Changed 13 years ago by carsomyr@…

I recently encountered this problem, and an upgrade to 0.48.1, which uses a newer version of poppler, fixes the problem. Please find the port patch archive attached.

Changed 13 years ago by carsomyr@…

Attachment: patch.tgz added

The port patch archive.

comment:2 in reply to:  1 ; Changed 13 years ago by a.y.harano@…

Replying to carsomyr@…:

I recently encountered this problem, and an upgrade to 0.48.1, which uses a newer version of poppler, fixes the problem. Please find the port patch archive attached.

I really don't know how to patch into port's files. I had being trying to do as subsection "4.5.3. Manually Applying Patches" of MacPort's guide says, but no lucky.

comment:3 in reply to:  2 ; Changed 13 years ago by a.y.harano@…

Replying to a.y.harano@…:

Replying to carsomyr@…:

I recently encountered this problem, and an upgrade to 0.48.1, which uses a newer version of poppler, fixes the problem. Please find the port patch archive attached.

I really don't know how to patch into port's files. I had being trying to do as subsection "4.5.3. Manually Applying Patches" of MacPort's guide says, but no lucky.

OK. I got it. Just change patch's revision from 2 to 3. Thank you very much!

comment:4 in reply to:  3 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

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

Replying to a.y.harano@…:

Just change patch's revision from 2 to 3.

Ryan Stonecipher increased the portfile's revision to 3 in r75626 to rebuild with the latest poppler.

Meanwhile, the patch attached to this ticket would update the port to version 0.48.1, which should presumably still be done by the maintainer.

comment:5 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: SlaunchaMan@… added

Has duplicate #28266.

comment:6 Changed 13 years ago by mklein-de (Michael Klein)

Cc: michael.klein@… added

Cc Me!

comment:7 Changed 13 years ago by captainproton1971 (Captain Proton)

Cc: captainproton1971@… added

Cc Me!

comment:8 Changed 13 years ago by jeremyhu (Jeremy Huddleston Sequoia)

Please attach patches as text/plain rather than a tarball...

comment:9 Changed 13 years ago by jeremyhu (Jeremy Huddleston Sequoia)

Cc: jeremyhu@… added

Cc Me!

comment:10 Changed 13 years ago by mail@…

Cc: mail@… added

Cc Me!

comment:11 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: turk@… added

Has duplicate #28301.

comment:12 in reply to:  8 ; Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to jeremyhu@…:

Please attach patches as text/plain rather than a tarball...

Yes, I am very curious to know what program produced this tarball. It contains a readme beginning "This is an autogenerated MacPorts patch archive" but I've never seen this before, and certainly it's not something I recall ever having been advocated...

comment:13 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: carsomyr@… ryandesign@… added

Cc'ing carsomyr to whom the above comment was directed.

comment:14 in reply to:  12 Changed 13 years ago by carsomyr@…

Replying to ryandesign@…:

Replying to jeremyhu@…:

Please attach patches as text/plain rather than a tarball...

Yes, I am very curious to know what program produced this tarball. It contains a readme beginning "This is an autogenerated MacPorts patch archive" but I've never seen this before, and certainly it's not something I recall ever having been advocated...

Sorry, that's just my quick and dirty bash script for comparing two directories and spitting out a patch archive. I've attached it as gen_macports_patches.sh. Sometime ago I got tired of formatting patches, and so I made the script.

Changed 13 years ago by carsomyr@…

Attachment: gen_macports_patches.sh added

The patch archive generation script.

comment:15 Changed 13 years ago by markus.falb@…

Cc: markus.falb@… added

Cc Me!

comment:16 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: newclosed

Interesting. Now we know!

I updated inkscape to 0.48.1 in r75746, since Dave hasn't shown up. Thanks for the update.

comment:17 Changed 13 years ago by turkowski

Successfully installed:

sudo port install inkscape @0.48.1
Note: See TracTickets for help on using tickets.