Opened 5 months ago

Closed 5 months ago

#20521 closed enhancement (fixed)

poppler-24.11.0

Reported by: Bruce Dubbs Owned by: Bruce Dubbs
Priority: normal Milestone: 12.3
Component: BOOK Version: git
Severity: normal Keywords:
Cc:

Description

New monthly version.

Change History (19)

comment:1 by Bruce Dubbs, 5 months ago

Owner: changed from blfs-book to Bruce Dubbs
Status: newassigned

comment:2 by martyj19, 5 months ago

Nontrivial API breaks affecting at least libreoffice.

in reply to:  2 comment:3 by Bruce Dubbs, 5 months ago

Replying to martyj19:

Nontrivial API breaks affecting at least libreoffice.

Libreoffice using private headers again?

comment:4 by martyj19, 5 months ago

Type change from GfxDevice{Gray,RGB}ColorSpace* to std::unique_ptr<GfxColorSpace>&&.

I have Scribus, not in the book, failing on redefinition of class LinkSubmitForm.

Last edited 5 months ago by martyj19 (previous) (diff)

comment:5 by Bruce Dubbs, 5 months ago

Release 24.10.0:

core:

  • Add support for SubmitForms Links
  • Internal code improvements
  • Fix crash in malformed documents

qt6:

  • Add support for SubmitForms Links
  • Add API for creating signature field without signing immediately

glib:

  • add auto_cleanup functions so that consumers can use g_auto*
  • fix closure GI annotation

qt5:

  • Add support for SubmitForms Links

build system:

  • Use the boosts own cmake file as suggested by cmake itself

comment:6 by Bruce Dubbs, 5 months ago

Resolution: fixed
Status: assignedclosed

Fixed at commits

3ff7b02eee Update to libxmlb-0.3.21.
a30fe894ac Update to poppler-24.10.0.
0225d7c35b Update to c-ares-1.34.2.
fa2565d031 Update to xf86-input-evdev-2.11.0 (Xorg Driver).
421729eccd Update to xf86-input-libinput-1.5.0 (Xorg Driver).

comment:7 by Douglas R. Reno, 5 months ago

It looks like Inkscape breaks as well (see https://gitlab.com/inkscape/inkscape/-/issues/5301 )

I wonder about all of the other users of poppler now

comment:8 by Xi Ruoyao, 5 months ago

I think many packages just use the private headers despite poppler developers tell not to.

comment:9 by Douglas R. Reno, 5 months ago

I would like to propose reverting this package back to 24.09.0 temporarily. My reasoning is that it makes it difficult to put in Libreoffice if this package is busted, we could force Libreoffice to use it's bundled copy of Poppler, but it is a little old. Alternatively, we could keep with this and file a bug report upstream at Libreoffice so that they're aware of the issue. I note that at least Inkscape is also broken, and I don't have poppler-24.10.0 installed to test anything else yet.

If we decide to keep it in the book, then I will try to make Libreoffice work with it's bundled copy of poppler, but I probably won't upgrade my systems until Inkscape is fixed.

comment:10 by Bruce Dubbs, 5 months ago

The only thing that was changed for poppler-24.10.0 was the version and the stats:

   <!ENTITY poppler-download-ftp  " ">
-  <!ENTITY poppler-md5sum        "c4a92a7ee9cd8137c50273bcc5252646">
+  <!ENTITY poppler-md5sum        "df69253d6fd4ad7c892d1d48039ecfe7">
   <!ENTITY poppler-size          "1.8 MB">
   <!ENTITY poppler-buildsize     "76 MB (with Qt6 library; add 13 MB for tests)">
-  <!ENTITY poppler-time          "0.9 SBU (with parallelism=4, tests, and Qt6 library)">
+  <!ENTITY poppler-time          "1.0 SBU (with parallelism=4, tests, and Qt6 library)"

For testing LO the development book does not yet need to be reverted, but we can do that at any point if necessary. Also we will probably get verison 24.11.0 in 12 days or so, but if the problem is packages using private headers then that probably will not make a difference.

comment:11 by Bruce Dubbs, 5 months ago

Hmm, looking at the differences between the 24.09.0 and 24.10.0 versions, I found a difference in poppler-link-private.h:

@@ -24,6 +24,7 @@
 
 #include <vector>
 
+#include "poppler-link.h"
 #include "Link.h"
 
 class LinkOCGState;
@@ -78,6 +79,17 @@
     bool m_exclude;
 };
 
+class LinkSubmitFormPrivate : public LinkPrivate
+{
+public:
+    LinkSubmitFormPrivate(const QRectF &area, const QVector<int> &fieldIds, const QString &url, LinkSubmitForm::SubmitFormFlags flags) : LinkPrivate(area), m_fieldIds(fieldIds), m_url(url), m_flags(flags) { }
+    ~LinkSubmitFormPrivate() override;
+
+    QVector<int> m_fieldIds;
+    QString m_url;
+    LinkSubmitForm::SubmitFormFlags m_flags;
+};
+
 }

Could that be the problem?

comment:12 by Douglas R. Reno, 5 months ago

comment:13 by Bruce Dubbs, 5 months ago

Resolution: fixed
Status: closedreopened

We reverted to version 24.09.0 due to incompatibilities with other programs. We will restore it (or version 24.11.0) when those get fixes for more recent versions of poppler.

comment:14 by Bruce Dubbs, 5 months ago

Summary: poppler-24.10.0poppler-24.10.0 (Wait for version 24.11.0)

comment:15 by Bruce Dubbs, 5 months ago

Owner: changed from Bruce Dubbs to blfs-book
Status: reopenednew

comment:16 by Bruce Dubbs, 5 months ago

Owner: changed from blfs-book to Bruce Dubbs
Status: newassigned

comment:17 by Bruce Dubbs, 5 months ago

Summary: poppler-24.10.0 (Wait for version 24.11.0)poppler-24.11.0

comment:18 by Bruce Dubbs, 5 months ago

Release 24.11.0:

core:

  • Fix rendering of some buttons
  • Fix selecting fonts in TTC font collections
  • CairoOutputDev: prevent infinite loop in functionShadedFill()
  • Fix some unstable headers not being usable on their own
  • Fix some headers not being usable on their own

glib:

  • Fix some headers not being usable on their own
  • add INSTALL_GLIB_DEMO option to install poppler-glib-demo
  • Fix Launch action with unicode encoded filenames

qt6:

  • Fix memory handling issue on signature creation
  • Fix Launch action with unicode encoded filenames

qt5:

  • Fix Launch action with unicode encoded filenames

build system:

  • Remove duplicate definitions
  • cmake: Set CMAKE_C_STANDARD to 17

comment:19 by Bruce Dubbs, 5 months ago

Resolution: fixed
Status: assignedclosed

Fixed at revision cf4a342c11.

However poppler still has to be disabled for inkscape. There is a new error different from the poppler-24.10 problem. Inkscape upstream will have to address this to incorporate the latest poppler.

Note: See TracTickets for help on using tickets.