Merge with version 0.0.12-24 from Debian unstable

Bug #156712 reported by Andrea Gasparini
4
Affects Status Importance Assigned to Milestone
yaird (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

Binary package hint: yaird

Merge yaird 0.0.12-24 from Debian unstable.
Ubuntu component: Universe
Debian component: Main

Andrea Gasparini (gaspa)
Changed in yaird:
assignee: nobody → gaspa
status: New → In Progress
Revision history for this message
Andrea Gasparini (gaspa) wrote :
Andrea Gasparini (gaspa)
Changed in yaird:
status: In Progress → Confirmed
Andrea Gasparini (gaspa)
Changed in yaird:
assignee: gaspa → nobody
Revision history for this message
Andrea Veri (av) wrote :

1) why do you still report the bug number (LP: #129828)? it's not needed as far as you are merging this package and not fixing it. (fix was done by Luca already in 0.0.12-22ubuntu1 revision)
2) I see not-documented changes in debian/docs
3) tmp stuff should be dropped to prevent any problem while applying this debdiff.

Would you mind fixing those? Thanks.

Changed in yaird:
importance: Undecided → Wishlist
status: Confirmed → Incomplete
Revision history for this message
Andrea Veri (av) wrote :

Please re-subscribe u-u-s when you will provide a working debdiff. Good work.

Changed in yaird:
assignee: nobody → gaspa
Revision history for this message
Andrea Gasparini (gaspa) wrote :

Well, all done.

Do you think it's ok?

Thanks.

Andrea Gasparini (gaspa)
Changed in yaird:
assignee: gaspa → nobody
status: Incomplete → Confirmed
Changed in yaird:
assignee: nobody → norsetto
status: Confirmed → In Progress
Revision history for this message
Cesare Tirabassi (norsetto) wrote :

This change: "docs mentions html files that now are xml" is not a previous ubuntu change and should therefore be reported as a different change.
In any case, why did you make such a change?
The xml are used during build to make the html which is then installed correctly, therefore please don't change it or you will introduce a bug.

Concerning the previous ubuntu change, can you check if it is still necessary?
Looking at the original bug report, and the change description for 0.0.12-24, I have the impression that this might not be needed anymore.

Thanks.

Changed in yaird:
assignee: norsetto → gaspa
status: In Progress → Incomplete
description: updated
Revision history for this message
Cesare Tirabassi (norsetto) wrote :

I would like to add that I changed your bug description (was talking about wesnoth) and the title. It seems you have been working on a previous debian version, as most probably 0.0.12-24 was not available when you first prepared your patch.

Revision history for this message
Andrea Gasparini (gaspa) wrote :

Yes, i start the merge when was available versione 0.0.12-23.
So, I'll regain the sources and merging, asap.

For the xml issue, I'll take a look, perhaps I misunderstand the debian/rules.
Thanks.

Revision history for this message
Cesare Tirabassi (norsetto) wrote :

You shouldn't look at debian/rules. If you check your build log, you should see something like:

make[2]: Entering directory `/tmp/buildd/yaird-0.0.12/doc'
/usr/bin/xmlto xhtml-nochunks yaird.xml

This is part of the upstream makefile, simply entering the doc directory and executing the xmlto command, which, in this case, convert the DocBook XML document (yaird.xml) to the HTML file (yaird.html).
This latter is the one which is (correctly) installed by debian/rules.

You were perhaps confused by the fact that this is an "on-the-fly" generation.

Revision history for this message
Andrea Gasparini (gaspa) wrote :

Ok, i took the last version for the merge.

About the patch, it seems it's still needed, this is the output in Hardy with installed the debian version 0.0.12-24:

gaspa@cthulhu:/mnt/sorgenti/ubuntu/yaird$ sudo dpkg-reconfigure linux-image-`uname -r`
Running depmod.
mkinitrd.yaird: opzione non valida -- u
Terminating...
Failed to create initrd image.

Perhaps it would be better patch kernel-package (or who for him) in order to make scripts indipendent from the ramdisk creator.
But this is available and IMHO there's no reasons for not patching it.

What's your opinion, Cesare? you're more experienced than me, so, i'll listen surely your suggestions.
(anyway i'm going on to remake the right diff...)

Revision history for this message
Cesare Tirabassi (norsetto) wrote :

We better patch it then, because I have the impression that there is no official support for yaird (at least this https://bugs.edge.launchpad.net/ubuntu/+source/yaird/+bug/129828/comments/1 seems to imply so).

Revision history for this message
Andrea Gasparini (gaspa) wrote :

ok. let's do it. ;)

Cesare, take a look if now I done all and well... thanks. ;)

Revision history for this message
Cesare Tirabassi (norsetto) wrote :

Looks good to me. Have you checked that it builds and installs correctly in an hardy environment?
Please attach a build log and installation report.

Revision history for this message
Andrea Gasparini (gaspa) wrote :

The build log is attached, here's the install through dpkg:

# dpkg -i yaird_0.0.12-24ubuntu1_i386.deb
(Reading database ... 26066 files and directories currently installed.)
Preparing to replace yaird 0.0.12-23ubuntu1 (using yaird_0.0.12-24ubuntu1_i386.deb) ...
Unpacking replacement yaird ...
Setting up yaird (0.0.12-24ubuntu1) ...
Installing new version of config file /etc/yaird/Templates.cfg ...

thanks.

Revision history for this message
Cesare Tirabassi (norsetto) wrote :

Uploaded, thanks for your contribution.

Changed in yaird:
assignee: gaspa → nobody
status: Incomplete → Fix Committed
Revision history for this message
Cesare Tirabassi (norsetto) wrote :

yaird (0.0.12-24ubuntu1) hardy; urgency=low

  * Merge from Debian unstable. Remaining Ubuntu changes:
        - debian/control:
            + changing maintainter to "Ubuntu MOTU Developers"
        - debian/mkinitrd.yaird
            + update-initramfs compatibility

yaird (0.0.12-24) unstable; urgency=medium

  * Add patch 1022 to fix passing dashed options to run_init. Closes:
    bug#416927, thanks to Michael Biebl and Roderick Schupp.
  * Raising to urgency=medium as this may have been the cause of various
    other problems passing dashed boot options to userspace tools.

yaird (0.0.12-23) unstable; urgency=low

  * Add Vcs-Svn and Vcs-Browser fields to debian/control.
  * Move Homepage to own field (from pseudo-field in long description).
  * Add patch 1021 to support new-style IEEE1394 (Firewire) driver.
  * Rewrite long description and add README.Debian, emphasizing that
    yaird produces highly optimized ramdisks useful for stable systems,
    and suggesting initramfs-tools for flexible ramdisk needs. Closes:
    bug#444253, #346134, #392072.
  * Update debian/copyright:
    + Rewrite to conform to proposed new format at
      http://wiki.debian.org/Proposals/CopyrightFormat .
    + Drop unneeded disclaimer.

 -- Andrea Gasparini <email address hidden> Thu, 25 Oct 2007 20:20:31 +0200

Changed in yaird:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.