Bug 306869

Summary:	dev-lang/lua ebuild does not copy *.css files into doc html dir
Product:	Gentoo Linux	Reporter:	Tommy Pettersson <ptp>
Component:	Current packages	Assignee:	Matti Bickel (RETIRED) <mabi>
Status:	RESOLVED FIXED
Severity:	trivial	CC:	jer, yngwin
Priority:	High
Version:	unspecified
Hardware:	All
OS:	All
Whiteboard:
Package list:		Runtime testing required:	---

Description Tommy Pettersson 2010-02-25 20:48:47 UTC

The html documentation for Lua is not completely installed. In addition to the .html files and the .gif images that are installed, there are also .css files and a .png file that are not installed.

Reproducible: Always

Steps to Reproduce:
1. Look at e.g. the manual.html page in a css-enabled web browser.
Actual Results:  
Default style.

Expected Results:  
Lua and Manual style specified in missing .css files.

Comment 1 Jeroen Roovers (RETIRED) gentoo-dev

2010-02-26 16:00:22 UTC

Would this do?

Index: lua-5.1.4-r5.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/dev-lang/lua/lua-5.1.4-r5.ebuild,v
retrieving revision 1.1
diff -u -B -r1.1 lua-5.1.4-r5.ebuild
--- lua-5.1.4-r5.ebuild 23 Feb 2010 12:19:34 -0000      1.1
+++ lua-5.1.4-r5.ebuild 26 Feb 2010 15:59:43 -0000
@@ -90,7 +90,7 @@
        || die "emake install gentoo_install failed"
 
        dodoc HISTORY README
-       dohtml doc/*.html doc/*.gif
+       dohtml doc/*.html doc/*.css doc/*.gif
 
        insinto /usr/share/pixmaps
        doins etc/lua.ico

Comment 2 Matti Bickel (RETIRED) gentoo-dev

2010-02-26 16:33:03 UTC

Sure, if you've verified it indeed fixes the issue, just go ahead and commit it.

Comment 3 Matti Bickel (RETIRED) gentoo-dev

2010-03-05 19:59:13 UTC

Fixed in cvs. Thanks for spotting this!