290921 – dev-util/subversion-1.6.6: dohtml needs -r for compatibility with Doxygen 1.6

Bug 290921 - dev-util/subversion-1.6.6: dohtml needs -r for compatibility with Doxygen 1.6

Summary: dev-util/subversion-1.6.6: dohtml needs -r for compatibility with Doxygen 1.6

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All All

Importance:	High normal (vote)
Assignee:	Arfrever Frehtes Taifersar Arahesis (RETIRED)

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2009-10-28 20:42 UTC by Dirk Heinrichs
Modified:	2009-11-04 12:04 UTC (History)
CC List:	4 users (show)

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

Attachments
paludis --info output (paludis_info.log,15.33 KB, text/plain) 2009-10-28 20:44 UTC, Dirk Heinrichs	Details
Build log (1256761498-install-dev-util_subversion-1.6.6:0::gentoo.out,473.23 KB, text/plain) 2009-10-28 21:07 UTC, Dirk Heinrichs	Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Dirk Heinrichs 2009-10-28 20:42:49 UTC

When updating subversion to 1.6.6, I get this:

install -m0644 doc/doxygen/html/svn__wc_8h_source.html /gentoo/build/dev-util-subversion-1.6.6/image/usr/share/doc/subversion-1.6.6/html//                                                                                                                      
install -m0644 doc/doxygen/html/svn__xml_8h.html /gentoo/build/dev-util-subversion-1.6.6/image/usr/share/doc/subversion-1.6.6/html//                                                                                                                            
install -m0644 doc/doxygen/html/svn__xml_8h_source.html /gentoo/build/dev-util-subversion-1.6.6/image/usr/share/doc/subversion-1.6.6/html//                                                                                                                     
install -m0644 doc/doxygen/html/tab_b.gif /gentoo/build/dev-util-subversion-1.6.6/image/usr/share/doc/subversion-1.6.6/html//   
install -m0644 doc/doxygen/html/tab_l.gif /gentoo/build/dev-util-subversion-1.6.6/image/usr/share/doc/subversion-1.6.6/html//   
install -m0644 doc/doxygen/html/tab_r.gif /gentoo/build/dev-util-subversion-1.6.6/image/usr/share/doc/subversion-1.6.6/html//   
install -m0644 doc/doxygen/html/tabs.css /gentoo/build/dev-util-subversion-1.6.6/image/usr/share/doc/subversion-1.6.6/html//    
install -m0644 doc/doxygen/html/todo.html /gentoo/build/dev-util-subversion-1.6.6/image/usr/share/doc/subversion-1.6.6/html//   
install -m0644 doc/doxygen/html/unionsvn__opt__revision__value__t.html /gentoo/build/dev-util-subversion-1.6.6/image/usr/share/doc/subversion-1.6.6/html//                                                                                                      

!!! ERROR in dev-util/subversion-1.6.6::gentoo:
!!! In src_install at line 8148
!!! Installation of Subversion HTML documentation failed

!!! Call stack:
!!!    * src_install (/gentoo/build/dev-util-subversion-1.6.6/temp/loadsaveenv:8148)
!!!    * ebuild_f_install (/usr/libexec/paludis/0/src_install.bash:48)
!!!    * ebuild_main (/usr/libexec/paludis/ebuild.bash:604)
!!!    * main (/usr/libexec/paludis/ebuild.bash:632)

No real error message whatsoever, it just stops at this state :(

Reproducible: Always

Steps to Reproduce:
1. paludis -i everything

Actual Results:  
See above.

Expected Results:  
Well, update subversion.

You want it, you get it ;)

# paludis --info dev-util/subversion

[Too long, will be attached]

Comment 1 Dirk Heinrichs 2009-10-28 20:44:56 UTC

Created attachment 208572 [details]
paludis --info output

Comment 2 Sebastian Luther (few) 2009-10-28 20:51:19 UTC

The complete log please.

Comment 3 Dirk Heinrichs 2009-10-28 21:07:52 UTC

Created attachment 208577 [details]
Build log

Here you are.

Comment 4 Stephan Friedrichs 2009-10-28 22:11:08 UTC

same here

Comment 5 Sebastian Luther (few) 2009-10-29 07:32:49 UTC

(In reply to comment #4)
> same here
> 

Paludis or portage?

Comment 6 Stephan Friedrichs 2009-10-29 11:05:37 UTC

(In reply to comment #5)
> (In reply to comment #4)
> > same here
> > 
> 
> Paludis or portage?
> 

Paludis

Comment 7 Ingmar Vanhassel 2009-10-29 11:38:06 UTC

Could anyone who can reproduce this put "set -x" at the top of /usr/libexec/paludis/utils/dohtml, then attach the output?

Comment 8 Dirk Heinrichs 2009-10-29 17:07:05 UTC

(In reply to comment #7)
> Could anyone who can reproduce this put "set -x" at the top of
> /usr/libexec/paludis/utils/dohtml, then attach the output?

I did, but it seems it isn't even called.

Comment 9 Sebastian Luther (few) 2009-10-29 21:41:17 UTC

If there is someone with portage who can reproduce this, please attach "emerge --info subversion" and the build.log.

Comment 10 Sebastian Luther (few) 2009-10-31 18:29:54 UTC

Ok, only paludis is hit by this. The problem is that if you have app-doc/doxygen-1.6.1 doc/doxygen/html contains a subdirectory called 'search'. Paludis errors out when it hits that directory, since -r is not passed to dohtml. Portage simply ignores this subdirectory. This directory contains html files, so it seems best to pass -r to dohtml.

Comment 11 Zac Medico gentoo-dev

2009-11-01 00:25:16 UTC

(In reply to comment #10)
> Portage simply ignores this subdirectory.

This is fixed in portage to produce a eqawarn message, and released in 2.1.7.3 and 2.2_rc48. It's just a warning for now. We can make it more strict later (revert bug 149745), after the warning has been release for awhile.

Comment 12 Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev

2009-11-04 12:04:17 UTC

Fixed.