Bug 349895 – media-libs/edje-1.0.0_beta3 is missing doc useflag

• Status: RESOLVED WONTFIX
• Product: Gentoo Linux

[Description Ralph Sennhauser (RETIRED) 2010-12-27 20:47:59 UTC]

edje has a configure option doc to either build doxygen docs or not. The ebuild should reflect that.

Reproducible: Always

[Comment 1 Ralph Sennhauser (RETIRED) 2010-12-27 20:49:00 UTC]

Created attachment 258208 [details, diff]
Add doc use flag patch

[Comment 2 Thomas Sachau 2010-12-28 12:29:17 UTC]

The ebuild has a doc USE flag, it is not required to be in the ebuild, it can also be found in the eclass like in this case.
Additionally, the current variant in the eclass does build and install additional docs, while the configure option doesnt seem to do anything from my quick testing. Because of this, i will close this bug as wontfix.

[Comment 3 Ralph Sennhauser (RETIRED) 2010-12-28 12:48:31 UTC]

(In reply to comment #2)
> The ebuild has a doc USE flag, it is not required to be in the ebuild, it can
> also be found in the eclass like in this case.
> Additionally, the current variant in the eclass does build and install
> additional docs, while the configure option doesnt seem to do anything from my
> quick testing. Because of this, i will close this bug as wontfix.
> 

When having doxygen installed and --disable-doc the doxygen docs will not be built.
When having doxygen not installed and --enable-doc (default configure option) and useflag doc disabled, configure complains about no doxygen being found and the doxygen docs are not built.
When having doxygen installed and use doc disabled the docs are built despite me having no interest in them.

This is a clear bug.

[Comment 4 Ralph Sennhauser (RETIRED) 2010-12-28 15:43:08 UTC]

As the question arose if the doc configure option does anything at all on IRC, I attach two build logs. Both are from the patched ebuild, the first is from the second consecutive build with doc use disabled and the second one the run right afterwards with doc use flag enabled.

A snipped from the merge log of the second build log to show additional files are being installed:

>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/globals_vars.html
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/group__Edje__cache__Group.html
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/group__Edje__main__Group.html
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/group__Edje__message__queue__Group.html
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/group__Edje__program__Group.html
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/head_bg.png
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/header_menu_background.png
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/header_menu_background_last.png
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/header_menu_current_background.png
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/header_menu_unselected_background.png
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/index.html
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/logo.png
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/lua__script_8edc-example.html
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/luaref.html
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/modules.html
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/pages.html
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/struct__Anchor.html
>>> [obj] /usr/share/doc/edje-1.0.0_beta3/html/html/struct__Edje.html

[Comment 5 Ralph Sennhauser (RETIRED) 2010-12-28 15:43:51 UTC]

Created attachment 258269 [details]
With doc use flag disabled

[Comment 6 Ralph Sennhauser (RETIRED) 2010-12-28 15:44:26 UTC]

Created attachment 258270 [details]
With doc use flag enabled

[Comment 7 Ralph Sennhauser (RETIRED) 2010-12-28 16:02:48 UTC]

My bad, I missed the emake doc in the eclass. So setting the resolution back to wontfix. Sorry for that one.