[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#858600: marked as done (unblock: gladtex/2.1-2)

Your message dated Fri, 24 Mar 2017 19:19:08 +0100
with message-id <6b64c7e2-3c4c-044d-922b-a230e6d807c3@debian.org>
and subject line Re: Bug#858600: unblock: gladtex/2.1-2
has caused the Debian Bug report #858600,
regarding unblock: gladtex/2.1-2
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
858600: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=858600
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock

Please unblock package gladtex

The version in testing depends on the wrong texlive packages and hence won't
work for users not using a full texlive setup.

Furthermore, the package has issues with the locale, which I believe is fixed
with a minimal patch.

I have tried the new package in a minimal, clean [Stretch] pbuilder chroot and
it works fine.

The (short) debdiff is attached.

Thanks

unblock gladtex/2.1-2

-- System Information:
Debian Release: 9.0
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'testing-debug'), (450, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.9.0-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

diff -Nru gladtex-2.1/debian/changelog gladtex-2.1/debian/changelog
--- gladtex-2.1/debian/changelog	2016-08-30 17:48:08.000000000 +0200
+++ gladtex-2.1/debian/changelog	2017-03-16 11:35:43.000000000 +0100
@@ -1,3 +1,10 @@
+gladtex (2.1-2) unstable; urgency=medium
+
+  * correct dependency information for library package (Closes: #857911)
+  * add ability to work in environments without locale set up
+
+ -- Sebastian Humenda <shumenda@gmx.de>  Thu, 16 Mar 2017 11:35:43 +0100
+
 gladtex (2.1-1) unstable; urgency=medium
 
   * new upstream release
diff -Nru gladtex-2.1/debian/control gladtex-2.1/debian/control
--- gladtex-2.1/debian/control	2016-08-30 17:48:08.000000000 +0200
+++ gladtex-2.1/debian/control	2017-03-16 11:35:43.000000000 +0100
@@ -10,7 +10,8 @@
 
 Package: python3-gleetex
 Architecture: all
-Depends: ${misc:Depends}, ${python3:Depends}, texlive-latex-base, preview-latex-style, dvipng
+Depends: ${misc:Depends}, ${python3:Depends}, texlive-latex-recommended,
+    texlive-fonts-recommended, preview-latex-style, dvipng
 Description: Module to embed LaTeX equations in HTML files
  GleeTeX is a python module to create images from formulas. It can parse HTML
  documents and replace all occurrences of <eq>some formula</eq> through images.
diff -Nru gladtex-2.1/debian/patches/ignore_unset_locale gladtex-2.1/debian/patches/ignore_unset_locale
--- gladtex-2.1/debian/patches/ignore_unset_locale	1970-01-01 01:00:00.000000000 +0100
+++ gladtex-2.1/debian/patches/ignore_unset_locale	2017-03-16 11:35:43.000000000 +0100
@@ -0,0 +1,28 @@
+Description: correctly handle unset locale
+ The program couldn't handle an unset locale and failed to work under such
+ circumstances. The minimal patch corrects this behaviour.
+ It has been applied upstream, too.
+Author: Sebastian Humenda <shumenda@gmx.de>
+Last-Update: <2017-03-16>
+
+Index: gladtex/gleetex/document.py
+===================================================================
+--- gladtex.orig/gleetex/document.py
++++ gladtex/gleetex/document.py
+@@ -190,12 +190,14 @@ class LaTeXDocument:
+             # try to guess language and hence character set (fontenc)
+             import locale
+             language = locale.getdefaultlocale()
+-            if language: # extract just the language code
++            if language and language[0]: # extract just the language code
+                 language = language[0].split('_')[0]
++            if not language or not language[0]:
++                language = 'en'
+             # check whether language on computer is within T1 and hence whether
+             # it should be loaded; I know that this can be a misleading
+             # assumption, but there's no better way that I know of
+-            if language in ['fr', 'es', 'it', 'de', 'nl', 'ro']:
++            if language in ['fr', 'es', 'it', 'de', 'nl', 'ro', 'en']:
+                 encoding_preamble += '\n\\usepackage[T1]{fontenc}'
+             else:
+                 raise ValueError(("Language not supported by T1 fontenc "
diff -Nru gladtex-2.1/debian/patches/series gladtex-2.1/debian/patches/series
--- gladtex-2.1/debian/patches/series	2016-08-30 17:48:08.000000000 +0200
+++ gladtex-2.1/debian/patches/series	2017-03-16 11:35:43.000000000 +0100
@@ -1 +1,2 @@
 fix_clean_command
+ignore_unset_locale

Attachment: signature.asc
Description: PGP signature


--- End Message ---
--- Begin Message --- 
On 24/03/17 10:37, Sebastian Humenda wrote:
> Package: release.debian.org
> Severity: normal
> User: release.debian.org@packages.debian.org
> Usertags: unblock
> 
> Please unblock package gladtex
> 
> The version in testing depends on the wrong texlive packages and hence won't
> work for users not using a full texlive setup.
> 
> Furthermore, the package has issues with the locale, which I believe is fixed
> with a minimal patch.
> 
> I have tried the new package in a minimal, clean [Stretch] pbuilder chroot and
> it works fine.
> 
> The (short) debdiff is attached.

This was already unblocked.

Hints in place:
==> ivodd
  # 20170319
  # #857911
  unblock gladtex/2.1-2
  age-days 5 gladtex/2.1-2

Cheers,
Emilio

--- End Message ---
Reply to: