Bug#566562: liblicense-dev: patch provided: needed Depends: hicolor-icon-theme

2010-02-18 Thread gregor herrmann
On Sat, 30 Jan 2010 12:28:27 +, Dennis Sheil wrote:

 The post-installation script of liblicense-dev touches 
 /usr/share/icons/hicolor so as to regenerate GNOME's
 icon cache.  The hicolor-icon-theme package is not a
 dependency though so liblicense-dev fails when it is
 not aready installed.
 
 Patch attached.

1) I don't see how adding hicolor-icon-theme to _Build_-Depends helps
   at installation time.
2) IMO the touch'ing in postinst is not needed; hicolor-icon-theme
   has a trigger on /usr/share/icons/hicolor anyway. And
   liblicense-_dev_ doesn't install anything into this directory.
   (Actually none of the binary packages does.)
  
I suggest to remove this line from the postinst; but maybe I'm
missing something ...

Cheers,
gregor

-- 
 .''`.   http://info.comodo.priv.at/ -- GPG Key IDs: 0x8649AA06, 0x00F3CFE4
 : :' :  Debian GNU/Linux user, admin,  developer - http://www.debian.org/
 `. `'   Member of VIBE!AT  SPI, fellow of Free Software Foundation Europe
   `-NP: Tracy Chapman: Open Arms


signature.asc
Description: Digital signature


Bug#566562: liblicense-dev: patch provided: needed Depends: hicolor-icon-theme

2010-02-18 Thread Dennis Sheil
On Thu, Feb 18, 2010 at 5:47 PM, gregor herrmann gre...@debian.org wrote:
 On Sat, 30 Jan 2010 12:28:27 +, Dennis Sheil wrote:

 The post-installation script of liblicense-dev touches
 /usr/share/icons/hicolor so as to regenerate GNOME's
 icon cache.  The hicolor-icon-theme package is not a
 dependency though so liblicense-dev fails when it is
 not aready installed.

 Patch attached.

 1) I don't see how adding hicolor-icon-theme to _Build_-Depends helps
   at installation time.
 2) IMO the touch'ing in postinst is not needed; hicolor-icon-theme
   has a trigger on /usr/share/icons/hicolor anyway. And
   liblicense-_dev_ doesn't install anything into this directory.
   (Actually none of the binary packages does.)

 I suggest to remove this line from the postinst; but maybe I'm
 missing something ...

Since the /usr/share/icons/hicolor directory is touched,
hicolor-icon-theme needs to be installed for the touch to happen
successfully.  Otherwise the touch fails and thus the installation
fails.  This was the main problem, the installation failed.

Of course, an if statement to check this directory exists before
attempting to touch it will solve the problem as well.  Or if a
trigger exists like you say, the touch can be removed.  The problem is
the postinst touch is touching a directory that may not exist on the
system, thus causing the installation to fail.  If a trigger exists,
and this line which breaks the installation is unnecessary and can be
removed, so much the better.

-- Dennis



-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#566562: liblicense-dev: patch provided: needed Depends: hicolor-icon-theme

2010-02-02 Thread Asheesh Laroia

On Sat, 30 Jan 2010, Dennis Sheil wrote:


Package: liblicense-dev
Severity: normal


The post-installation script of liblicense-dev touches
/usr/share/icons/hicolor so as to regenerate GNOME's
icon cache.  The hicolor-icon-theme package is not a
dependency though so liblicense-dev fails when it is
not aready installed.


Hi! You rule. Fixing shortly. Ping me in 3 days if it's not uploaded by 
then.


-- Asheesh.

--
Option Paralysis:
The tendency, when given unlimited choices, to make none.
-- Douglas Coupland, Generation X: Tales for an Accelerated
   Culture



--
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#566562: liblicense-dev: patch provided: needed Depends: hicolor-icon-theme

2010-01-30 Thread Dennis Sheil
Package: liblicense-dev
Severity: normal


The post-installation script of liblicense-dev touches 
/usr/share/icons/hicolor so as to regenerate GNOME's
icon cache.  The hicolor-icon-theme package is not a
dependency though so liblicense-dev fails when it is
not aready installed.

Patch attached.


-- System Information:
Debian Release: squeeze/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: i386 (i686)

Kernel: Linux 2.6.31-17-generic (SMP w/2 CPU cores)
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968)
Shell: /bin/sh linked to /bin/dash

Versions of packages liblicense-dev depends on:
ii  liblicense3   0.8-2.dws1 Stores and retrieves license infor

liblicense-dev recommends no packages.

liblicense-dev suggests no packages.

-- no debconf information
diff -u liblicense-0.8/debian/control liblicense-0.8/debian/control
--- liblicense-0.8/debian/control
+++ liblicense-0.8/debian/control
@@ -3,7 +3,7 @@
 Priority: extra
 Maintainer: Asheesh Laroia ashe...@asheesh.org
 DM-Upload-Allowed: yes
-Build-Depends: debhelper (= 5.0.38), libcurl4-gnutls-dev, libvorbis-dev, libtag1-dev, libflac-dev, libraptor-dev, libexempi-dev, libid3-3.8.3-dev, libgsf-1-dev, python-all-dev (= 2.3.5-11), python-central (= 0.5.6), libglib2.0-dev, xsltproc, docbook-xsl, libtool, libexpat-dev
+Build-Depends: debhelper (= 5.0.38), libcurl4-gnutls-dev, libvorbis-dev, libtag1-dev, libflac-dev, libraptor-dev, libexempi-dev, libid3-3.8.3-dev, libgsf-1-dev, python-all-dev (= 2.3.5-11), python-central (= 0.5.6), libglib2.0-dev, xsltproc, docbook-xsl, libtool, libexpat-dev, hicolor-icon-theme
 Standards-Version: 3.7.2
 Section: libs