Opened 12 years ago

Closed 12 years ago

#32924 closed enhancement (fixed)

poppler: Add library dependencies

Reported by: someuser12 Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc: dbevans (David B. Evans)
Port: poppler

Description

Poppler and libpoppler actually depend on several dependencies that were not mentioned in the Portfile (can be checked e.g. with otool -L /opt/local/lib/libpoppler.19.dylib). I attach a new version of the Portfile that adds explicit dependencies to tiff, libpng, bzip2, zlib, fontconfig, freetype, expat. This is probably not exhaustive, but, hopefully, better than nothing.

Incidentally, the new Portfile updates to version 0.18.3 that was released on January, 15th, 2012.

The use of such explicit dependencies can be to find which ports one has to rebuild after an upgrade. For instance, after an upgrade of tiff, port dependents tiff would show the extensive list of ports that depend on tiff and that one should rebuild. Presently, poppler is not listed, so that the user discovers that he/she should have rebuilt poppler only at the time he/she fails to launch e.g. pdflatex.

Thank you for your consideration!

Attachments (3)

patch-Portfile.diff (938 bytes) - added by mf2k (Frank Schima) 12 years ago.
Portfile (2.4 KB) - added by someuser12 12 years ago.
patch-Portfile (349 bytes) - added by someuser12 12 years ago.
patch to Portfile 0.18.2

Download all attachments as: .zip

Change History (9)

comment:1 Changed 12 years ago by mf2k (Frank Schima)

Owner: changed from macports-tickets@… to devans@…
Version: 2.0.3

Please attach a universal diff of the Portfile so we can see the changes you made.

comment:2 Changed 12 years ago by someuser12

Sorry. Here it is.

Changed 12 years ago by mf2k (Frank Schima)

Attachment: patch-Portfile.diff added

comment:3 Changed 12 years ago by mf2k (Frank Schima)

Thanks. I meant unified diff of course. Also if you append ".diff" to the name, as I have just done, then Trac even colorizes it.

Changed 12 years ago by someuser12

Attachment: Portfile added

Changed 12 years ago by someuser12

Attachment: patch-Portfile added

patch to Portfile 0.18.2

comment:4 in reply to:  3 Changed 12 years ago by someuser12

Meanwhile, the poppler port has been updated to version 0.18.4. I updated my Portfile proposal and patch-Portfile accordingly (but not patch-Portfile.diff since I dont't own it).

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

Cc: devans@… added
Keywords: haspatch added
Owner: changed from devans@… to ryandesign@…
Summary: Add library dependencies to poppler + version bumppoppler: Add library dependencies

The fontconfig dependency was already added in r90941.

In #32793 the gtk2 dependency was deleted, which also had the effect of deleting the indirect glib2 dependency, which is needed to build the poppler-glib library, so this dependency has to be added again.

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

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.