Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 172174 - app-admin/diradm-{2.9,2.9.1} - bad IUSE
Summary: app-admin/diradm-{2.9,2.9.1} - bad IUSE
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Robin Johnson
URL:
Whiteboard:
Keywords: QAbadiuse
Depends on:
Blocks:
 
Reported: 2007-03-25 14:17 UTC by Jakub Moc (RETIRED)
Modified: 2007-04-10 09:59 UTC (History)
0 users

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 Jakub Moc (RETIRED) gentoo-dev 2007-03-25 14:17:32 UTC
app-admin/diradm-2.9: attr(depends) uses unstated flags [ test ]
app-admin/diradm-2.9: attr(depends) uses unstated flags [ test ]
app-admin/diradm-2.9.1: attr(depends) uses unstated flags [ test ]
app-admin/diradm-2.9.1: attr(depends) uses unstated flags [ test ]

Please fix, thanks. :)
Comment 1 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2007-03-30 11:30:22 UTC
in cvs.
Comment 2 Jakub Moc (RETIRED) gentoo-dev 2007-04-09 11:56:04 UTC
Looks like you forgot to commit this in 2.9... ;) BTW, the below snippet is redundant, portage now adds test to USE w/ FEATURES="test" automagically.

<snip>
use test || die "You must build diradm with USE=test to run the testcases!"
</snip>
Comment 3 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2007-04-10 09:18:51 UTC
2.9 IUSE fixed.

You're wrong about the USE=test, it definetly doesn't work - I can see that it builds without USE=test in src_compile, and the die is triggered in src_test as well.

!!! ERROR: app-admin/diradm-2.9.3 failed.
Call stack:
  ebuild.sh, line 1614:   Called dyn_test
  ebuild.sh, line 1026:   Called qa_call 'src_test'
  ebuild.sh, line 44:   Called src_test
  diradm-2.9.3.ebuild, line 67:   Called die

!!! You must build diradm with USE=test to run the testcases!
!!! If you need support, post the topmost build error, and the call stack if relevant.
!!! A complete build log is located at '/var/log/portage/app-admin:diradm-2.9.3:20070410-091408.log'.
Comment 4 Jakub Moc (RETIRED) gentoo-dev 2007-04-10 09:27:31 UTC
(In reply to comment #3)
> You're wrong about the USE=test, it definetly doesn't work - I can see that it
> builds without USE=test in src_compile, and the die is triggered in src_test as
> well.

We don't understand each other, or...?

USE="-*" FEATURES="test" portageq envvar USE -> see that USE=test is enabled
Comment 5 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2007-04-10 09:54:54 UTC
only if >=portage-2.1.2 it seems (the box I test diradm on still has 2.1.1 of portage, hasn't upgraded yet). I don't think just depending on that version of portage is suitable for diradm yet. when the old ones fall out of the tree, then I'll change that line of code. 

The check does also catch cases like:
"ebuild diradm-2.9.3.ebuild digest compile test"
(notice the FEATURES are not set).
Comment 6 Jakub Moc (RETIRED) gentoo-dev 2007-04-10 09:59:03 UTC
(In reply to comment #5)
> only if >=portage-2.1.2 it seems (the box I test diradm on still has 2.1.1 of
> portage, hasn't upgraded yet).

Ah yeah, outdated portage explain this. :)