Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

mediatomb fails to build on OSX 10.8.2 #17999

Closed
linickx opened this issue Feb 21, 2013 · 11 comments
Closed

mediatomb fails to build on OSX 10.8.2 #17999

linickx opened this issue Feb 21, 2013 · 11 comments

Comments

@linickx
Copy link

linickx commented Feb 21, 2013

Hello,
I have a clean install of ML, other brews install ok, brew doctor ok but mediatomb fails.

The verbose output, logfile and doctor output is in this gist https://gist.github.com/linickx/5007880

The summary of the fail is below.

NickelMac:~ nick$ brew install mediatomb
==> Downloading http://downloads.sourceforge.net/mediatomb/mediatomb-0.12.1.tar.
Already downloaded: /Library/Caches/Homebrew/mediatomb-0.12.1.tar.gz
==> Downloading patches
######################################################################## 100.0%
==> Patching
patching file src/metadata/ffmpeg_handler.cc
==> ./configure --prefix=/usr/local/Cellar/mediatomb/0.12.1
==> make install
         ^
2 errors generated.
make[1]: *** [libmediatomb_a-content_manager.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [install-recursive] Error 1

NickelMac:~ nick$

rgds,
Nick

@adamv
Copy link
Contributor

adamv commented Feb 21, 2013

I updated the fails_with to 425, but now get this build error:

e/ffmpeg  -c -o mpeg_parse.o `test -f '../src/mpegdemux/mpeg_parse.c' || echo './'`../src/mpegdemux/mpeg_parse.c
brew: superenv removed: -g -O2
../src/metadata/libmp4v2_handler.cc: In function ‘void addMetaField(metadata_fields_t, void*, zmm::Ref<CdsItem>)’:
../src/metadata/libmp4v2_handler.cc:77: error: ‘MP4GetMetadataName’ was not declared in this scope
../src/metadata/libmp4v2_handler.cc:80: error: ‘MP4GetMetadataArtist’ was not declared in this scope
../src/metadata/libmp4v2_handler.cc:83: error: ‘MP4GetMetadataAlbum’ was not declared in this scope
../src/metadata/libmp4v2_handler.cc:86: error: ‘MP4GetMetadataYear’ was not declared in this scope
../src/metadata/libmp4v2_handler.cc:98: error: ‘MP4GetMetadataGenre’ was not declared in this scope
../src/metadata/libmp4v2_handler.cc:101: error: ‘MP4GetMetadataComment’ was not declared in this scope
../src/metadata/libmp4v2_handler.cc:104: error: ‘MP4GetMetadataTrack’ was not declared in this scope

@adamv
Copy link
Contributor

adamv commented Feb 23, 2013

The MacPorts port has additional patches for ffmpeg: https://trac.macports.org/browser/trunk/dports/net/mediatomb/Portfile

@linickx
Copy link
Author

linickx commented Feb 24, 2013

linickx@b0d70c8#commitcomment-2682206 => I tried just dropping those patches into the formula, but it's doesn't appear to be that simple 😟

@jacknagel
Copy link
Contributor

MacPorts patches have to be applied with -p0, grep for :p0 => for examples.

@linickx
Copy link
Author

linickx commented Mar 11, 2013

Yep, that does the trick => linickx@b5d9d62#commitcomment-2776120 ... anyone wanna test/confirm before I raise a pull request?

@adamv
Copy link
Contributor

adamv commented Mar 11, 2013

I'm a little confused here; since the formula has no declared dependencies, either it is finding installed software anyway, or the compiler errors are coming from the software's own code.

@adamv
Copy link
Contributor

adamv commented Mar 11, 2013

I see on 10.6 it's picking up the taglib I have installed. Looks like some deps need to be declared.

@linickx
Copy link
Author

linickx commented Mar 11, 2013

It builds with two of the three macports patches => linickx@3ed9fb7
The config log ( https://gist.github.com/linickx/5135251 ) shows that it will pickup your taglib, I will put in some explicit dependencies to see if that makes the build better

@linickx
Copy link
Author

linickx commented Mar 11, 2013

Build with dependencies Works For Me => linickx@9c2fba5 & https://gist.github.com/linickx/5135619

@adamv
Copy link
Contributor

adamv commented Mar 13, 2013

Please submit a pull request with the change

@nstCactus
Copy link
Contributor

I have the same issue and I think it may be due to some API changes in mp4v2. The logs refers to functions (MP4GetMetadata*) that were deprecated (http://mp4v2.googlecode.com/svn/doc/1.9.0/api/group__mp4__meta.html) and has been removed in mp4v2-2.0.0 (https://code.google.com/p/mp4v2/).

I tried to compile the latest master and the ./configure script now checks if these functions are present in the mp4v2 version installed on the system. I have mp4v2-2.0.0. Here are the last lines of my ./configure script output

CONFIGURATION SUMMARY ----

sqlite3               : yes
mysql                 : missing
libjs                 : yes
libmagic              : yes
inotify               : missing
libexif               : yes
expat                 : yes
id3lib                : disabled
taglib                : yes
libmp4v2              : not compatible
ffmpeg                : yes
ffmpegthumbnailer     : yes
lastfmlib             : yes
external transcoding  : yes
curl                  : yes
YouTube               : yes
db-autocreate         : yes

As you can see mp4v2 is flagged as not compatible. When I run make, mediatomb builds succesfully.

I think we just need an older version of mp4v2.

@adamv adamv closed this as completed in 1e3e829 Jun 27, 2013
handyman5 pushed a commit to handyman5/homebrew that referenced this issue Oct 7, 2013
Fixes Homebrew#17999.
Closes Homebrew#18451.
Closes Homebrew#19445.
Closes Homebrew#20759.
Closes Homebrew#20838.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
@Homebrew Homebrew locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants