Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 173592 - add docdir to econf
Summary: add docdir to econf
Status: RESOLVED DUPLICATE of bug 468202
Alias: None
Product: Gentoo Hosted Projects
Classification: Unclassified
Component: PMS/EAPI (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: PMS/EAPI
URL:
Whiteboard:
Keywords:
: 287266 (view as bug list)
Depends on:
Blocks:
 
Reported: 2007-04-06 19:48 UTC by Alon Bar-Lev (RETIRED)
Modified: 2014-08-26 21:10 UTC (History)
6 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alon Bar-Lev (RETIRED) gentoo-dev 2007-04-06 19:48:57 UTC
Hello,

I don't know if this was discussed in the past.

But autoconf has docdir which is default to "prefix/share/doc/${PN}".
Our convention is "prefix/share/doc/${PF}".

Any reason not add --docdir="/usr/share/doc/${PF}" to default econf?

Thanks!
Comment 1 Zac Medico gentoo-dev 2007-04-18 03:31:29 UTC
Some configure scripts will generate an error.  For example, bash does:

>>> Compiling source in /var/tmp/portage/app-shells/bash-3.2_p15-r1/work/bash-3.2 ...
 * econf: updating bash-3.2/support/config.guess with /usr/share/gnuconfig/config.guess
 * econf: updating bash-3.2/support/config.sub with /usr/share/gnuconfig/config.sub
./configure --prefix=/usr --host=i586-pc-linux-gnu --mandir=/usr/share/man --infodir=/usr/share/info --datadir=/usr/share --sysconfdir=/etc --localstatedir=/var/lib --without-afs --disable-profiling --without-gnu-malloc --without-installed-readline --with-curses --build=i586-pc-linux-gnu
configure: error: unrecognized option: --docdir=/usr/share/doc/bash-3.2_p15-r1
Try `./configure --help' for more information.

!!! ERROR: app-shells/bash-3.2_p15-r1 failed.
Comment 2 Alon Bar-Lev (RETIRED) gentoo-dev 2007-04-20 16:56:14 UTC
Thanks for trying this!
It seems that it was added after autoconf-2.59...

Simple approach will be to configure --help | grep docdir...
More complex may be ./configure --version | sed -n -e '2 p' | sed 's/.* //' in order to get version end enable/disable features...

Or I can continue to add docdir= to econf.
Comment 3 Marius Mauch (RETIRED) gentoo-dev 2007-06-23 16:42:56 UTC
Mike, what do you think about this?
Comment 4 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2009-10-01 20:29:41 UTC
*** Bug 287266 has been marked as a duplicate of this bug. ***
Comment 5 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2009-10-01 20:31:04 UTC
Enough time went by that we can add this now?
Comment 6 Ciaran McCreesh 2009-10-02 12:16:54 UTC
(In reply to comment #5)
> Enough time went by that we can add this now?

A lot of things still break with it... If we're adding it to EAPI 4, we'll probably need a way for ebuilds to forcibly disable it...
Comment 7 Martin Väth 2009-10-03 14:39:24 UTC
How about calling the extended version "econf2" (or econf2.5 since it
works with autoconf:2.5)?
This seems much cleaner to me than manually hacking the --docdir argument
in the ebuild or having arguments magically enabled/disabled.
Comment 8 Anthony Basile gentoo-dev 2011-02-01 16:26:26 UTC
*** Bug 350423 has been marked as a duplicate of this bug. ***
Comment 9 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2011-09-23 13:35:30 UTC
Ping. We already have to grep configure scripts for EAPI 4, and we already have controllable compression there. Maybe it's time to add this one along with --htmldir?
Comment 10 Pacho Ramos gentoo-dev 2014-07-08 10:55:02 UTC
(In reply to Michał Górny from comment #9)
> Ping. We already have to grep configure scripts for EAPI 4, and we already
> have controllable compression there. Maybe it's time to add this one along
> with --htmldir?

Any updates on this? I also agree with passing it when configure option is available to prevent random bugs when a package installs docs in ${PN} location 

Thanks a lot
Comment 11 Ulrich Müller gentoo-dev 2014-08-26 21:10:33 UTC
Marking this as a duplicate of the extended proposal in bug 468202.

*** This bug has been marked as a duplicate of bug 468202 ***