Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 203647 - media-libs/libmms-0.4 version bump and SRC_URI change
Summary: media-libs/libmms-0.4 version bump and SRC_URI change
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Gentoo Media-video project
URL:
Whiteboard:
Keywords:
: 246930 (view as bug list)
Depends on:
Blocks:
 
Reported: 2007-12-29 04:13 UTC by Gordon Stratton
Modified: 2008-12-12 13:59 UTC (History)
5 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
libmms-0.4 ebuild (libmms-0.4.ebuild,804 bytes, text/plain)
2007-12-29 04:14 UTC, Gordon Stratton
Details
updated libmms-0.4 ebuild (libmms-0.4.ebuild,786 bytes, text/plain)
2007-12-29 21:59 UTC, Gordon Stratton
Details
new ebuild with C++ header fixes (libmms-0.4.ebuild,888 bytes, text/plain)
2007-12-30 00:03 UTC, Hans Rakers
Details
Patch to fix unclosed comment and C++ reserved keyword in the header files (libmms-0.4-cpp-header-fix.patch,2.29 KB, patch)
2007-12-30 00:04 UTC, Hans Rakers
Details | Diff
libmms-0.4.ebuild (from 0.3 with cpp-headers fix) (libmms-0.4.ebuild,910 bytes, text/plain)
2008-04-10 07:48 UTC, Jan Vansteenkiste
Details
CPP headers fix and unclosed comment fix patch (libmms-0.4-cpp_headers.patch,1.98 KB, patch)
2008-04-10 07:50 UTC, Jan Vansteenkiste
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gordon Stratton 2007-12-29 04:13:09 UTC
I've attached an updated ebuild for media-libs/libmms. I made the following changes:

* I updated the SRC_URI to point to the new project location at Launchpad (changed from Sourceforge)
* I removed the sed command that changed malloc.h to stdlib.h, it's not necessary anymore
* I added ia64 and sparc-fbsd architectures (probably could add more, these are safe bets based on the architectures already in the libmms-0.3 ebuild) and flagged all arches as testing
* Version bump from 0.3 to 0.4

I've tested this on an x86. Let me know if I can help with anything else related to this ebuild.

Reproducible: Always

Steps to Reproduce:
Comment 1 Gordon Stratton 2007-12-29 04:14:37 UTC
Created attachment 139555 [details]
libmms-0.4 ebuild
Comment 2 Samuli Suominen (RETIRED) gentoo-dev 2007-12-29 11:06:24 UTC
(In reply to comment #0)
> I've attached an updated ebuild for media-libs/libmms. I made the following
> changes:
> * I added ia64 and sparc-fbsd architectures (probably could add more, these are
> safe bets based on the architectures already in the libmms-0.3 ebuild) and
> flagged all arches as testing
> * Version bump from 0.3 to 0.4
> 
> I've tested this on an x86. Let me know if I can help with anything else
> related to this ebuild.

So you didn't test running it on Sparc Gentoo/FreeBSD or Itanium? Then those keywords can't be added either.
Comment 3 Gordon Stratton 2007-12-29 21:59:25 UTC
Created attachment 139580 [details]
updated libmms-0.4 ebuild
Comment 4 Hans Rakers 2007-12-30 00:03:54 UTC
Created attachment 139593 [details]
new ebuild with C++ header fixes

I took the liberty of adapting your ebuild for libmms-0.4 and added a patch that fixes some issues in the header files, including an unclosed comment and the use of C++ reserved keywords (this) in the header files.

ebuild attached, patch follows
Comment 5 Hans Rakers 2007-12-30 00:04:43 UTC
Created attachment 139595 [details, diff]
Patch to fix unclosed comment and C++ reserved keyword in the header files
Comment 6 jamatik 2008-03-09 07:18:03 UTC
why not in portage already? mms streams hanging a lot
e.g. http://wwitv.com/tv_channels/7244.htm
Comment 7 jamatik 2008-03-09 07:18:46 UTC
(In reply to comment #6)
> why not in portage already? mms streams hanging a lot
> e.g. http://wwitv.com/tv_channels/7244.htm
> 

with version 0.3 i mean
Comment 8 jamatik 2008-03-09 08:21:41 UTC
by the way...
the ebuilds don't work (ebuild *.ebuild digest)
the patch doesn't work to.

so i copied the 0.3 ebuild to libmms-0.4.ebuild in my overlay
Comment 9 Jan Vansteenkiste 2008-04-10 07:47:13 UTC
(In reply to comment #8)
> by the way...
> the ebuilds don't work (ebuild *.ebuild digest)
> the patch doesn't work to.
> 
> so i copied the 0.3 ebuild to libmms-0.4.ebuild in my overlay
> 
I had the same problem, I copied the libmms 0.3 ebuild, and added the patch (I got it working by making it again).

Could anybody else check if the ebuilds Hans provided work? If so, feel free to remove the following attachments. And otherwise: give these a try. :)

Comment 10 Jan Vansteenkiste 2008-04-10 07:48:29 UTC
Created attachment 149276 [details]
libmms-0.4.ebuild (from 0.3 with cpp-headers fix)
Comment 11 Jan Vansteenkiste 2008-04-10 07:50:06 UTC
Created attachment 149277 [details, diff]
CPP headers fix and unclosed comment fix patch

Re-inventing the wheel from Hans. 
It seems to work though, still haven't really figured out why yours didn't do the trick.
Comment 12 Samuli Suominen (RETIRED) gentoo-dev 2008-12-12 13:59:08 UTC
*libmms-0.4 (12 Dec 2008)

  12 Dec 2008; <ssuominen@gentoo.org>
  +files/libmms-0.4-nested_comments.patch,
  +files/libmms-0.4-this_keyword.patch, +libmms-0.4.ebuild:
  Version bump for bug 203647, thanks to Gordon and Hans.
Comment 13 Samuli Suominen (RETIRED) gentoo-dev 2008-12-12 13:59:54 UTC
*** Bug 246930 has been marked as a duplicate of this bug. ***