Bug 45204

Summary: Add Vala bindings
Product: poppler Reporter: Evan Nemerson <evan>
Component: glib frontendAssignee: poppler-bugs <poppler-bugs>
Status: RESOLVED MOVED QA Contact:
Severity: normal    
Priority: medium CC: robert.ancell
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments: glib: add Vala bindings
glib: add Vala bindings

Description Evan Nemerson 2012-01-24 21:59:01 UTC
Title pretty much says it all. They're pretty easy to generate from the GIR, so it would be nice to have them distributed with Poppler.
Comment 1 Evan Nemerson 2012-01-24 21:59:47 UTC
Created attachment 56114 [details] [review]
glib: add Vala bindings
Comment 2 Evan Nemerson 2012-01-29 23:29:44 UTC
Created attachment 56311 [details] [review]
glib: add Vala bindings

Slightly updated version which corrects some field ownership issues.
Comment 3 Carlos Garcia Campos 2012-01-30 05:56:33 UTC
Is this actually needed? We don't provide any bindings in poppler source code. Isn't it enough with the gobject-introspections stuff?
Comment 4 Robert Ancell 2015-08-09 01:54:41 UTC
The Vala bindinds require additional information than just the GIR file and it's better for them to be distributed with poppler so they match.
Comment 5 Albert Astals Cid 2015-08-09 13:54:54 UTC
You seem to be contradicting the original reporter, can you guys agree on wheter gir is enough or not?
Comment 6 Evan Nemerson 2015-08-09 17:32:55 UTC
(In reply to Albert Astals Cid from comment #5)
> You seem to be contradicting the original reporter, can you guys agree on
> wheter gir is enough or not?

He isn't contradicting me at all.  I only said it is *pretty easy* to generate them from the GIR, not that the GIR alone is enough.  It requires a metadata fileā€”the current version is at https://git.gnome.org/browse/vala/tree/vapi/metadata/Poppler-0.18.metadata

We can keep distributing them with Vala, but there are a few problems; for one thing the Vala bindings will not necessarily match the version of poppler which is installed.  See the beginning of https://wiki.gnome.org/Projects/Vala/UpstreamGuide for several other reasons.
Comment 7 Al Thomas 2018-02-12 15:46:46 UTC
This would make it easier to keep API changes in sync. For example Debian had a recent request to package 0.54 and noted "the Vala bindings could be updated, and
then pdf-presenter-console could take advantage of the new
API":
http://linux.debian.bugs.dist.narkive.com/e4ouwHWb/bug-860955-poppler-please-package-new-upstream-version-0-54-0-as-soon-as-possible

The current patch would need to be updated for CMake to check for vapigen.
Comment 8 GitLab Migration User 2018-08-21 10:42:53 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/poppler/poppler/issues/341.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.