Bug#640493: [Fwd: Re: Bug#640493: postgresql-common: logrotate 3.8.0 breaks postgresql log rotation.]
Am Donnerstag, den 22.12.2011, 10:10 +0100 schrieb Martin Pitt: Jens Lody [2011-12-22 7:08 +0100]: But I still do not understand, why it can not be done this way. Please see http://www.debian.org/doc/debian-policy/ap-pkg-conffiles.html#sE.1 Thanks for clarifying this. Jens -- Jens Lody Code::Blocks core developer http://www.codeblocks.org/ (Un-)official debian-repository for Code::Blocks: http://apt.jenslody.de/ |\ _,,,---,,_ /,`.-'`'-. ;-;;,_ |,4- ) )-,_..;\ ( `'-' '---''(_/--' `-'\_) signature.asc Description: This is a digitally signed message part
Bug#640493: [Fwd: Re: Bug#640493: postgresql-common: logrotate 3.8.0 breaks postgresql log rotation.]
Hello Martin,
Am Dienstag, den 20.12.2011, 18:14 +0100 schrieb Martin Pitt:
Hello Jens,
Jens Lody [2011-11-21 10:13 +0100]:
diff -Nru postgresql-common-125.orig/debian/postgresql-common.postinst
postgresql-common-125/debian/postgresql-common.postinst
--- postgresql-common-125.orig/debian/postgresql-common.postinst
2011-08-08 10:21:40.0 +0200
+++ postgresql-common-125/debian/postgresql-common.postinst 2011-11-21
08:48:42.0 +0100
@@ -6,6 +6,24 @@
SSL_ROOT=/etc/postgresql-common/root.crt
+fix_logrotate()
+{
+# logrotate 3.8+ requires an additional su command which is not
+# backportable, see #640493
+lrv=`dpkg -s logrotate 2 /dev/null | grep ^Version | cut -f2- -d' '`
+if [ x$lrv = x ] ; then \
+lrv=0; \
+fi
+
+sed -i '/su root root/d' /etc/logrotate.d/postgresql-common; \
+
+if dpkg --compare-versions $lrv ge 3.8; then \
+ echo Fixing logrotate configuration for logrotate = 3.8...; \
+ sed -i '/^}/ i\ su root root'
/etc/logrotate.d/postgresql-common; \
That's precisely what we cannot do, as
/etc/logrotate.d/postgresql-common is a dpkg managed conffile.
But I guess this is all moot now that logrotate 3.8 is in testing as
well.
Martin
I know, that it works now.
But I still do not understand, why it can not be done this way.
I also added a trigger on /usr/sbin/logrotate, so that the postinst
script will be called (with the paramezer triggered), if dpkg changes
this file.
I thought that this is exactly (one of) the purpose(s) triggers have
been implemented for.
Or do I misunderstand something ?
Jens
--
Jens Lody
Code::Blocks core developer
http://www.codeblocks.org/
(Un-)official debian-repository for Code::Blocks:
http://apt.jenslody.de/
|\ _,,,---,,_
/,`.-'`'-. ;-;;,_
|,4- ) )-,_..;\ ( `'-'
'---''(_/--' `-'\_)
signature.asc
Description: This is a digitally signed message part
Bug#640493: [Fwd: Re: Bug#640493: postgresql-common: logrotate 3.8.0 breaks postgresql log rotation.]
Hi Martin,
I did not here anything from you, so I created a patch to make clear
what I meant.
Weitergeleitete Nachricht
Von: Jens Lody deb...@jenslody.de
An: Martin Pitt mp...@debian.org
Kopie: 640...@bugs.debian.org
Betreff: Re: Bug#640493: postgresql-common: logrotate 3.8.0 breaks
postgresql log rotation.
Datum: Tue, 25 Oct 2011 15:09:32 +0200
Am Dienstag, den 25.10.2011, 14:53 +0200 schrieb Martin Pitt:
Jens Lody [2011-10-25 13:44 +0200]:
Wouldn't it be possible to add the line in the postinst-step ?
No, you can't change dependencies in maintainer scripts. The point
is
to stop apt from trying to install it in the first place.
This should work for all versions including the backports.
The current solution should already work for backports. What is
broken
right now?
It's impossible to install logrotate 3.8 (from experimental) together
with the actual postgresql-common from testing.
Thanks,
Martin
I know that dependencies are hardcoded at compile-time (or more
exactly during package-build).
What I meant is to add the su root root-line to the logrotate-conf
file for p-common dynamically in the postinst-script depending on the
version of logrotate.
No need to change dependency, as p-common already depends on
logrotate.
I attach a patch that fixes the logrotate conf-file in postinst, either
at install time, or via a trigger if a new logrotate is installed.
It works here on my local system, but no backports are tested.
It might be needed to change from depends to pre-depends on logrotate,
so a probably newer version is surely isntalled before p-commons's
postinst is reached. I'm not sure if this would really be needed.
Seems to work without predepends.
Jens
--
Jens Lody
(Un-)official debian-repository for Code::Blocks:
http://apt.jenslody.de/
|\ _,,,---,,_
/,`.-'`' -. ;-;;,_
|,4- ) )-,_..;\ ( `'-'
'---''(_/--' `-'\_)
Best regards,
Jens
--
Jens Lody
(Un-)official debian-repository for Code::Blocks:
http://apt.jenslody.de/
|\ _,,,---,,_
/,`.-'`'-. ;-;;,_
|,4- ) )-,_..;\ ( `'-'
'---''(_/--' `-'\_)
diff -Nru postgresql-common-125.orig/debian/control postgresql-common-125/debian/control
--- postgresql-common-125.orig/debian/control 2011-10-12 15:18:27.0 +0200
+++ postgresql-common-125/debian/control 2011-11-21 08:49:36.0 +0100
@@ -16,10 +16,9 @@
debconf (= 0.5.00) | debconf-2.0,
lsb-base (= 3.0-3),
ssl-cert (= 1.0.11),
- logrotate ${logrotate:version},
+ logrotate,
${misc:Depends}
-Breaks: postgresql-9.1 ( 9.1.1-3~),
- ${common-breaks}
+Breaks: postgresql-9.1 ( 9.1.1-3~)
Description: PostgreSQL database-cluster manager
The postgresql-common package provides a structure under which
multiple versions of PostgreSQL may be installed and/or multiple
diff -Nru postgresql-common-125.orig/debian/postgresql-common.postinst postgresql-common-125/debian/postgresql-common.postinst
--- postgresql-common-125.orig/debian/postgresql-common.postinst 2011-08-08 10:21:40.0 +0200
+++ postgresql-common-125/debian/postgresql-common.postinst 2011-11-21 08:48:42.0 +0100
@@ -6,6 +6,24 @@
SSL_ROOT=/etc/postgresql-common/root.crt
+fix_logrotate()
+{
+# logrotate 3.8+ requires an additional su command which is not
+# backportable, see #640493
+lrv=`dpkg -s logrotate 2 /dev/null | grep ^Version | cut -f2- -d' '`
+if [ x$lrv = x ] ; then \
+lrv=0; \
+fi
+
+sed -i '/su root root/d' /etc/logrotate.d/postgresql-common; \
+
+if dpkg --compare-versions $lrv ge 3.8; then \
+ echo Fixing logrotate configuration for logrotate = 3.8...; \
+ sed -i '/^}/ i\ su root root' /etc/logrotate.d/postgresql-common; \
+fi
+}
+
+
if [ $1 = configure ]; then
# Make sure the administrative user exists
if ! getent passwd postgres /dev/null; then
@@ -74,13 +92,22 @@
/usr/share/postgresql-common/pg_checksystem || true
+fix_logrotate
+
if dpkg --compare-versions $2 lt 94; then
pg_updatedicts || true
fi
fi
if [ $1 = triggered ]; then
-pg_updatedicts || true
+for i in $2; do
+if [ $i == /usr/share/myspell/dicts ]; then
+pg_updatedicts || true
+fi
+if [ $2 == /usr/sbin/logrotate ]; then
+fix_logrotate
+fi
+done
fi
#DEBHELPER#
diff -Nru postgresql-common-125.orig/debian/postgresql-common.triggers postgresql-common-125/debian/postgresql-common.triggers
--- postgresql-common-125.orig/debian/postgresql-common.triggers 2010-08-05 17:51:34.0 +0200
+++ postgresql-common-125/debian/postgresql-common.triggers 2011-11-21 08:32:53.0 +0100
@@ -1 +1,2 @@
interest /usr/share/myspell/dicts
+interest /usr/sbin/logrotate
diff -Nru postgresql-common-125.orig/debian/rules postgresql-common-125/debian/rules
--- postgresql-common-125.orig/debian/rules 2011-09-25 21:25:10.0 +0200
Bug#640493: postgresql-common: logrotate 3.8.0 breaks postgresql log rotation.
Am Sonntag, den 25.09.2011, 17:34 +0200 schrieb Martin Pitt: Hello Martin, Hello Christoph, Christoph Berg [2011-09-18 19:37 +0200]: I think it is necessary, just because a logrotate version was (un)available at build time doesn't mean that this will be the case at run time. Actually, the problem exists both ways, so unless I forgot some detail, p-common will probably need a Breaks: logrotate ( 3.8) with the hack (or a Depends) and a Breaks: logrotate (= 3.8) without it. Right, that'd be really correct. Done in bzr now. Martin Wouldn't it be possible to add the line in the postinst-step ? Or remove the line, if logrotate-version is less than 3.8 (I guess removing is easier and less errorprone) ? This should work for all versions including the backports. -- Jens Lody (Un-)official debian-repository for Code::Blocks: http://apt.jenslody.de/ |\ _,,,---,,_ /,`.-'`'-. ;-;;,_ |,4- ) )-,_..;\ ( `'-' '---''(_/--' `-'\_) signature.asc Description: This is a digitally signed message part
Bug#640493: postgresql-common: logrotate 3.8.0 breaks postgresql log rotation.
Am Dienstag, den 25.10.2011, 14:53 +0200 schrieb Martin Pitt: Jens Lody [2011-10-25 13:44 +0200]: Wouldn't it be possible to add the line in the postinst-step ? No, you can't change dependencies in maintainer scripts. The point is to stop apt from trying to install it in the first place. This should work for all versions including the backports. The current solution should already work for backports. What is broken right now? Thanks, Martin I know that dependencies are hardcoded at compile-time (or more exactly during package-build). What I meant is to add the su root root-line to the logrotate-conf file for p-common dynamically in the postinst-script depending on the version of logrotate. No need to change dependency, as p-common already depends on logrotate. It might be needed to change from depends to pre-depends on logrotate, so a probably newer version is surely isntalled before p-commons's postinst is reached. I'm not sure if this would really be needed. Jens -- Jens Lody (Un-)official debian-repository for Code::Blocks: http://apt.jenslody.de/ |\ _,,,---,,_ /,`.-'`'-. ;-;;,_ |,4- ) )-,_..;\ ( `'-' '---''(_/--' `-'\_) signature.asc Description: This is a digitally signed message part
Bug#635019: Patch that fixes bug #635019
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I attach a patch, that fixes this issue. It happens because in kernel_version.mk spaces are used as seperators instead of tabs. It seems that older versions of make ignore this incorrect synatx, but make 3.82 is not as generous. As told, this patch fixes the issue here on debian unstable/experimental with kernel-package 12.036+nmu1 and linux-source-3.0.0 3.0.0-2. I did not search other parts of the makefile-includes for this issue, but it might exist there as well in parts I do not use in my build. - -- Jens Lody (Un-)official debian-repository for Code::Blocks: http://apt.jenslody.de/ |\ _,,,---,,_ /,`.-'`'-. ;-;;,_ |,4- ) )-,_..;\ ( `'-' '---''(_/--' `-'\_) -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAk5XNc0ACgkQyW5ifG7rJcklRgCgyh44LoVn7Gj03TWPDBKT4QV6 o5IAn15k457LV0/PZOQ1ysh316Tj6iUQ =s1Vu -END PGP SIGNATURE- diff -Nru /usr/share/kernel-package/ruleset/kernel_version.mk /usr/share/kernel-package.save/ruleset/kernel_version.mk --- /usr/share/kernel-package/ruleset/kernel_version.mk 2008-05-02 07:06:28.0 +0200 +++ /usr/share/kernel-package.save/ruleset/kernel_version.mk 2011-08-26 06:34:10.0 +0200 @@ -70,24 +70,24 @@ debian_conf_var: -@echo ARCH = $(ARCH) -@echo HOSTCC = $(HOSTCC) -@echo HOSTCFLAGS = $(HOSTCFLAGS) -@echo CROSS_COMPILE= $(CROSS_COMPILE) -@echo AS = $(AS) -@echo LD = $(LD) -@echo CC = $(CC) -@echo CPP = $(CPP) -@echo AR = $(AR) -@echo NM = $(NM) -@echo STRIP= $(STRIP) -@echo OBJCOPY = $(OBJCOPY) -@echo OBJDUMP = $(OBJDUMP) -@echo MAKE = $(MAKE) -@echo GENKSYMS = $(GENKSYMS) -@echo CFLAGS = $(CFLAGS) -@echo AFLAGS = $(AFLAGS) -@echo MODFLAGS = $(MODFLAGS) + @echo ARCH = $(ARCH) + @echo HOSTCC = $(HOSTCC) + @echo HOSTCFLAGS = $(HOSTCFLAGS) + @echo CROSS_COMPILE= $(CROSS_COMPILE) + @echo AS = $(AS) + @echo LD = $(LD) + @echo CC = $(CC) + @echo CPP = $(CPP) + @echo AR = $(AR) + @echo NM = $(NM) + @echo STRIP= $(STRIP) + @echo OBJCOPY = $(OBJCOPY) + @echo OBJDUMP = $(OBJDUMP) + @echo MAKE = $(MAKE) + @echo GENKSYMS = $(GENKSYMS) + @echo CFLAGS = $(CFLAGS) + @echo AFLAGS = $(AFLAGS) + @echo MODFLAGS = $(MODFLAGS) # arch-tag: ecfa9843-6306-470e-8ab9-2dfca1d40613 kernel_package_fix_635019.patch.sig Description: Binary data
Bug#618465: bug can be closed
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 The issue is fixed in 2.91.91-1, because gnome-shell now depends on libmozjs4d. It can be closed. Jens -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAk2EYoUACgkQyW5ifG7rJclsFgCfTPaSJxlHqs7MTfPfZH+BMurd 8h8AoLr78M6Wb74X163DUD23ATtJyWaL =mTz3 -END PGP SIGNATURE- -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#618465: gnome-shell not installable due to missing libmozjs3d
Package: gnome-shell Version: 2.91.90-1 Severity: grave Justification: renders package unusable gnome-shell depends on libmozjs3d, but it's no longer available. -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#585105: radiotray: doesn't start
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Am -10.01.-28163 20:59, schrieb Bin Zhang: It's a dependencies problem. Radiotray depends on python-gst (see http://radiotray.sourceforge.net/ , it also depends on python-lxml). Installing python-gst fixes this bug. You can close it. Thanks, Bin It's correct, that it is a dependency problem, but it not only depends on the packages listed on radiotray's sf-site you mentioned above. It also depends on python-xdg. Jens - -- Jens Lody Unofficial debian-repository for Code::Blocks: http://apt.jenslody.de/ |\ _,,,---,,_ /,`.-'`'-. ;-;;,_ |,4- ) )-,_..;\ ( `'-' '---''(_/--' `-'\_) -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.10 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAkwQlbEACgkQyW5ifG7rJcnW3QCffG9xbcKU8LvUeKCXvhz+mM+l H6UAoODJjTHA5niviW27DYFepkQu/ahh =7Z4A -END PGP SIGNATURE- -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#461410: xserver-xorg-core: BadAlloc after upgrade to 2:1.4.1~git20080105-2
Package: xserver-xorg-core Followup-For: Bug #461410 A similar error like in etch happens in sid too: The program 'SPS_Log' received an X Window System error. This probably reflects a bug in the program. The error was 'BadAlloc (insufficient resources for operation)'. (Details: serial 169 error_code 11 request_code 146 minor_code 5) The program is compiled with wxWidgets 2.8.7 from apt.wxwidgets.org . This happens also to other programs compiled against wxWidgets 2.8.7 . -- Package-specific info: Contents of /var/lib/x11/X.roster: xserver-xorg /var/lib/x11/X.md5sum does not exist. X server symlink status: lrwxrwxrwx 1 root root 13 21. Aug 11:53 /etc/X11/X - /usr/bin/Xorg -rwxr-xr-x 1 root root 1673404 6. Jan 23:36 /usr/bin/Xorg Contents of /var/lib/x11/xorg.conf.roster: xserver-xorg VGA-compatible devices on PCI bus: 01:00.0 VGA compatible controller: ATI Technologies Inc Radeon Mobility X1400 /etc/X11/xorg.conf does not match checksum in /var/lib/x11/xorg.conf.md5sum. Xorg X server configuration file status: -rw-r--r-- 1 root root 3689 2. Jan 12:15 /etc/X11/xorg.conf Contents of /etc/X11/xorg.conf: Section ServerLayout Identifier X.org Configured Screen 0 Screen0 0 0 InputDeviceTouchpad0 CorePointer InputDeviceMouse0 SendCoreEvents InputDeviceKeyboard0 CoreKeyboard EndSection Section Files RgbPath /etc/X11/rgb ModulePath /usr/lib/xorg/modules FontPath /usr/share/fonts/X11/misc FontPath /usr/share/fonts/X11/cyrillic FontPath /usr/share/fonts/X11/100dpi/:unscaled FontPath /usr/share/fonts/X11/75dpi/:unscaled FontPath /usr/share/fonts/X11/Type1 FontPath /usr/share/fonts/X11/100dpi FontPath /usr/share/fonts/X11/75dpi FontPath /var/lib/defoma/x-ttcidfont-conf.d/dirs/TrueType EndSection Section Module Load extmod Load dbe Load dri Load GLcore Load glx Load record Load xtrap EndSection Section InputDevice Identifier Keyboard0 Driver kbd EndSection Section InputDevice Identifier Mouse0 Driver mouse Option Protocol auto Option Device /dev/input/mice Option ZAxisMapping 4 5 6 7 EndSection Section InputDevice Driversynaptics IdentifierTouchpad0 OptionDevice/dev/psaux OptionProtocol auto-dev # # Following lines are optional thus commented in this example. # Please read the synaptics(5) manpage for a detailed explanation # of the various options' meaning as well as a full list of options # # Option LeftEdge 1700 # Option RightEdge 5300 # Option TopEdge 1700 # Option BottomEdge4200 # Option FingerLow 25 # Option FingerHigh30 # Option MaxTapTime180 # Option MaxTapMove220 # Option VertScrollDelta 100 # Option MinSpeed 0.06 # Option MaxSpeed 0.12 # Option AccelFactor 0.0010 Option SHMConfig on # Option Repeater /dev/ps2mouse EndSection Section Monitor Identifier Panel EndSection Section Monitor Identifier Analog Option RightOf Panel EndSection Section Device ### Available Driver options are:- ### Values: i: integer, f: float, bool: True/False, ### string: String, freq: f Hz/kHz/MHz ### [arg]: arg optional #Option NoAccel # [bool] #Option SWcursor # [bool] #Option pciBurst # [bool] #Option shadowfb # [bool] #Option probe_i2c # [bool] #Option ignoreconnector # [str] Identifier Card0 Driver radeonhd # Driver vesa Option monitor-VGA_1 Analog Option monitor-PANEL Panel Option RHDOutputOrderPanel # Option HPD swap VendorName ATI Technologies Inc BoardName Radeon Mobility X1400 BusID PCI:1:0:0 # Option NoRandr EndSection Section Screen Identifier Screen0 Device Card0 DefaultDepth24 # MonitorMonitor0 SubSection Display Virtual3840 1200 Viewport 0 0 Depth 1 EndSubSection SubSection Display Virtual3840 1200 Viewport 0 0 Depth 4 EndSubSection SubSection Display Virtual3840 1200 Viewport 0 0 Depth 8 EndSubSection
Bug#385443: gcc-4.0.dpatch is errornous, new patch
Package: bootsplash
Version: 3.3-4
Severity: normal
Tags: patch
The gcc-4.0.dpatch needed to make it possible to compile fbmngplay with
gcc-4.0 is errornous.
I have attached a working version of the patch to my mail.
Jens
---
Jens Lody
#! /bin/sh /usr/share/dpatch/dpatch-run
## gcc-4.0.dpatch by [EMAIL PROTECTED]
##
## All lines beginning with `## DP:' are a description of the patch.
## DP: No description.
@DPATCH@
diff -urNad bootsplash-3.1~/Utilities/mng.c bootsplash-3.1/Utilities/mng.c
--- bootsplash-3.3~/Utilities/mng.c 2006-02-05 20:42:22.0 -0500
+++ bootsplash-3.3/Utilities/mng.c 2007-01-20 18:09:57.0 +0100
@@ -228,7 +228,8 @@
a = a * mymng-alpha / 100;
switch (mymng-fbbpp) {
case 16:
- input = *((unsigned short *) background)++;
+ input = *((unsigned short *) background);
+ background += sizeof(*((unsigned short *) background));
br = (input mng-fbredo) (8 - mng-fbredl);
bg = (input mng-fbgreeno) (8 -
@@ -279,7 +280,8 @@
fbgreeno) | (b mng-
fbblueo);
- *((unsigned short *) dest)++ = output;
+ *((unsigned short *) dest) = output;
+ dest += sizeof(*((unsigned short *) dest));
break;
case 24:
*dest++ = b;
signature.asc
Description: OpenPGP digital signature
