Please update mplinuxman to 1.5 version

Bug #181372 reported by Marco Rodrigues
10
Affects Status Importance Assigned to Milestone
mplinuxman (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

Binary package hint: mplinuxman

21/12/06 Version 1.5
* Ported open/save/about dialogs to the new stock Gtk dialogs
* Files played with a external player are now added to the list of recent files.
* Added a launcher (.desktop file) for Mplinuxman
* Improved the GUI to be more GNOME HIG compliant
* Updated the Dutch translation

+ watch file needed

Related branches

Changed in mplinuxman:
importance: Undecided → Wishlist
Revision history for this message
Miguel Ruiz (mruiz) wrote :

I'm working on it.

Cheers!

Changed in mplinuxman:
assignee: nobody → mruiz
status: New → In Progress
Miguel Ruiz (mruiz)
Changed in mplinuxman:
assignee: mruiz → nobody
status: In Progress → Incomplete
Artur Rona (ari-tczew)
description: updated
Changed in mplinuxman (Ubuntu):
status: Incomplete → New
description: updated
summary: - New upstream version 1.5 - The watch file is broken
+ [karmic] Please update mplinuxman to 1.5 version
Artur Rona (ari-tczew)
summary: - [karmic] Please update mplinuxman to 1.5 version
+ Please update mplinuxman to 1.5 version
Revision history for this message
Sebastian Carneiro (scarneiro) wrote :

Hi,

  I don't know if anoyone is working on it. I made a package for version 1.5. Attached is the corresponding debdiff.

Would you please indicate me if this package is correct, and in case this is correct, what has to be done to upload this package?

Best regards.

tags: added: patch
Revision history for this message
Daniel Holbach (dholbach) wrote :

A few comments:
 - in debian/changelog please update 'natty' to 'oneiric' and please remove the empty line in there as well
 - it seems like no binaries are installed in the package - did you try to build and install the new package?

Changed in mplinuxman (Ubuntu):
status: New → Incomplete
Revision history for this message
Sebastian Carneiro (scarneiro) wrote :

Daniel,

    you are right about both comments.

I have made a mistake when modifying the makefile for this package. Even though I have built the package, i did not check what the package installed. I will have to be much more careful in the future about testing packages, to avoid this huge errors.

Let me make the necessary changes, and i will re-submit this patch.

Thanks for your time and patience with this newbie.... :)

Best regards,

Revision history for this message
Daniel Holbach (dholbach) wrote :

Don't worry. Thanks a lot for working on it!

Revision history for this message
Sebastian Carneiro (scarneiro) wrote :

Hi,

   I have made the abovementioned changes to the package, and yet i am having problems buildind the package in oneiric under gcc-4.6. This issue is related to (LP: #770855).

In the meantime, i have put together a PPA package for natty, that does build correctly. If anyone need to use it, i give you the ppa source:

 http://ppa.launchpad.net/scarneiro/ppa/ubuntu

When i can solve the issue i am having, i will upload the patch.

Revision history for this message
Sebastian Carneiro (scarneiro) wrote :

I have corrected the problem I was having in building the package in gcc-4.6. Attached is a debdiff of the updated package.

Please review theses changes and let me know anything that needs to be corrected for these to be ok. It is one of my first contributions and I need your comments in order to improve my packaging skills.

Best regards,

Sebastian.

Revision history for this message
Sebastian Carneiro (scarneiro) wrote :
Changed in mplinuxman (Ubuntu):
status: Incomplete → Confirmed
tags: added: upgrade-software-version
Revision history for this message
Ilya Barygin (randomaction) wrote :

Hello Sebastian,
There was one more upload of mplinuxman (1.4-0ubuntu3) during the Oneiric release cycle, so I merged your changes with it.
There was a couple of problems with debian/rules: it didn't call dh_shlibdeps (so the package didn't have necessary binary dependencies) and dh_strip (so the binaries weren't stripped). I solved both by switching to debhelper7-style debian/rules file.
I added some more patches to solve old problems with this package (such as using a bundled copy of a library).
Thank you for your work on this bug!

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package mplinuxman - 1.5-0ubuntu1

---------------
mplinuxman (1.5-0ubuntu1) precise; urgency=low

  [ Sebastian Carneiro ]
  * New upstream version (LP: #181372)

  * debian/patches:
    - Patches 01_gcc-4.0-fixes.dpatch and 02_makefile-install.dpatch removed
      as they already are applied in upstream.
    - Added 04_fix_old_dot_desktop_values.patch to remove deprecated values
      and update references to absolute directories, and non compliant
      values.
    - Added 05_upgrade_makefile.patch to modify original
      makefile, removing unnecesary targets, parameterizing installation
      directories, and changing parameters order to link successfully under
      gcc 4.6
    - Added 07_correct_po_files.patch to correct encoding values in file
      es.po.
    - Refreshed 05_upgrade_makefile.patch to correct wrong permission settings
      on non-executable files.
    - Added 06_makefile-catch-rm-error.patch to correct clean target on
      makefile for binary mp_util, adding -f modifier to rm command.

  [ Ilya Barygin ]
  * debian/rules: use dh 7.
  * Renumber and refresh remaining patches.
  * 08_missing_declaration.patch: fix implicit conversion of pointer to
    integer and back.
  * 09_translations_installation.patch: improve translations generation and
    installation procedure.
  * 10_system_libusb.patch: use system libusb instead of bundled version.
  * 11_system_flags.patch: use system CFLAGS and LDFLAGS.
  * Use upstream desktop file.
  * Add a manpage for new binary, mputil/mputil_smart.
  * debian/control: add Homepage field.
 -- Ilya Barygin <email address hidden> Sat, 22 Oct 2011 11:39:38 +0400

Changed in mplinuxman (Ubuntu):
status: Confirmed → Fix Released
Revision history for this message
Sebastian Carneiro (scarneiro) wrote :

Thanks Ilya!

I'm sorry i haven't been able to reply to this sooner. Your remarks and corrections are very helpful to a newbie like me!

Again, thanks for your work.

Sebastian.

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.