bugsaction

Differences between revisions 46 and 47
Revision 46 as of 2008-08-15 16:39:11
Size: 13400
Editor: put92-5-82-243-237-71
Comment:
Revision 47 as of 2008-08-15 18:09:32
Size: 13442
Editor: put92-5-82-243-237-71
Comment:
Deletions are marked like this. Additions are marked like this.
Line 82: Line 82:
   * debian/lm-sensors.fancontrol.init:# Default-Stop: 0 1 6 -> 1    * debian/lm-sensors.fancontrol.init:# Default-Stop: 0 1 6 -> (Not changed as fancontrol is not impacted)

Actions made to correct bugs

This page has been written to have a trace about the correction of the packages I have done yet.


Remove multiuser for nvidia-kernel-common. Bug #254264

  • grep -r multiuser *
    • old in debian/rules:

dh_installinit --name=nvidia-kernel -- multiuser
  • I replaced with:

dh_installinit --name=nvidia-kernel -- start 20 2 3 4 5 . stop 20 1 .
  • grep -r Default-Stop *

-> Nothing

  • See also the change in debian/control (cf below). Remove depends on sysv-rc (>= 2.86.ds1-14.1ubuntu2)

  • Assure that a debian/*.postinst exists to remove kill links in 0 et 6 status
  • dch -i
  • debuild -S, debdiff, pbuilder
  • unassigned, state -> confirmed, suscribed u-m-s (rmadison <package_name> -> nothing shown, so, it's under main section)

  • have to send to upstream

Remove multiuser for nis. Bug #254262

  • grep -r multiuser *
    • old in debian/postinst:

update-rc.d nis multiuser 18 >/dev/null 2>&1
  • I replaced with:

update-rc.d nis start 18 2 3 4 5 . stop 18 1 . >/dev/null 2>&1
  • grep -r Default-Stop *

 * debian/rc.nis:# Default-Stop:                0 1 6          ->   1
 * ypbind-mt-1.20.1/etc/ypbind.init:# Default-Stop: 0 1 2 6   -> do not change it as it is on the source (and we do not want a patch for that) and not for debian/ubuntu
  • See also the change in debian/control (cf below). Remove depends on sysv-rc (>= 2.86.ds1-14.1ubuntu2)

  • Assure that a debian/*.postinst exists to remove kill links in 0 et 6 status
  • dch -i
  • debuild -S, debdiff, pbuilder

-> FTBFS

  • unassigned, state -> confirmed, suscribed u-m-s (rmadison <package_name> -> nothing shown, so, it's under main section)

  • have to send to upstream

Remove multiuser for lm-sensors. Bug #254259

  • grep -r multiuser *
    • old in debian/rules:

dh_installinit -psensord -- multiuser
  • I replaced with:

dh_installinit -psensord -- start 20 2 3 4 5 . stop 20 1 .
  • grep -r Default-Stop *

   * debian/lm-sensors.fancontrol.init:# Default-Stop:      0 1 6     -> (Not changed as fancontrol is not impacted)
   * debian/sensord.init:# Default-Stop:      0 1 6                   -> 1
   * debian/lm-sensors.lm-sensors.init:# Default-Stop:                (Not changed as even Default-Start as "S" as a value instead of 2 3 4 5…)
   * prog/init/lm_sensors.init.suse:# Default-Stop:                     0 1 2 6   (Not changed as in the source and specific to suse)
  • See also the change in debian/control (cf below). Remove depends on sysv-rc (>= 2.86.ds1-14.1ubuntu2) on one of the binary package

  • Assure that a debian/*.postinst exists to remove kill links in 0 et 6 status
  • dch -i
  • debuild -S, debdiff, pbuilder
  • unassigned, state -> confirmed, suscribed u-m-s (rmadison <package_name> -> nothing shown, so, it's under main section)

  • have to send to upstream

Remove multiuser for keepalived. Bug #254257

  • grep -r multiuser *
    • old in debian/rules:

DEB_UPDATE_RCD_PARAMS := multiuser
  • This package use cdbs. I replaced with:

DEB_UPDATE_RCD_PARAMS := start 20 2 3 4 5 . stop 20 1 .
  • grep -r Default-Stop *
    • old in debian/init.d:

# Default-Stop:      0 1 6
  • replaced with:

# Default-Stop:      1
  • See also the change in debian/control (cf below). Remove depends on sysv-rc (>= 2.86.ds1-14.1ubuntu2)

  • Assure that a debian/*.postinst exists to remove kill links in 0 et 6 status
  • dch -i
  • debuild -S, debdiff, pbuilder
  • unassigned, state -> confirmed, suscribed u-m-s (rmadison <package_name> -> nothing shown, so, it's under main section)

  • have to send to upstream

Remove multiuser for dictd. Bug #254252

  • grep -r multiuser *
    • old in debian/rules:

dh_installinit -- multiuser
  • replaced with:

dh_installinit -- start 20 2 3 4 5 . stop 20 1 .
  • grep -r Default-Stop *
    • old in debian/dictd.init:

# Default-Stop:      0 1 6
  • replaced with:

# Default-Stop:      1
  • Since the maintainer is not Ubuntu Core Developers <ubuntu-devel-discuss@lists.ubuntu.com> (even if the previous version was xxxubuntu1), change it as debuild -S shout about it.

  • See also the change in debian/control (cf below). No change seems necessary.
  • Assure that a debian/*.postinst exists to remove kill links in 0 et 6 status
  • dch -i
  • debuild -S, debdiff, pbuilder
  • unassigned, state -> confirmed, suscribed u-m-s (rmadison <package_name> -> nothing shown, so, it's under main section)

  • have to send to upstream

Remove multiuser for acpid. Bug #254249

  • grep -r multiuser *
    • old in debian/rules:

dh_installinit -- multiuser 10 21

dh_installinit -- start 10 2 3 4 5 . stop 21 1 .
  • grep -r Default-Stop *
    • old in debian/acpid.init.d:

# Default-Stop:      0 1 6
  • replaced with:

# Default-Stop:      1
  • further information from james_w show that I have to remove sysvinit special dependency has multiuser forced that (just use debian old rules from patches.ubuntu.com or packages.debian.org) apart from lsb-base (>= 3.2-14) which is used for status action in init script

  • Assure that a debian/*.postinst exists to remove kill links in 0 et 6 status
  • dch -i

  * Replace "multiuser" deprecated init mode in debian/rules with manual
    arguments. Adapt the corresponding change in LSB init script header and 
    runtime dependency in debian/control (LP: #254249)
  • debuild -S, debdiff, pbuilder
  • unassigned, state -> confirmed, suscribed u-m-s (rmadison <package_name> -> nothing shown, so, it's under main section)

  • have to send to upstream, cleaning all modifications that are not Ubuntu-dependent (example: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=494951)

Fix NBS for icecpp

Fix NBS for icedtea-java7-jre, bug #203636

-- intrepid/universe xxx deps on icedtea-java7-jre:
azureus
freecol
project-x
-- intrepid/multiverse xxx deps on icedtea-java7-jre:
sun-javadb-client
sun-javadb-core
sun-javadb-demo
  • for each binary package, to find the source package:
    • apt-cache showsrc <package> | grep Package

Package: azureus
Package: freecol
Package: project-x
Package: sun-javadb
  • None of those are NBS
    • download all of them and look into debian/control

azureus: 
Depends: openjdk-6-jre | icedtea-java7-jre | sun-java6-jre | sun-java5-jre,
 libcommons-cli-java, liblog4j1.2-java, libseda-java, libswt3.2-gtk-java

freecol:
Depends: openjdk-6-jre | icedtea-java7-jre | sun-java6-jre | j2re1.6 , 
 libhiglayout-java, libwoodstox-java, , java-wrappers (>= 0.1.4)

project-x:
Depends:${misc:Depends}, libcommons-net-java, openjdk-6-jre | icedtea-java7-jre | sun-java6-jre | sun-java5-jre

sun-javadb:
Depends: sun-javadb-common, openjdk-6-jre | sun-java5-jre | sun-java6-jre | icedtea-java7-jre | java2-runtime
  • Removed icedtea-java7-jre from dependencies (depends and build-depends) of each package and change maintainer field for non Ubuntu specific package
  • dch -i for each of them with xxxubuntu1 (if not specifically packaged for Ubuntu: not Build1 as there is a change in debian/control)
  • Build source package as runtime one
  • ERRORS in building azureus (even the original source package), but not found in FTBFS from harvest. A large merge is waiting in MoM for it

example:

   ./org/gudy/azureus2/ui/console/multiuser/commands/UserCommand.java:30: package org.apache.commons.cli does not exist 

Bug #61039 (no desktop entry) and #255086 (FTBFS)


File: sgt-puzzles_7983-1ubuntu1


export QUILT_PATCHES=debian/patches
quilt push 202_online-help.diff
quilt push 203_follow-fd.o-clipboards-spec.diff
quilt push 304_combine-binaries.diff (but error has weren't is the series file)
quilt new 306_fix-ftbfs-with-fortify-source-2.diff
quilt add gtk.c
vim gtk.c
quilt refresh
quilt pop -a
  • I think that unstead searching for the patches including this files, I could use push -a

  • echo "306_fix-ftbfs-with-fortify-source-2.diff" >> debian/patches/series

  • Changed original maintainer (see below)
  • dch -i and add "add 306_fix-ftbfs-with-fortify-source-2.diff to fix FTBFS in gtk.c (LP: #255086)"

  • Created and validated the 27 desktop files (one for each game) in debian/desktop/ (created directory)
  • Create pixmap icons

for source in `ls icons/*48d8*`
do
dest=$(basename $source | sed -e 's/\([a-z][A-Z]*\)-48d8.png/\1/')
convert $source debian/desktop/${dest}.xpm
done

(and rename the 3 debian renamed files to avoid conflicts)

  • Added to debian/rules, after dh_installmenu in binary-arch target

        mkdir -p -m755  debian/sgt-puzzles/usr/share/applications
        install -m644 debian/desktop/*.desktop debian/sgt-puzzles/usr/share/applications
        mkdir -p -m755  debian/sgt-puzzles/usr/share/pixmaps/
        install -m644 debian/desktop/*.xpm debian/sgt-puzzles/usr/share/pixmaps/
  • run dch -a and add :

  * add desktop files and xpm files (on for each game) to the menu in
    debian/dekstop (LP: #61039)
  • build it and generate the debdiff file attached to the bugs and add intrepid origin-ubuntu ubuntu-patch as a tag (I cannot put any milestone for the bug? Intrepid alpha 4 milestone would have been great), place them back to "confirmed" status and assigned to nobody, subscribe u-u-s

  • Send a mail to upstream

'''#245633: wrong text for a .desktop file''' https://bugs.launchpad.net/bzr-gtk/+bug/245633, add also some french translation to it and remove deprecated/wrong key in the .desktop


File: bzr-gtk-0.94.0-1ubuntu1 PATCH COMMITED TO UPSTREAM with rev 0.95.0 https://bugs.launchpad.net/bzr-gtk/+bug/245633/comments/17


  • Place the bug status to "in progress" and assigned to me
  • Get the source and had a look at it.

As the file to correct is at the root of the source, I had to patch the package.

  • The original maintainer was a Debian one, so I changed the debian/control file:

Maintainer: Ubuntu MOTU Developers <ubuntu-motu@lists.ubuntu.com>
XSBC-Original-Maintainer: ...
  • used dpatch (as the original maintainer use it) and runned dpatch-edit-patch 04-change-menu-entry.dpatch

    • - vim the right file and change the content of it. Then exit 0 - vim 04-change-menu-entry.dpatch to change the first lines (DP field to give a description of the patch)

(if multiple lines add ## DP: at the begining of each line)

  • - add 04-change-menu-entry.dpatch to debian/patches/00list

  • run dch -i to update the debian/changelog file and added (LP: #245633)

bzr-gtk (0.94.0-1ubuntu1) intrepid; urgency=low

  * add 04-change-menu-entry.dpatch to fix name, add french translation and
    remove some deprecated keys in olive-gtk.desktop (LP: #245633)

 -- Didier Roche <didrocks@...>  Fri, 01 Aug 2008 15:14:06 +0200
  • rebuild the source package with debuild -S (for new diff.gz and .dsc file)

  • Finally: debdiff bzr-gtk_0.94.0-1.dsc bzr-gtk_0.94.0-1ubuntu1.dsc > debdiff

  • build the package for a try: sudo pbuilder build bzr-gtk_0.94.0-1ubuntu1.dsc

    • - the .deb is in /var/cache/pbuilder/result/ - install in a VM

  • run desktop-file-validate olive-gtk.desktop to validate the desktop file

  • test the debdiff by patch -p1 < debdiff

  • Attach the debdiff to the bug report, place it back to "confirmed" status and assigned to nobody, subscribe u-u-s
  • Added tags intrepid origin-ubuntu ubuntu-patch (cf https://wiki.ubuntu.com/Debian/Usertagging)

  • Send a mail to upstream


CategoryMOTULog

DidierRoche/MOTU/bugsaction (last edited 2009-03-23 11:59:52 by 91)