Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 127299 - dev-lang/gprolog-1.2.19 docs install paths
Summary: dev-lang/gprolog-1.2.19 docs install paths
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Gentoo Prolog project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-03-23 05:29 UTC by TGL
Modified: 2006-03-23 23:46 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
gprolog-1.2.19-doc_paths.patch (gprolog-1.2.19-doc_paths.patch,1.05 KB, patch)
2006-03-23 05:29 UTC, TGL
Details | Diff
gprolog-1.2.19-doc_paths.patch (gprolog-1.2.19-doc_paths.patch,1.03 KB, patch)
2006-03-23 15:09 UTC, TGL
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description TGL 2006-03-23 05:29:30 UTC
This ebuild installs some documentation in three differents paths:
 - /usr/share/doc/${PN}-${PVR} (dodoc files)
 - /usr/share/doc/${P} (html files - it happens to be the same as the previous one for now, but that won't be true anymore in case of revision bump)
 - /usr/share/${P} (examples files)

I think it would be better to put everything under the standard location (/usr/share/doc/${PN}-${PVR}), and will attach an ebuild patch for this change. Also, it uses the ./configure options instead of forcing Makefiles variables, but that's just a personnal preference (i can submit a patch using the current method if you like it better).
Comment 1 TGL 2006-03-23 05:29:58 UTC
Created attachment 82935 [details, diff]
gprolog-1.2.19-doc_paths.patch
Comment 2 TGL 2006-03-23 15:09:54 UTC
Created attachment 82969 [details, diff]
gprolog-1.2.19-doc_paths.patch

I had completly forgotten existence of the $PF variable. This version of the patch uses it (instead of $PN-$PVR).
Comment 3 Keri Harris gentoo-dev 2006-03-23 23:46:22 UTC
Thanks for the bug report (and patches)!

Your patch looks good; I've applied it and committed the changes to cvs.

Thanks,

Keri.