Please merge maximus (0.4.14-1) from Debian Testing

[Vikram Dhillon]

Binary package hint: maximus

Changelogs:

Latest Ubuntu Changelog:
maximus (0.4.14-0ubuntu4) lucid; urgency=low

  * data/maximus-autostart.desktop: Move autostart phase from desktop to
    applications, so that it doesn't block the startup of other programs, and
    the scheduler can sort out which program to give the CPU to. This compacts
    the CPU usage on startup.

 -- Martin Pitt <email address hidden> Fri, 12 Feb 2010 12:07:30 +0100

Latest Debian Changelog:
maximus (0.4.14-1) unstable; urgency=low

   * [454633a] Imported Upstream version 0.4.14
 -- Guido Günther <email address hidden> Thu, 14 Jan 2010 15:24:46 +0100

Since this is a merge, I'll provide a patch in a minute. The package also build fine in a lucid-chroot:

dpkg-deb: building package `maximus' in `../maximus_0.4.14-1_i386.deb'.
 dpkg-genchanges >../maximus_0.4.14-1_i386.changes
dpkg-genchanges: including full source code in upload
dpkg-buildpackage: full upload (original source is included)
I: Copying back the cached apt archive contents
I: unmounting dev/pts filesystem
I: unmounting proc filesystem
I: cleaning the build env
I: removing directory /var/cache/pbuilder/build//7397 and its subdirectories
I: Current time: Sat Feb 20 17:57:05 EST 2010
I: pbuilder-time-stamp: 1266706625

[Vikram Dhillon (dhillon-v10)]

Applies cleanly on -p0:

$ patch -p0 < debian_maximus.diff
patching file data/maximus-autostart.desktop
patching file debian/changelog
patching file debian/control
patching file debian/maximus.install
patching file debian/maximus.postinst
patching file debian/maximus.preinst
patching file debian/source_maximus.py
$

[Anzenketh (anzenketh)]

Thanks for reporting this bug and any supporting documentation. Since this bug has enough information provided for a developer to begin work, I'm going to mark it as confirmed and let them handle it from here. Thanks for taking the time to make Ubuntu better!

[Anzenketh (anzenketh)]

Changed Back to New per policy.

[Daniel Holbach (dholbach)]

Unfortunately the patch does not apply any more?

daniel@miyazaki:~/maximus-0.4.14$ patch -p1 < ../debian_maximus.diff
patching file maximus-autostart.desktop
Hunk #1 FAILED at 8.
1 out of 1 hunk FAILED -- saving rejects to file maximus-autostart.desktop.rej
patching file changelog
Hunk #1 FAILED at 1.
Hunk #2 FAILED at 14.
Hunk #3 FAILED at 84.
3 out of 3 hunks FAILED -- saving rejects to file changelog.rej
patching file control
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file control.rej
patching file maximus.install
patching file maximus.postinst
patching file maximus.preinst
patching file source_maximus.py
daniel@miyazaki:~/maximus-0.4.14$

[Vikram Dhillon (dhillon-v10)]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 03/01/2010 07:12 AM, Daniel Holbach wrote:
> Unfortunately the patch does not apply any more?
>
> daniel@miyazaki:~/maximus-0.4.14$ patch -p1 < ../debian_maximus.diff
> patching file maximus-autostart.desktop
> Hunk #1 FAILED at 8.
> 1 out of 1 hunk FAILED -- saving rejects to file maximus-autostart.desktop.rej
> patching file changelog
> Hunk #1 FAILED at 1.
> Hunk #2 FAILED at 14.
> Hunk #3 FAILED at 84.
> 3 out of 3 hunks FAILED -- saving rejects to file changelog.rej
> patching file control
> Hunk #1 FAILED at 1.
> 1 out of 1 hunk FAILED -- saving rejects to file control.rej
> patching file maximus.install
> patching file maximus.postinst
> patching file maximus.preinst
> patching file source_maximus.py
> daniel@miyazaki:~/maximus-0.4.14$
>

Sorry Daniel, I am unable to understand why it doesn't work, I am
attaching a little log file that shows the process I went through and it
works fine. Also in Debian, I just noticed a new patch [1] that someone
made and it hasn't been included into this package so this diff I am
making would be kind of useless.

[1] http://packages.qa.debian.org/m/maximus.html

- --
Regards,
Vikram Dhillon

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iEYEARECAAYFAkuTBdIACgkQcoBavQdRHcyKpQCfTGZAhrx3w3BWPY7yCvQ/mFIP
cxwAoMw/4+qHMIWhcH5piUle/Gybf/bw
=YQP0
-----END PGP SIGNATURE-----

[Daniel Holbach (dholbach)]

Regarding the patch: sorry, my mistake.

A few things:
 - The merged version should be 04.14-1ubuntu1.
 - I think you have the changelog entries also a bit wrong. "remaining changes" should list all the changes that we have on top of the debian version.

[Daniel Holbach (dholbach)]

(I don't see how your patch does these things for example:

+ - Bumped up the Standards version to 3.8.4
+ - Added VCS information
+ - Updated package description
+ - Removed x-window manager from Provides field and moved it to Depends

but there's a number of other things it does.

[Vikram Dhillon (dhillon-v10)]

Debian has a patch in BTS that hasn't yet been incorporated into the package, so should I go on without the patch (we are already on feature freeze) or stop?

[Benjamin Drung (bdrung)]

unsubscribed ubuntu-sponsors, please resubscribe once you have addressed Daniels reclamations.

You can incorporate the patch from the BTS if you want (but you don't have to) - it's just adding a Homepage entry to debian/control. The feature freeze will not prevent the merge, because you do not merge a new upstream release.