Bug 158536 – media-video/vlc-0.8.6 duplicates docs in /usr/share/doc/vlc w/ USE="-nsplugin"

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Chris Mayo 2006-12-19 04:55:20 UTC]

vlc-0.8.6 installs docs as:

/usr/portage/local/media-video/vlc $ ls /usr/share/doc/vlc
bugreport-howto.txt  fortunes.txt  intf-cdda.txt  intf-vcd.txt

/usr/portage/local/media-video/vlc $ ls /usr/share/doc/vlc-0.8.6/
AUTHORS.gz       HACKING.gz        intf-vcd.txt.gz  NEWS.gz    THANKS.gz
fortunes.txt.gz  intf-cdda.txt.gz  MAINTAINERS.gz   README.gz


fi slipped? Patch attached.

[Comment 1 Chris Mayo 2006-12-19 04:55:48 UTC]

Created attachment 104354 [details, diff]
vlc-0.8.6-ebuild-doc.patch

[Comment 2 Jakub Moc (RETIRED) 2006-12-19 04:57:41 UTC]

ls ls /usr/share/doc/vlc
ls: cannot access ls: No such file or directory
ls: cannot access /usr/share/doc/vlc: No such file or directory

# equery f vlc | grep /doc
/usr/share/doc
/usr/share/doc/vlc-0.8.6
/usr/share/doc/vlc-0.8.6/AUTHORS.gz
/usr/share/doc/vlc-0.8.6/HACKING.gz
/usr/share/doc/vlc-0.8.6/MAINTAINERS.gz
/usr/share/doc/vlc-0.8.6/NEWS.gz
/usr/share/doc/vlc-0.8.6/README.gz
/usr/share/doc/vlc-0.8.6/THANKS.gz
/usr/share/doc/vlc-0.8.6/fortunes.txt.gz
/usr/share/doc/vlc-0.8.6/intf-cdda.txt.gz
/usr/share/doc/vlc-0.8.6/intf-vcd.txt.gz

[Comment 3 Jakub Moc (RETIRED) 2006-12-19 05:00:45 UTC]

Meh, typo above... Anyway, no such files installed here.

# ls /usr/share/doc/vlc
ls: cannot access /usr/share/doc/vlc: No such file or directory

[Comment 4 Chris Mayo 2006-12-19 06:10:29 UTC]

Do you have USE="nsplugin" ?

[Comment 5 Jakub Moc (RETIRED) 2006-12-19 07:06:18 UTC]

(In reply to comment #4)
> Do you have USE="nsplugin" ?

 

# emerge -pv vlc

These are the packages that would be merged, in order:

Calculating dependencies... done!
[ebuild   R   ] media-video/vlc-0.8.6  USE="-3dfx X a52 aalib alsa (-altivec) -arts -avahi -bidi cdda cddb corba daap -debug directfb dts dvb dvd -esd fbcon flac ggi gnutls -hal httpd libcaca libnotify lirc live matroska mod mp3 mpeg ncurses nsplugin ogg opengl -optimisememory -oss png rtsp samba sdl -seamonkey shout skins speex stream svg -svga theora truetype -upnp v4l vcd vlm vorbis win32codecs wxwindows xinerama xml xosd xv" 0 kB 

Total: 1 package (1 reinstall), Size of downloads: 0 kB

[Comment 6 Chris Mayo 2006-12-19 07:54:34 UTC]

Here's a snip from the ebuild that the patch addresses:

if use nsplugin; then
	dodir "/usr/$(get_libdir)/${PLUGINS_DIR}"
	mv "${D}"/usr/$(get_libdir)/mozilla/{components,plugins}/* \
		"${D}/usr/$(get_libdir)/${PLUGINS_DIR}/"

	rm -rf "${D}/usr/share/doc/vlc" \
	"${D}"/usr/share/vlc/vlc{16x16,32x32,48x48,128x128}.{png,xpm,ico}
fi

The rm doc (& icons) has slipped inside the if for nsplugin. The patch just moves it out for the benefit of those of us that don't use nsplugin.

[Comment 7 Jakub Moc (RETIRED) 2006-12-19 12:05:43 UTC]

Noone will notice again if you don't reopen the bug.

[Comment 8 Alexis Ballier 2007-02-10 19:06:08 UTC]

Fixed, thanks for reporting.