all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Liliana Marie Prikler <liliana.prikler@ist.tugraz.at>
To: Taiju HIGASHI <higashi@taiju.info>, 53536@debbugs.gnu.org
Cc: me@tobias.gr, leo@famulari.name, marius@gnu.org
Subject: [bug#53536] [PATCH 0/1] Add poppler-with-data.
Date: Wed, 26 Jan 2022 08:37:30 +0100	[thread overview]
Message-ID: <db3cd12ea6148521d8fb60dda0812693ceb82ad0.camel@ist.tugraz.at> (raw)
In-Reply-To: <20220125235931.4451-1-higashi@taiju.info>

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




  parent reply	other threads:[~2022-01-26  7:39 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 [this message]
2022-01-26 13:38   ` [bug#53536] [PATCH 0/1] " 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

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=db3cd12ea6148521d8fb60dda0812693ceb82ad0.camel@ist.tugraz.at \
    --to=liliana.prikler@ist.tugraz.at \
    --cc=53536@debbugs.gnu.org \
    --cc=higashi@taiju.info \
    --cc=leo@famulari.name \
    --cc=marius@gnu.org \
    --cc=me@tobias.gr \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.