unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#53536] [PATCH 0/1] Add poppler-with-data.
@ 2022-01-25 23:59 Taiju HIGASHI
  2022-01-26  0:20 ` [bug#53536] [PATCH 1/1] gnu: " Taiju HIGASHI
                   ` (5 more replies)
  0 siblings, 6 replies; 20+ messages in thread
From: Taiju HIGASHI @ 2022-01-25 23:59 UTC (permalink / raw)
  To: 53536; +Cc: Taiju HIGASHI

Hi,

I would like to view PDF files in Japanese with a viewer such as Evince, but
it seems that it cannot render Japanese at present, so I wrote a patch.

As far as I know, in order for Poppler to render CJK text, one of the
following conditions must be met

1. Install poppler with poppler-data preinstalled.
2. Install poppler, then poppler-data in the path expected by
poppler. (`POPPLER_INSTALL_PREFIX/share/poppler`)

ref:
https://github.com/freedesktop/poppler/blob/277f5de9684b3392f0d585bd36ad1a5e9e9e9ed7/CMakeLists.txt#L348-L362

Guix provides poppler and poppler-data as standalone packages, but installing
both will not satisfy either of the above prerequisites.

So I defined poppler with poppler-data as a package with the name
popper-with-data.

This package is intended to be used in the installation of packages that also
have poppler as a dependency, as shown below.

   guix package -i evince --with-input=poppler=poppler-with-data

However, As a user, this is still a bit of a hassle, so if you have a better
idea, I'd like to see it.

Incidentally, it seems that Nix had defined its own environment variable
(PLOPPER_DATADIR) to deal with this problem.

The usage of this variable seems to have been changed in the next commit, but
I personally thought it was a generic and useful way to deal with the problem.

ref:
https://github.com/NixOS/nixpkgs/pull/17819/commits/1bde33074efa11fa2edcf71032d2e634f852f349

If it is allowed to integrate poppler and poppler-data, that would be the
simplest solution.

Thank you.

Taiju HIGASHI (1):
  gnu: Add poppler-with-data.

 gnu/packages/pdf.scm | 11 +++++++++++
 1 file changed, 11 insertions(+)

--
2.34.0




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH 1/1] gnu: Add poppler-with-data.
  2022-01-25 23:59 [bug#53536] [PATCH 0/1] Add poppler-with-data Taiju HIGASHI
@ 2022-01-26  0:20 ` Taiju HIGASHI
  2022-01-26  7:37 ` [bug#53536] [PATCH 0/1] " Liliana Marie Prikler
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 20+ messages in thread
From: Taiju HIGASHI @ 2022-01-26  0:20 UTC (permalink / raw)
  To: 53536; +Cc: Taiju HIGASHI

---
 gnu/packages/pdf.scm | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm
index d4e97e1d82..6b4e629438 100644
--- a/gnu/packages/pdf.scm
+++ b/gnu/packages/pdf.scm
@@ -22,6 +22,7 @@
 ;;; Copyright © 2020 Timotej Lazar <timotej.lazar@araneo.si>
 ;;; Copyright © 2020 Maxim Cournoyer <maxim.cournoyer@gmail.com>
 ;;; Copyright © 2021 Maxime Devos <maximedevos@telenet.be>
+;;; Copyright © 2022 Taiju HIGASHI <higashi@taiju.info>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -308,6 +309,16 @@ (define-public poppler-data
     (license (list license:bsd-3
                    license:gpl2))))
 
+(define-public poppler-with-data
+  (package/inherit poppler
+    (name "poppler-with-data")
+    (inputs `(("poppler-data" ,poppler-data)
+             ,@(package-inputs poppler)))
+    (synopsis "Poppler PDF rendering library with poppler-data.")
+    (description
+     "Poppler is a PDF rendering library based on the xpdf-3.0 code base.
+This package includes Poppler encoding files for rendering of CJK and Cyrillic text")))
+
 (define-public poppler-qt5
   (package/inherit poppler
    (name "poppler-qt5")
-- 
2.34.0





^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH 0/1] Add poppler-with-data.
  2022-01-25 23:59 [bug#53536] [PATCH 0/1] Add poppler-with-data Taiju HIGASHI
  2022-01-26  0:20 ` [bug#53536] [PATCH 1/1] gnu: " Taiju HIGASHI
@ 2022-01-26  7:37 ` Liliana Marie Prikler
  2022-01-26 13:38   ` Taiju HIGASHI
  2022-01-26 13:42   ` Taiju HIGASHI
  2022-02-02  3:48 ` [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler Taiju HIGASHI
                   ` (3 subsequent siblings)
  5 siblings, 2 replies; 20+ messages in thread
From: Liliana Marie Prikler @ 2022-01-26  7:37 UTC (permalink / raw)
  To: Taiju HIGASHI, 53536; +Cc: me, leo, marius

Hi Taiju,

Am Mittwoch, dem 26.01.2022 um 08:59 +0900 schrieb Taiju HIGASHI:
> Hi,
> 
> I would like to view PDF files in Japanese with a viewer such as
> Evince, but it seems that it cannot render Japanese at present, so I
> wrote a patch.
> 
> As far as I know, in order for Poppler to render CJK text, one of the
> following conditions must be met
> 
> 1. Install poppler with poppler-data preinstalled.
> 2. Install poppler, then poppler-data in the path expected by
> poppler. (`POPPLER_INSTALL_PREFIX/share/poppler`)
> 
> ref:
> https://github.com/freedesktop/poppler/blob/277f5de9684b3392f0d585bd36ad1a5e9e9e9ed7/CMakeLists.txt#L348-L362
> 
> Guix provides poppler and poppler-data as standalone packages, but
> installing both will not satisfy either of the above prerequisites.
> 
> So I defined poppler with poppler-data as a package with the name
> popper-with-data.
> 
> This package is intended to be used in the installation of packages
> that also have poppler as a dependency, as shown below.
> 
>    guix package -i evince --with-input=poppler=poppler-with-data
> 
> However, As a user, this is still a bit of a hassle, so if you have a
> better idea, I'd like to see it.
That seems to be one solution, but note that the Qt5 variants of
poppler would still be affected by that bug.  Now note, that poppler-
data itself does not depend on poppler, so we could simply add it as
input to the poppler package.  However, this can not be done on master,
because it'd cause 7k+ rebuilds.  Instead, I suggest we make poppler-
with-data a replacement for poppler, which should by package/inherit
then also apply to the other variants.

I've CC'd Marius, Tobias and Leo to aid me in my judgement here, but I
think grafts would be necessary if we don't want to do input rewriting
with several variants.

> Incidentally, it seems that Nix had defined its own environment
> variable (PLOPPER_DATADIR) to deal with this problem.
Are there any other packages you might want to install into
POPPLER_INSTALL_PREFIX?  If so, a colon-separated POPPLER_DATA_PATH
should be preferred.  Note that if we add that feature, we'd still have
to graft it on master currently.

Cheers




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH 0/1] Add poppler-with-data.
  2022-01-26  7:37 ` [bug#53536] [PATCH 0/1] " Liliana Marie Prikler
@ 2022-01-26 13:38   ` Taiju HIGASHI
  2022-01-26 14:16     ` Liliana Marie Prikler
  2022-01-26 13:42   ` Taiju HIGASHI
  1 sibling, 1 reply; 20+ messages in thread
From: Taiju HIGASHI @ 2022-01-26 13:38 UTC (permalink / raw)
  To: Liliana Marie Prikler, 53536; +Cc: me, leo, marius

Hi Liliana,

> That seems to be one solution, but note that the Qt5 variants of
> poppler would still be affected by that bug.

You're right, poppler-qt5 also needs a variant with poppler-data added.


> Instead, I suggest we make poppler-with-data a replacement for
> poppler, which should by package/inherit then also apply to the other
> variants.
>
> I've CC'd Marius, Tobias and Leo to aid me in my judgement here, but I
> think grafts would be necessary if we don't want to do input rewriting
> with several variants.

I apologize if I didn't understand exactly what you meant.
Am I correct in understanding that the direction of this patch is
correct?

The idea is to create a variant called poppler-qt5-with-data and replace
the input of packages that depend on poppler-qt5 with it.

I'm starting to think that this patch proposal is a realistic and
reasonable solution.

I'm using poppler-with-data to write the manifest, but without
poppler-with-data, I'm not sure what I'd do.

In fact, I'm using poppler-with-data to write manifests, and it's much
better than not using poppler-with-data.

At least I don't have to redefine evince etc. on my own anymore.

ref:
https://git.sr.ht/~taiju/taix/tree/8a3ab4407eefe720193e401cf8f11d96550733e9/item/guix-config/package-config.scm


If I am interpreting your reply incorrectly, I would appreciate it if
you could be more specific.


> Are there any other packages you might want to install into
> POPPLER_INSTALL_PREFIX?  If so, a colon-separated POPPLER_DATA_PATH
> should be preferred.  Note that if we add that feature, we'd still have
> to graft it on master currently.

At the moment, there is nothing other than poppler-data that we want to
install in POPPLER_INSTALL_PREFIX.

However, this idea, while generic, may be confusing to users, as shown
in the reason why it was deprecated in Nix.
At least, if the package poppler-with-data exists, we can speculate that
it might be able to solve the problem.

QUOTE:
    Previously we relied on an environment variable POPPLER_DATADIR
    which practically noone used and everyone was expected to set. This
    is a good candidate for a feature option because noone really
    _noticed_ that this data is not available. Disabled by default
    because of this and size of the data (22M).

ref:
https://github.com/NixOS/nixpkgs/pull/17819/commits/1bde33074efa11fa2edcf71032d2e634f852f349


Thanks




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH 0/1] Add poppler-with-data.
  2022-01-26  7:37 ` [bug#53536] [PATCH 0/1] " Liliana Marie Prikler
  2022-01-26 13:38   ` Taiju HIGASHI
@ 2022-01-26 13:42   ` Taiju HIGASHI
  1 sibling, 0 replies; 20+ messages in thread
From: Taiju HIGASHI @ 2022-01-26 13:42 UTC (permalink / raw)
  To: Liliana Marie Prikler, 53536; +Cc: me, leo, marius

Sorry, the reference URL was wrong.

ref:
https://git.sr.ht/~taiju/taix/tree/fcafe2ccb92975c9273c9fb769cb577e9d64de59/item/guix-config/package-config.scm




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH 0/1] Add poppler-with-data.
  2022-01-26 13:38   ` Taiju HIGASHI
@ 2022-01-26 14:16     ` Liliana Marie Prikler
  2022-01-27  0:55       ` Taiju HIGASHI
  0 siblings, 1 reply; 20+ messages in thread
From: Liliana Marie Prikler @ 2022-01-26 14:16 UTC (permalink / raw)
  To: Taiju HIGASHI, 53536; +Cc: me, leo, marius

Hi,

Am Mittwoch, dem 26.01.2022 um 22:38 +0900 schrieb Taiju HIGASHI:
> > Instead, I suggest we make poppler-with-data a replacement for
> > poppler, which should by package/inherit then also apply to the
> > other variants.
> > 
> > I've CC'd Marius, Tobias and Leo to aid me in my judgement here,
> > but I think grafts would be necessary if we don't want to do input
> > rewriting with several variants.
> 
> I apologize if I didn't understand exactly what you meant.
> Am I correct in understanding that the direction of this patch is
> correct?
> 
> The idea is to create a variant called poppler-qt5-with-data and
> replace the input of packages that depend on poppler-qt5 with it.
> 
> I'm starting to think that this patch proposal is a realistic and
> reasonable solution.
I think we can agree that the patch moves in the right direction, but
I'm not sure whether we can claim to be "there yet" with just that
patch alone.  If we want to make it so that evince handles CJK out of
the box (without the user needing to rewrite inputs), we would have to
replace poppler with poppler-with-data. On master first by grafts, and
on core-updates by adding it as input directly.

> I'm using poppler-with-data to write the manifest, but without
> poppler-with-data, I'm not sure what I'd do.
> 
> In fact, I'm using poppler-with-data to write manifests, and it's
> much better than not using poppler-with-data.
> 
> At least I don't have to redefine evince etc. on my own anymore.
> 
> ref:
> https://git.sr.ht/~taiju/taix/tree/8a3ab4407eefe720193e401cf8f11d96550733e9/item/guix-config/package-config.scm
> 
> 
> If I am interpreting your reply incorrectly, I would appreciate it if
> you could be more specific.
What works for you in a manifest is not necessarily something that
should be pushed to upstream as-is.  As you can easily see, it's a
transformation you're able to do locally regardless of the state of the
Guix repo.  For the main channel, "quick hacks to alleviate problems"
are typically discouraged in favour of varying degrees of "proper
solutions".

> > Are there any other packages you might want to install into
> > POPPLER_INSTALL_PREFIX?  If so, a colon-separated POPPLER_DATA_PATH
> > should be preferred.  Note that if we add that feature, we'd still
> > have to graft it on master currently.
> 
> At the moment, there is nothing other than poppler-data that we want
> to install in POPPLER_INSTALL_PREFIX.
> 
> However, this idea, while generic, may be confusing to users, as
> shown in the reason why it was deprecated in Nix.
> At least, if the package poppler-with-data exists, we can speculate
> that it might be able to solve the problem.
> 
> QUOTE:
>     Previously we relied on an environment variable POPPLER_DATADIR
>     which practically noone used and everyone was expected to set.  
> This is a good candidate for a feature option because noone
>     really _noticed_ that this data is not available.  Disabled by
>     default because of this and size of the data (22M).

Unlike Nix, we don't have feature options (not until Ludo pushes the
code to add them, at least, but that too appears to be at least one
core-updates cycle away), so it's either the all-or-nothing approach of
having or not having it as input to poppler, or the environment
variable.  Looking at the output of `guix size', poppler takes up 138.1
MiB on a disk with 7.3 MiB being poppler itself, whereas poppler-data
takes up 12.4 MiB.  I personally think that tradeoff to be worth it --
as in "let's include poppler-data in poppler so as to not discriminate
against our Non-Latin script users".  It's an increase of less than 10%
to support clearly more than 10% of the world's population (though how
many of them use PDFs is an uncertain number).

Cheers




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH 0/1] Add poppler-with-data.
  2022-01-26 14:16     ` Liliana Marie Prikler
@ 2022-01-27  0:55       ` Taiju HIGASHI
  0 siblings, 0 replies; 20+ messages in thread
From: Taiju HIGASHI @ 2022-01-27  0:55 UTC (permalink / raw)
  To: Liliana Marie Prikler, 53536; +Cc: me, leo, marius

Hi,

You're absolutely right.
This patch can only alleviate the pain, it cannot eliminate it.

This was a painstaking effort to accommodate users who do not use CJK,
but if you would consider making poppler-with-data a replacement for
poppler, that would be better.

> Unlike Nix, we don't have feature options (not until Ludo pushes the
> code to add them, at least, but that too appears to be at least one
> core-updates cycle away), so it's either the all-or-nothing approach
> of having or not having it as input to poppler, or the environment
> variable.  Looking at the output of `guix size', poppler takes up
> 138.1 MiB on a disk with 7.3 MiB being poppler itself, whereas
> poppler-data takes up 12.4 MiB.  I personally think that tradeoff to
> be worth it as in "let's include poppler-data in poppler so as to not
> discriminate against our Non-Latin script users".  It's an increase of
> less than 10% to support clearly more than 10% of the world's
> population (though how many of them use PDFs is an uncertain number).

I'm on the minority side, so I can't speak strongly, but I'd be very
happy if you would consider bundling poppler-data with poppler by
default.

I had a strong desire to use Guix, so I solved small problems on my own.
However, people who are just trying out Guix may decide that Guix is
useless just because it cannot display Japanese PDFs.

Displaying and outputting PDFs in Japanese is a very basic task, so
struggling with this is not impressive.

It's much better now, but in the past, We often encountered problems
specific to the Japanese environment in GNU/Linux.

In light of this, it is usually best to use a distribution with many
Japanese users.

If they are using Guix and run into such problems, they will think, "I
knew it".

I want more Japanese users to use this wonderful Guix, but I think it
would be a shame if these experiences affect them and they leave.

The above is my opinion as a Japanese, but other non-Latin script users
may have the same opinion.

Thanks




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler.
  2022-01-25 23:59 [bug#53536] [PATCH 0/1] Add poppler-with-data Taiju HIGASHI
  2022-01-26  0:20 ` [bug#53536] [PATCH 1/1] gnu: " Taiju HIGASHI
  2022-01-26  7:37 ` [bug#53536] [PATCH 0/1] " Liliana Marie Prikler
@ 2022-02-02  3:48 ` Taiju HIGASHI
  2022-02-02  7:27   ` Liliana Marie Prikler
  2022-02-02  3:58 ` [bug#53536] [PATCH 0/1] Add poppler-with-data Taiju HIGASHI
                   ` (2 subsequent siblings)
  5 siblings, 1 reply; 20+ messages in thread
From: Taiju HIGASHI @ 2022-02-02  3:48 UTC (permalink / raw)
  To: liliana.prikler, 53536; +Cc: Taiju HIGASHI, me, leo, marius

---
 gnu/packages/djvu.scm | 1 -
 gnu/packages/pdf.scm  | 1 +
 2 files changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/djvu.scm b/gnu/packages/djvu.scm
index 07c77f318e..4773e408da 100644
--- a/gnu/packages/djvu.scm
+++ b/gnu/packages/djvu.scm
@@ -159,7 +159,6 @@ (define-public pdf2djvu
        ("exiv2" ,exiv2)
        ("graphicsmagick" ,graphicsmagick)
        ("poppler" ,poppler)
-       ("poppler-data" ,poppler-data)
        ("util-linux-lib" ,util-linux "lib"))) ; for libuuid
     (arguments
      `(#:test-target "test"
diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm
index d4e97e1d82..6894640317 100644
--- a/gnu/packages/pdf.scm
+++ b/gnu/packages/pdf.scm
@@ -241,6 +241,7 @@ (define-public poppler
              ("libtiff" ,libtiff)
              ("lcms" ,lcms)
              ("openjpeg" ,openjpeg)
+             ("poppler-data", poppler-data)
              ("zlib" ,zlib)

              ;; To build poppler-glib (as needed by Evince), we need Cairo and
--
2.34.0




^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH 0/1] Add poppler-with-data.
  2022-01-25 23:59 [bug#53536] [PATCH 0/1] Add poppler-with-data Taiju HIGASHI
                   ` (2 preceding siblings ...)
  2022-02-02  3:48 ` [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler Taiju HIGASHI
@ 2022-02-02  3:58 ` Taiju HIGASHI
  2022-02-02  6:22 ` [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler Taiju HIGASHI
  2022-02-03  6:23 ` [bug#53536] [PATCH v3 1/2] gnu: poppler: Add poppler-data Taiju HIGASHI
  5 siblings, 0 replies; 20+ messages in thread
From: Taiju HIGASHI @ 2022-02-02  3:58 UTC (permalink / raw)
  To: liliana.prikler, 53536; +Cc: me, leo, marius

Hi,

The patch has been reworked to include poppler-data in the poppler.

gimp and glimp have poppler-data as input from the start, but if you
remove it, you can't build, so I left it in.

Thanks




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler.
  2022-01-25 23:59 [bug#53536] [PATCH 0/1] Add poppler-with-data Taiju HIGASHI
                   ` (3 preceding siblings ...)
  2022-02-02  3:58 ` [bug#53536] [PATCH 0/1] Add poppler-with-data Taiju HIGASHI
@ 2022-02-02  6:22 ` Taiju HIGASHI
  2022-02-03  6:23 ` [bug#53536] [PATCH v3 1/2] gnu: poppler: Add poppler-data Taiju HIGASHI
  5 siblings, 0 replies; 20+ messages in thread
From: Taiju HIGASHI @ 2022-02-02  6:22 UTC (permalink / raw)
  To: liliana.prikler, 53536; +Cc: me, leo, marius

Sorry, I wrote it earlier in a hurry during a break, so when I reread
it, the text was wrong in many ways.

Removing poppler-data from gimp and glimpse caused the build to fail, so
I'm keeping it in input.




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler.
  2022-02-02  3:48 ` [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler Taiju HIGASHI
@ 2022-02-02  7:27   ` Liliana Marie Prikler
  2022-02-02 13:35     ` Taiju HIGASHI
  0 siblings, 1 reply; 20+ messages in thread
From: Liliana Marie Prikler @ 2022-02-02  7:27 UTC (permalink / raw)
  To: Taiju HIGASHI, 53536; +Cc: me, leo, marius

Am Mittwoch, dem 02.02.2022 um 12:48 +0900 schrieb Taiju HIGASHI:
> ---
>  gnu/packages/djvu.scm | 1 -
>  gnu/packages/pdf.scm  | 1 +
>  2 files changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/gnu/packages/djvu.scm b/gnu/packages/djvu.scm
> index 07c77f318e..4773e408da 100644
> --- a/gnu/packages/djvu.scm
> +++ b/gnu/packages/djvu.scm
> @@ -159,7 +159,6 @@ (define-public pdf2djvu
>         ("exiv2" ,exiv2)
>         ("graphicsmagick" ,graphicsmagick)
>         ("poppler" ,poppler)
> -       ("poppler-data" ,poppler-data)
>         ("util-linux-lib" ,util-linux "lib"))) ; for libuuid
>      (arguments
>       `(#:test-target "test"
> diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm
> index d4e97e1d82..6894640317 100644
> --- a/gnu/packages/pdf.scm
> +++ b/gnu/packages/pdf.scm
> @@ -241,6 +241,7 @@ (define-public poppler
>               ("libtiff" ,libtiff)
>               ("lcms" ,lcms)
>               ("openjpeg" ,openjpeg)
> +             ("poppler-data", poppler-data)
>               ("zlib" ,zlib)
> 
>               ;; To build poppler-glib (as needed by Evince), we need
> Cairo and
> --
> 2.34.0
Note that the ChangeLog is missing here.  It should be something along
the lines of

* gnu/packages/pdf.scm (poppler)[inputs]: Add poppler-data.
* gnu/packages/djvu.scm (pdf2djvu)[inputs]: Remove poppler-data.

In my personal opinion, adding and removing should be done in separate
commits, as it appears that there are other ways in which a package can
depend on poppler-data.  On that note, I haven't built the package, so
I need to ask you: are the contents of poppler-data now included in the
install of poppler?  If not, that might be a reason as to why gimp and
glimpse are failing.

Cheers




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler.
  2022-02-02  7:27   ` Liliana Marie Prikler
@ 2022-02-02 13:35     ` Taiju HIGASHI
  2022-02-02 14:16       ` Liliana Marie Prikler
  0 siblings, 1 reply; 20+ messages in thread
From: Taiju HIGASHI @ 2022-02-02 13:35 UTC (permalink / raw)
  To: Liliana Marie Prikler; +Cc: 53536, me, leo, marius

> I need to ask you: are the contents of poppler-data now included in the
> install of poppler?  If not, that might be a reason as to why gimp and
> glimpse are failing.

It is understood as follows.

poppler (with poppler-data)

  1. During the build process, poppler will check to see if poppler-data
  has been installed by pkg-config.
  2. Newer versions of poppler depend on poppler-data directly, and
  pkg-config can detect poppler-data during the build process.

gimp and glimpse (without poppler-data)

  1. During the build process, gimp and glimpse will check to see if poppler-data
  has been installed by pkg-config.
  2. gimp or glimpse depend on poppler-data indirectly, and pkg-config
  cannot detect poppler-data during the build process.

The above is just a problem in the build process.
Poppler knows where the poppler-data is during the build process, so the
installed poppler can use it in its processing.

I have confirmed in the poppler build log that poppler-data is detected.

I have also confirmed that evince can render Japanese PDFs with the
contents of this patch.


I'll add the changelog and split the commits together at the end.

Thanks




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler.
  2022-02-02 13:35     ` Taiju HIGASHI
@ 2022-02-02 14:16       ` Liliana Marie Prikler
  2022-02-02 15:47         ` Taiju HIGASHI
  0 siblings, 1 reply; 20+ messages in thread
From: Liliana Marie Prikler @ 2022-02-02 14:16 UTC (permalink / raw)
  To: Taiju HIGASHI; +Cc: 53536, me, leo, marius

Hi,

Am Mittwoch, dem 02.02.2022 um 22:35 +0900 schrieb Taiju HIGASHI:
> 
> [...]
> gimp and glimpse (without poppler-data)
> 
>   1. During the build process, gimp and glimpse will check to see if
> poppler-data
>   has been installed by pkg-config.
>   2. gimp or glimpse depend on poppler-data indirectly, and pkg-
> config
>   cannot detect poppler-data during the build process.
Assertion (2) is wrong here: gimp's configure.ac directly checks for
the existence of poppler-data.  That makes it a direct dependency as
far as I'm concerned, even if they only ever use it through poppler.

> I have also confirmed that evince can render Japanese PDFs with the
> contents of this patch.
Good to know.

> I'll add the changelog and split the commits together at the end.
Thanks.

@Leo I've heard that there's a Rust related world rebuild coming soon.
Can we tack this feature onto that?

Cheers




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler.
  2022-02-02 14:16       ` Liliana Marie Prikler
@ 2022-02-02 15:47         ` Taiju HIGASHI
  2022-02-03  6:54           ` Liliana Marie Prikler
  0 siblings, 1 reply; 20+ messages in thread
From: Taiju HIGASHI @ 2022-02-02 15:47 UTC (permalink / raw)
  To: Liliana Marie Prikler; +Cc: 53536, me, leo, marius

> Assertion (2) is wrong here: gimp's configure.ac directly checks for
> the existence of poppler-data.  That makes it a direct dependency as
> far as I'm concerned, even if they only ever use it through poppler.

I did not convey the context correctly.

I thought I was describing the behavior of the gimp package with the
poppler-data dependency removed (= gimp,which indirectly depend on
poppler-data).
In that state, gimp will fail to build because pkg-config cannot detect
poppler-data during the build process.
Therefore, I left the poppler-data dependency in gimp.

I'm really sorry for my poor explanation.




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH v3 1/2] gnu: poppler: Add poppler-data.
  2022-01-25 23:59 [bug#53536] [PATCH 0/1] Add poppler-with-data Taiju HIGASHI
                   ` (4 preceding siblings ...)
  2022-02-02  6:22 ` [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler Taiju HIGASHI
@ 2022-02-03  6:23 ` Taiju HIGASHI
  2022-02-03  6:23   ` [bug#53536] [PATCH v3 2/2] gnu: pdf2djvu: Remove poppler-data Taiju HIGASHI
  2022-02-06  0:00   ` bug#53536: [PATCH v3 1/2] gnu: poppler: Add poppler-data Liliana Marie Prikler
  5 siblings, 2 replies; 20+ messages in thread
From: Taiju HIGASHI @ 2022-02-03  6:23 UTC (permalink / raw)
  To: liliana.prikler; +Cc: 53536, me, Taiju HIGASHI, leo, marius

---
 gnu/packages/pdf.scm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm
index d4e97e1d82..6894640317 100644
--- a/gnu/packages/pdf.scm
+++ b/gnu/packages/pdf.scm
@@ -241,6 +241,7 @@ (define-public poppler
              ("libtiff" ,libtiff)
              ("lcms" ,lcms)
              ("openjpeg" ,openjpeg)
+             ("poppler-data", poppler-data)
              ("zlib" ,zlib)

              ;; To build poppler-glib (as needed by Evince), we need Cairo and
--
2.34.0




^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH v3 2/2] gnu: pdf2djvu: Remove poppler-data.
  2022-02-03  6:23 ` [bug#53536] [PATCH v3 1/2] gnu: poppler: Add poppler-data Taiju HIGASHI
@ 2022-02-03  6:23   ` Taiju HIGASHI
  2022-02-03  6:28     ` Taiju HIGASHI
  2022-02-06  0:00   ` bug#53536: [PATCH v3 1/2] gnu: poppler: Add poppler-data Liliana Marie Prikler
  1 sibling, 1 reply; 20+ messages in thread
From: Taiju HIGASHI @ 2022-02-03  6:23 UTC (permalink / raw)
  To: liliana.prikler; +Cc: 53536, me, Taiju HIGASHI, leo, marius

---
 gnu/packages/djvu.scm | 1 -
 1 file changed, 1 deletion(-)

diff --git a/gnu/packages/djvu.scm b/gnu/packages/djvu.scm
index 07c77f318e..4773e408da 100644
--- a/gnu/packages/djvu.scm
+++ b/gnu/packages/djvu.scm
@@ -159,7 +159,6 @@ (define-public pdf2djvu
        ("exiv2" ,exiv2)
        ("graphicsmagick" ,graphicsmagick)
        ("poppler" ,poppler)
-       ("poppler-data" ,poppler-data)
        ("util-linux-lib" ,util-linux "lib"))) ; for libuuid
     (arguments
      `(#:test-target "test"
--
2.34.0




^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH v3 2/2] gnu: pdf2djvu: Remove poppler-data.
  2022-02-03  6:23   ` [bug#53536] [PATCH v3 2/2] gnu: pdf2djvu: Remove poppler-data Taiju HIGASHI
@ 2022-02-03  6:28     ` Taiju HIGASHI
  0 siblings, 0 replies; 20+ messages in thread
From: Taiju HIGASHI @ 2022-02-03  6:28 UTC (permalink / raw)
  To: liliana.prikler; +Cc: 53536, me, leo, marius

I separated the commits.




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler.
  2022-02-02 15:47         ` Taiju HIGASHI
@ 2022-02-03  6:54           ` Liliana Marie Prikler
  2022-02-03  7:23             ` Taiju HIGASHI
  0 siblings, 1 reply; 20+ messages in thread
From: Liliana Marie Prikler @ 2022-02-03  6:54 UTC (permalink / raw)
  To: Taiju HIGASHI; +Cc: 53536, me, leo, marius

Am Donnerstag, dem 03.02.2022 um 00:47 +0900 schrieb Taiju HIGASHI:
> > Assertion (2) is wrong here: gimp's configure.ac directly checks for
> > the existence of poppler-data.  That makes it a direct dependency as
> > far as I'm concerned, even if they only ever use it through poppler.
> 
> I did not convey the context correctly.
> 
> I thought I was describing the behavior of the gimp package with the
> poppler-data dependency removed (= gimp,which indirectly depend on
> poppler-data).
> In that state, gimp will fail to build because pkg-config cannot detect
> poppler-data during the build process.
> Therefore, I left the poppler-data dependency in gimp.
> 
> I'm really sorry for my poor explanation.
Again, w.r.t. your explanation I claim that gimp does not "indirectly"
depend on poppler-data, but directly.  Voilà, le configure.ac [1].

[1]
https://gitlab.gnome.org/GNOME/gimp/-/blob/GIMP_2_10_30/configure.ac#L2023-L2024




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler.
  2022-02-03  6:54           ` Liliana Marie Prikler
@ 2022-02-03  7:23             ` Taiju HIGASHI
  0 siblings, 0 replies; 20+ messages in thread
From: Taiju HIGASHI @ 2022-02-03  7:23 UTC (permalink / raw)
  To: Liliana Marie Prikler; +Cc: 53536, me, leo, marius

> > > Assertion (2) is wrong here: gimp's configure.ac directly checks for
> > > the existence of poppler-data.  That makes it a direct dependency as
> > > far as I'm concerned, even if they only ever use it through poppler.
> >
> > I did not convey the context correctly.
> >
> > I thought I was describing the behavior of the gimp package with the
> > poppler-data dependency removed (= gimp,which indirectly depend on
> > poppler-data).
> > In that state, gimp will fail to build because pkg-config cannot detect
> > poppler-data during the build process.
> > Therefore, I left the poppler-data dependency in gimp.
> >
> > I'm really sorry for my poor explanation.
> Again, w.r.t. your explanation I claim that gimp does not "indirectly"
> depend on poppler-data, but directly.  Voilà, le configure.ac [1].

> > Assertion (2) is wrong here: gimp's configure.ac directly checks for
> > the existence of poppler-data.  That makes it a direct dependency as
> > far as I'm concerned, even if they only ever use it through poppler.
>
> I did not convey the context correctly.
>
> I thought I was describing the behavior of the gimp package with the
> poppler-data dependency removed (= gimp,which indirectly depend on
> poppler-data).
> In that state, gimp will fail to build because pkg-config cannot detect
> poppler-data during the build process.
> Therefore, I left the poppler-data dependency in gimp.
>
> I'm really sorry for my poor explanation.

Your assertion that the Gimp depends on poppler-data directly, not
indirectly, is absolutely true and correct.

As you pointed out, the gimp source code definitely depends directly on
poppler-data, so it can't be built without poppler-data.

I was describing the removal of poppler-data from the input in the Gimp
package definition as "gimp indirectly depends on poppler-data", which
complicated the claim. I apologize for that.




^ permalink raw reply	[flat|nested] 20+ messages in thread

* bug#53536: [PATCH v3 1/2] gnu: poppler: Add poppler-data.
  2022-02-03  6:23 ` [bug#53536] [PATCH v3 1/2] gnu: poppler: Add poppler-data Taiju HIGASHI
  2022-02-03  6:23   ` [bug#53536] [PATCH v3 2/2] gnu: pdf2djvu: Remove poppler-data Taiju HIGASHI
@ 2022-02-06  0:00   ` Liliana Marie Prikler
  1 sibling, 0 replies; 20+ messages in thread
From: Liliana Marie Prikler @ 2022-02-06  0:00 UTC (permalink / raw)
  To: Taiju HIGASHI; +Cc: me, 53536-done, marius, leo

Am Donnerstag, dem 03.02.2022 um 15:23 +0900 schrieb Taiju HIGASHI:
> ---
>  gnu/packages/pdf.scm | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm
> index d4e97e1d82..6894640317 100644
> --- a/gnu/packages/pdf.scm
> +++ b/gnu/packages/pdf.scm
> @@ -241,6 +241,7 @@ (define-public poppler
>               ("libtiff" ,libtiff)
>               ("lcms" ,lcms)
>               ("openjpeg" ,openjpeg)
> +             ("poppler-data", poppler-data)
>               ("zlib" ,zlib)
Pushed to core-updates along with some other changes.  You still need
to keep your own poppler-with-data around for some while, but it should
be fixed with the next merge, so I'll mark this as done :)

Cheers




^ permalink raw reply	[flat|nested] 20+ messages in thread

end of thread, other threads:[~2022-02-06  0:01 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-01-25 23:59 [bug#53536] [PATCH 0/1] Add poppler-with-data Taiju HIGASHI
2022-01-26  0:20 ` [bug#53536] [PATCH 1/1] gnu: " Taiju HIGASHI
2022-01-26  7:37 ` [bug#53536] [PATCH 0/1] " Liliana Marie Prikler
2022-01-26 13:38   ` Taiju HIGASHI
2022-01-26 14:16     ` Liliana Marie Prikler
2022-01-27  0:55       ` Taiju HIGASHI
2022-01-26 13:42   ` Taiju HIGASHI
2022-02-02  3:48 ` [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler Taiju HIGASHI
2022-02-02  7:27   ` Liliana Marie Prikler
2022-02-02 13:35     ` Taiju HIGASHI
2022-02-02 14:16       ` Liliana Marie Prikler
2022-02-02 15:47         ` Taiju HIGASHI
2022-02-03  6:54           ` Liliana Marie Prikler
2022-02-03  7:23             ` Taiju HIGASHI
2022-02-02  3:58 ` [bug#53536] [PATCH 0/1] Add poppler-with-data Taiju HIGASHI
2022-02-02  6:22 ` [bug#53536] [PATCH v2] gnu: Include poppler-data with poppler Taiju HIGASHI
2022-02-03  6:23 ` [bug#53536] [PATCH v3 1/2] gnu: poppler: Add poppler-data Taiju HIGASHI
2022-02-03  6:23   ` [bug#53536] [PATCH v3 2/2] gnu: pdf2djvu: Remove poppler-data Taiju HIGASHI
2022-02-03  6:28     ` Taiju HIGASHI
2022-02-06  0:00   ` bug#53536: [PATCH v3 1/2] gnu: poppler: Add poppler-data Liliana Marie Prikler

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).