the mplayer ebuild is installing a crapload of .svn directories into the docdir. this patch is against mplayer-1.0_rc2_pre20070321-r4 but other versions are probably affected too.
Created attachment 114389 [details, diff] mplayer-1.0_rc2_pre20070321-r4.ebuild.diff
I would prefer to strip .svn dirs as early as possible, instead of copying files just to delete them again immediately afterwards. I would like it best if I could even avoid unpacking those directories, but right now it seems to be impossible to pass arguments for tar to the unpack function. I just filed bug #172322 with this suggestion, but it will probably take some time till a new portage version is prevalent. Duplicating unpack inside the ebuild seems too much work to be woth the effort. So I'd run the delete on ${S} immediately after the main tarball has been unpacked, instead of waiting till the install phase. While working on bug #172314 I found out that at least dohtml has the option to exclude specific dirs, but this solution would not help TOOLS for example. Another possibility to realize a copy with excludes would be a pipe connecting two tar commands. An alternative if there is a good reason why you don't want to delete files from the source tree, and it might perhaps help with permissions as well.
Come-on guys, this one's a no-brainer. And it's even a *stable* package now.
(In reply to comment #3) > Come-on guys, this one's a no-brainer. And it's even a *stable* package now. > Stable because of bug 168917. Either way, fixed in CVS in mplayer-1.0.20070427.ebuild
Finally fixed correctly in mplayer-1.0.20070321-r1.ebuild
