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

Bug#825396: marked as done (aptitude: locale-dependent formatting URL port numbers, /var/lib/apt/lists/* files)

Your message dated Sat, 11 Jun 2016 16:34:42 +0000
with message-id <E1bBlru-0008Pn-RX@franck.debian.org>
and subject line Bug#825396: fixed in apt 1.3~exp2
has caused the Debian Bug report #825396,
regarding aptitude: locale-dependent formatting URL port numbers, /var/lib/apt/lists/* files
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.)


-- 
825396: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=825396
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: aptitude
Version: 0.8.1-1
Severity: important
Tags: patch

Dear Maintainer,

A while back, I noticed that aptitude and apt could no longer be used
together on my system: If aptitude had been used to update Packages and
Sources lists too, apt would not see the lists. (And vice versa.)

This happened because aptitude now formats port numbers in URLs with
thousands' grouping characters as defined by the locale (in this case
en_US.utf-8):

,----
| $ sudo aptitude update
| Hit http://localhost:9,999/debian sid InRelease
| Get: 1 http://localhost:9,999/debian stretch InRelease [232 kB]
| [...]
| $ sudo LANG=C aptitude update
| Get: 1 http://localhost:9999/debian sid InRelease [205 kB]
| Get: 2 http://localhost:9999/debian stretch InRelease [232 kB]
| [...]
`----

The extra comma also ended up as part of the filenames in
/var/lib/apt/lists, which explains the described behavior.

Version 0.7.5-3 does not have this problem.

Further investigation shows that commit
198c06d1d481728147f61c1482ac7e52337363db (Change filename of lock file
from tag "literal" to "filename") contains a seemingly unrelated change
where setlocale() is replaced with a call to the C++ stdlib.

Reverting that bit fixes the problem for me; I have attached a patch.

Cheers,
-Hilko

-- Package-specific info:

Terminal: eterm-color
$DISPLAY is set.
which aptitude: /usr/bin/aptitude

aptitude version information:
aptitude 0.8.1
Compiler: g++ 5.3.1 20160429
Compiled against:
  apt version 5.0.0
  NCurses version 6.0
  libsigc++ version: 2.8.0
  Gtk+ support disabled.
  Qt support disabled.

Current library versions:
  NCurses version: ncurses 6.0.20160319
  cwidget version: 0.5.17
  Apt version: 5.0.0

aptitude linkage:
	linux-vdso.so.1 (0x00007ffd8030e000)
	libapt-pkg.so.5.0 => /usr/lib/x86_64-linux-gnu/libapt-pkg.so.5.0 (0x00007f845e6d2000)
	libncursesw.so.5 => /lib/x86_64-linux-gnu/libncursesw.so.5 (0x00007f845e4a2000)
	libtinfo.so.5 => /lib/x86_64-linux-gnu/libtinfo.so.5 (0x00007f845e277000)
	libsigc-2.0.so.0 => /usr/lib/x86_64-linux-gnu/libsigc-2.0.so.0 (0x00007f845e070000)
	libcwidget.so.3 => /usr/lib/x86_64-linux-gnu/libcwidget.so.3 (0x00007f845dd73000)
	libsqlite3.so.0 => /usr/lib/x86_64-linux-gnu/libsqlite3.so.0 (0x00007f845da6e000)
	libboost_iostreams.so.1.58.0 => /usr/lib/x86_64-linux-gnu/libboost_iostreams.so.1.58.0 (0x00007f845d854000)
	libboost_filesystem.so.1.58.0 => /usr/lib/x86_64-linux-gnu/libboost_filesystem.so.1.58.0 (0x00007f845d63b000)
	libboost_system.so.1.58.0 => /usr/lib/x86_64-linux-gnu/libboost_system.so.1.58.0 (0x00007f845d436000)
	libxapian.so.22 => /usr/lib/x86_64-linux-gnu/libxapian.so.22 (0x00007f845d032000)
	libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007f845ce15000)
	libstdc++.so.6 => /usr/lib/x86_64-linux-gnu/libstdc++.so.6 (0x00007f845ca94000)
	libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007f845c796000)
	libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007f845c580000)
	libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f845c1db000)
	libutil.so.1 => /lib/x86_64-linux-gnu/libutil.so.1 (0x00007f845bfd8000)
	libdl.so.2 => /lib/x86_64-linux-gnu/libdl.so.2 (0x00007f845bdd4000)
	libresolv.so.2 => /lib/x86_64-linux-gnu/libresolv.so.2 (0x00007f845bbbc000)
	libz.so.1 => /lib/x86_64-linux-gnu/libz.so.1 (0x00007f845b9a1000)
	libbz2.so.1.0 => /lib/x86_64-linux-gnu/libbz2.so.1.0 (0x00007f845b791000)
	liblzma.so.5 => /lib/x86_64-linux-gnu/liblzma.so.5 (0x00007f845b56d000)
	liblz4.so.1 => /usr/lib/x86_64-linux-gnu/liblz4.so.1 (0x00007f845b35b000)
	librt.so.1 => /lib/x86_64-linux-gnu/librt.so.1 (0x00007f845b152000)
	libuuid.so.1 => /lib/x86_64-linux-gnu/libuuid.so.1 (0x00007f845af4d000)
	/lib64/ld-linux-x86-64.so.2 (0x000055e8e522b000)

-- System Information:
Debian Release: stretch/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.4.0-1-amd64 (SMP w/6 CPU cores)
Locale: LANG=en_US.utf8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages aptitude depends on:
ii  aptitude-common            0.8.1-1
ii  libapt-pkg5.0              1.2.12
ii  libboost-filesystem1.58.0  1.58.0+dfsg-5+b1
ii  libboost-iostreams1.58.0   1.58.0+dfsg-5+b1
ii  libboost-system1.58.0      1.58.0+dfsg-5+b1
ii  libc6                      2.22-9
ii  libcwidget3v5              0.5.17-4+b1
ii  libgcc1                    1:6.1.1-4
ii  libncursesw5               6.0+20160319-1
ii  libsigc++-2.0-0v5          2.8.0-1
ii  libsqlite3-0               3.13.0-1
ii  libstdc++6                 6.1.1-4
ii  libtinfo5                  6.0+20160319-1
ii  libxapian22v5              1.2.23-1

Versions of packages aptitude recommends:
ii  libparse-debianchangelog-perl  1.2.0-8
ii  sensible-utils                 0.0.9

Versions of packages aptitude suggests:
ii  apt-xapian-index                0.47+nmu2
ii  aptitude-doc-en [aptitude-doc]  0.8.1-1
pn  debtags                         <none>
pn  tasksel                         <none>

-- no debconf information

Index: aptitude/src/main.cc
===================================================================
--- aptitude.orig/src/main.cc
+++ aptitude/src/main.cc
@@ -659,16 +659,7 @@ int main(int argc, char *argv[])
   //cw::util::transcode_mbtow_err=handle_mbtow_error;
   //cw::util::transcode_wtomb_err=handle_wtomb_error;
 
-  // Use the C++ classes if possible, otherwise if ::global() is not called the
-  // internal state of std::locale is not initialized properly
-  //
-  // it can throw an exception if the locale defined in the environment is not
-  // valid
-  try {
-    std::locale::global(std::locale(""));
-  } catch (...) {
-    setlocale(LC_ALL, "");
-  }
+  setlocale(LC_ALL, "");
   bindtextdomain(PACKAGE, LOCALEDIR);
   textdomain(PACKAGE);

--- End Message ---
--- Begin Message --- 
Source: apt
Source-Version: 1.3~exp2

We believe that the bug you reported is fixed in the latest version of
apt, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 825396@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Julian Andres Klode <jak@debian.org> (supplier of updated apt package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Sat, 11 Jun 2016 17:23:19 +0200
Source: apt
Binary: apt libapt-pkg5.0 libapt-inst2.0 apt-doc libapt-pkg-dev libapt-pkg-doc apt-utils apt-transport-https
Architecture: source
Version: 1.3~exp2
Distribution: experimental
Urgency: medium
Maintainer: APT Development Team <deity@lists.debian.org>
Changed-By: Julian Andres Klode <jak@debian.org>
Description:
 apt        - commandline package manager
 apt-doc    - documentation for APT
 apt-transport-https - https download transport for APT
 apt-utils  - package management related utility programs
 libapt-inst2.0 - deb package format runtime library
 libapt-pkg-dev - development files for APT's libapt-pkg and libapt-inst
 libapt-pkg-doc - documentation for APT development
 libapt-pkg5.0 - package management runtime library
Closes: 819697 824456 824503 824702 825396 825742 826043 826291
Changes:
 apt (1.3~exp2) experimental; urgency=medium
 .
   [ Johannes 'josch' Schauer ]
   * edsp: document unique package identifiers
 .
   [ David Kalnischkies ]
   * edsp: warn if unexpected stanzas appear in the solution
   * show globalerrors before asking for confirmation
   * show final solution in --no-download --fix-missing mode
   * document --no- as --show-upgraded is the default (Closes: 824456)
   * fail instead of segfault on unreadable config files (Closes: 824503)
   * convert EDSP to be based on FileFd instead of FILE*
   * edsp: dump: support dumping into compressed file
   * edsp: add Forbid-{New-Install,Remove} and Upgrade-All
   * update symbols file
   * no-change bump of Standards-Version to 3.9.8
   * override lintian on doxygens embedded-javascript-library
   * fix and document on the fly compressor config
   * prevent C++ locale number formatting in text APIs (Closes: #825396)
   * accept only the expected UTC timezones in date parsing (Closes: 819697)
   * avoid changing the global LC_TIME for Release writing
   * use de-localed std::put_time instead rolling our own
   * use std::locale::global instead of setlocale
   * look into the right textdomain for apt-utils again
   * try to detect sudo spawned root-shell in prefixing (Closes: 825742)
   * ignore std::locale exception on non-existent "" locale
   * apt-key: change to / before find to satisfy its CWD needs.
     Thanks to Samuel Thibault for 'finding' the culprit! (Closes: 826043)
   * edsp: use an ID mapping for the internal solver
   * edsp: use a stanza based interface for solution writing
   * edsp: optionally store a compressed copy of the last scenario
   * move 'dump' solver from apt-utils to apt package
   * edsp: if internal is used, keep this decision
   * edsp: if logging is requested, do it for internal, too
   * edsp: drop privileges before executing solvers
   * don't explicitly configure the last round of packages
   * drop Dpkg::MaxArgs in favor of Dpkg::MaxArgsBytes
   * do not hang on piped input in PipedFileFdPrivate
   * don't leak an FD in lz4 (de)compression
   * don't leak EDSP solver output fd
   * don't leak FD in AutoProxyDetect command return parsing
 .
   [ Julian Andres Klode ]
   * Provide complete apt bash completion.
     Thanks to Elias Fröhner and Svyatoslav Gryaznov for the initial work
     (LP: #1573547)
   * apt.systemd.daily: Put opening brace of check_power on extra line
     (LP: #1581985)
   * Add conflicting Signed-By values to error message
   * Normalize Signed-By values by removing trailing commas everywhere
   * Pass -fvisibility-inlines-hidden to g++
 .
   [ Zhou Mo ]
   * zh_CN.po: update simplified Chinese translation.
 .
   [ Yuri Kozlov ]
   * Russian program translation update (Closes: 824702)
 .
   [ Takuma Yamada ]
   * Japanese program translation update (Closes: 826291)
Checksums-Sha1:
 20fbb0afa8a99cbe3f6a2d9fd2461280d7fe369b 2353 apt_1.3~exp2.dsc
 7a586fb02e7035bf34adad7108dd67a10e8d7990 2066088 apt_1.3~exp2.tar.xz
Checksums-Sha256:
 e6b3d0f9870343260da30f946a751120991d0a2edf22483dbd7bc903cf845983 2353 apt_1.3~exp2.dsc
 29a88df31a680a31d7035ac8f8befebe5a34c8843c45dfc915f43f3c8dba6d43 2066088 apt_1.3~exp2.tar.xz
Files:
 210c78a3191970ee669aeef8e00c8147 2353 admin important apt_1.3~exp2.dsc
 0b0a68913dd3934141f340f7230ae7a1 2066088 admin important apt_1.3~exp2.tar.xz

-----BEGIN PGP SIGNATURE-----

iQIcBAEBCgAGBQJXXC2jAAoJENc8OeVlgLOGV+kQAIQLwkxnEyX9dqMGTdqLGmGF
XuWC1BrgLRaKpepSfRTpeOqZUeJWFIC0AkYC/dzkBTBncwc9LWAvnyubTU9cr8Wz
QmxWn6HDsVlaG9gnSsdi30d2wOA6fCIgaTppvJh9Aax7Dpntf4nUSraovMFKaxLw
/JBGrqcmsBSDQC+Knw9svImC24tkuhBVB4oVqvCC+RGbkL1eZD7DjQWU4laf6JIA
LsQoQC4nczzR0kx5FgEZTc78mVosd5xOnZhV/iAk68mK3bs8z1FU/qVgdpUC4VFf
soYvp+kL3kBO3Y7O3RSyP9/uhn6wbk+4pZx7CUHq4cRWtNMknajii9MKZ7HYiN4T
4uU2BZgxDyLm5m8RSRnLHqmpKthgPatfS0lSzzO4Qcsyi5RAZnJ+9R8eVAQch4Mr
F7OYzXPH0vcO3WTXqZqhPIiDMhdX7YUvv0jdPmneakDYdSu9bhlBEm16aghFH/h7
nAtMWW7/pWtsOwo6crCTyIiQUT/LPdNC6rY7CJbMz05D8dER/qUullMtvWffIe0T
7OcIUftwsRGhWjAAfVwZSAR+DJFTjQMEO9pKEgaA9ypyJ/jadf7U/kfvti7qe1Q8
QWiGu3KDt+ncDEZ1h9vKzl6s5d20WVeGwCZdH3C2vbxQ775klQKliu8cv01wlvIg
c8Do3nRp6wiBtuKSoxYe
=Ehha
-----END PGP SIGNATURE-----

--- End Message ---
Reply to: