[MPlayer-dev-eng] [PATCH] Fix FTBFS in kfreebsd + GNU/Hurd info

[Roberto Togni]

On Tue, 15 Sep 2015 18:16:59 -0400
Miguel A. Colón Vélez <debian.micove at gmail.com> wrote:

> Hello:
> 
> Mplayer currently fails to build on kfreebsd.
> https://buildd.debian.org/status/package.php?p=mplayer&suite=unstable
> 
> Hopefully this makes it in before the new release.
> 
> Also there is a fairly old patch to fix a FTBFS with GNU/Hurd
> http://anonscm.debian.org/cgit/pkg-multimedia/mplayer.git/tree/debian/patches/50_Hurd_PATH_MAX.patch
> More info:
> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=654974
> https://www.gnu.org/software/hurd/community/gsoc/project_ideas/maxpath.html
> 
> I'm currently building it in qemu since the GNU build has not been
> compiled in a while so I'm not sure if there are more issues in there.
I'm not sure about this.
Can't we just define PATH_MAX when compiling for HURD?
I understand that the patch does the ideal thing (no path length
limit), but do we really want all those ifdefs for a system that is
barely used?

> 
> It's unrelated but:
>   echo "Compiling without FFmpeg is currently not supported/working."
>   echo "Please contact us if that is an issue for you."
>   die "both internal and external FFmpeg missing"
> 
> Could a flag be added to at least not require ffmpeg and yasm when
> ONLY building the documentation?
> 
> Something like
> ./configure --language=all --build-only-docs
> make html-chunked
To build the docs we normally do (see the release script in DOCS/tech)
touch ffmpeg/libavutil/x86/asm.h
configure --language=all --yasm=''

> 
> The new bbuild for arch=all packaged ends up installing ~400 packages
> when only ~30 are really needed. I can drop most of them except ffmpeg
> and yasm. Yasm can be dropped with --yasm=''" but a flag to skip those
> two or most of the unneeded checks could be useful.
> 
I think it was never planned to have the docs buildable separately from
the code. Probably the easiest thing to do is to provide a separate
Makefile (and configure if really needed) for the docs.


Ciao,
 Roberto