Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#10507 closed Bug Report - General (fixed)

Gentoo: media-plugins/mythgame-0.24.2_p20120221 does not build

Reported by: Alec <alecm@…> Owned by: Raymond Wagner
Priority: minor Milestone: 0.25.1
Component: Packaging Version: 0.24.2
Severity: medium Keywords:
Cc: Ticket locked: no

Description

I've been unable to build mythgame for quite a while (the last version I built was 0.24.1_p20111116). I am running on ~amd64.

I'm attaching the build log.

Attachments (2)

build.log (8.4 KB) - added by Alec M <alecm@…> 6 years ago.
build.log
zlib_macro.patch (5.2 KB) - added by SiliconFiend@… 6 years ago.
Patch to accomodate Gentoo's renamed zlib macros

Download all attachments as: .zip

Change History (9)

Changed 6 years ago by Alec M <alecm@…>

Attachment: build.log added

build.log

comment:1 Changed 6 years ago by christopher.j.reeves@…

The following link may help. It appears that gentoo has changed the zlib header files due to a namespace pollution problem.

https://bugs.gentoo.org/show_bug.cgi?id=383179

Changed 6 years ago by SiliconFiend@…

Attachment: zlib_macro.patch added

Patch to accomodate Gentoo's renamed zlib macros

comment:2 Changed 6 years ago by beirdo

Resolution: Won't Fix
Status: newclosed

I don't think this will be accepted unless it will compile also for non-Gentoo systems. Until Gentoo fixes their customizations to work properly, you may be out of luck, or need to keep a custom local patch.

As this will not build on my setup (Ubuntu-based) and Ubuntu is one of our most popular platforms... Sorry, no can do.

comment:3 Changed 6 years ago by beirdo

Owner: set to Raymond Wagner

What might work is to get the ebuilds to use your patch. Handing this to wagnerrp for consideration there.

comment:4 Changed 6 years ago by Raymond Wagner <rwagner@…>

In bde24c5cd3050252687890f9261e4706b513acd3/mythtv:

Remove use of OF zlib macro in MythGame?.

The macro seems to only be of any use for compilers that are not ANSI-C
compliant, which should be... nothing these days. Chances are the rest
of the code base wouldn't compile on it anyway. On anything modern, the
macro doesn't do anything. Remove it and see what happens. Comments in
zconf.h seem to indicate it may cause problems for OSX.

Refs #10507

comment:5 Changed 6 years ago by Raymond Wagner <rwagner@…>

Resolution: Won't Fixfixed

In a572f18f512661af0e9a531357caca3df46fec9d/packaging:

Big ebuild bump.

This includes a long overdue update of the ebuilds for 0.25.1 and 0.26.
This discontinues the older set of 0.24 ebuilds, as well as the
MythVideo? plugin, which is included as part of core in 0.25 and beyond.

The ebuild generator 'mythtv-buildebuild.py' is no longer viable due to
changes in the Github API, so it has been removed, to be replaced
shortly with an updated version.

This updates to a versio of 0.26_pre that fixes #10507 and the fix is
included as a patch for 0.25.1.

This resets the MythNews? ebuild back to the 0.24 version, as it had
accidentally been overwritten by the ebuild for MythGallery. It also
adds MythBrowser? as a prerequisite.

comment:6 Changed 6 years ago by Raymond Wagner

Milestone: unknown0.25.1

comment:7 in reply to:  4 Changed 6 years ago by Doug Goldstein <cardoe@…>

Replying to Raymond Wagner <rwagner@…>:

In bde24c5cd3050252687890f9261e4706b513acd3/mythtv:

Remove use of OF zlib macro in MythGame?.

The macro seems to only be of any use for compilers that are not ANSI-C
compliant, which should be... nothing these days. Chances are the rest
of the code base wouldn't compile on it anyway. On anything modern, the
macro doesn't do anything. Remove it and see what happens. Comments in
zconf.h seem to indicate it may cause problems for OSX.

Refs #10507

Can this get backported to fixes/0.25?

Note: See TracTickets for help on using tickets.