Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 475114 - emerge --verbose output suggests testing is disabled when FEATURES=test is enabled in /etc/portage/env/*
Summary: emerge --verbose output suggests testing is disabled when FEATURES=test is en...
Status: RESOLVED DUPLICATE of bug 388781
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Core - Interface (emerge) (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Portage team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-06-28 15:25 UTC by Jeroen Roovers (RETIRED)
Modified: 2013-06-28 16:58 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 Jeroen Roovers (RETIRED) gentoo-dev 2013-06-28 15:25:45 UTC
# emergefromgentoox86 -va youtube-dl
 * Repository: /newaches/gentoo/cvs/gentoo-x86/

These are the packages that would be merged, in order:

[ebuild     U ~] net-misc/youtube-dl-2013.06.34.2 [2013.06.34] USE="offensive* {-test*}" PYTHON_SINGLE_TARGET="python2_7 -python2_5 -python2_6 -python3_3%" PYTHON_TARGETS="python2_7 -python2_5 -python2_6 -python3_3%" 0 kB

Total: 1 package (1 upgrade), Size of downloads: 0 kB

Would you like to merge these packages? [Yes/No]

[...]

>>> Source compiled.
make -j5 test
#nosetests --with-coverage --cover-package=youtube_dl --cover-html --verbose --processes 4 test
nosetests --verbose test
test_justin_tv_channelid_matching (test_all_urls.TestAllURLsMatching) ... ok
test_justin_tv_chapterid_matching (test_all_urls.TestAllURLsMatching) ... ok
test_justintv_videoid_matching (test_all_urls.TestAllURLsMatching) ... ok
test_youtube_channel_matching (test_all_urls.TestAllURLsMatching) ... ok
test_youtube_extract (test_all_urls.TestAllURLsMatching) ... ok
test_youtube_matching (test_all_urls.TestAllURLsMatching) ... ok
test_youtube_playlist_matching (test_all_urls.TestAllURLsMatching) ... ok
test_ARD (test_download.TestDownload) ... ok
test_Bandcamp (test_download.TestDownload) ... ok

It does run the test phase, but it suggests that it is not going to do this.
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2013-06-28 15:27:50 UTC
Forgot to mention that emergefromgentoox86 does this:
   PORTDIR_OVERLAY=${CVS_GENTOO_X86} \
       emerge --package-moves=n -v --nodeps

Maybe that's significant.
Comment 2 Zac Medico gentoo-dev 2013-06-28 15:56:57 UTC
It looks like maybe you used bashrc instead of package.env. The correct way to do it would be with package.env like this:

echo 'FEATURES="test"' > /etc/portage/env/features_test.conf
echo "net-misc/youtube-dl features_test.conf" >> /etc/portage/package.env
Comment 3 Jeroen Roovers (RETIRED) gentoo-dev 2013-06-28 16:13:54 UTC
No, I don't use bashrc for that.

# cat /etc/portage/env/net-misc/youtube-dl
FEATURES="${FEATURES} test"
Comment 4 Zac Medico gentoo-dev 2013-06-28 16:23:37 UTC
(In reply to Jeroen Roovers from comment #3)
> No, I don't use bashrc for that.
> 
> # cat /etc/portage/env/net-misc/youtube-dl
> FEATURES="${FEATURES} test"

That is bashrc. It's a common source of confusion. You need to use package.env exactly as I've described.
Comment 5 Jeroen Roovers (RETIRED) gentoo-dev 2013-06-28 16:33:23 UTC
(In reply to Zac Medico from comment #4)
> (In reply to Jeroen Roovers from comment #3)
> > No, I don't use bashrc for that.
> > 
> > # cat /etc/portage/env/net-misc/youtube-dl
> > FEATURES="${FEATURES} test"
> 
> That is bashrc. It's a common source of confusion. You need to use
> package.env exactly as I've described.

Oh. Maybe portage(5) should be updated? :)

Apparently package.env is loaded during depcalc and such, and /etc/portage/env is only sources during build phases? Both configuration mechanisms using /etc/portage/env isn't helping here.

       /etc/portage/env/
              package-specific bashrc files
[...]
                     Additional package-specific bashrc files can be created in /etc/portage/env.
[...]
              package.env
                     Per-package environment variable settings. Entries refer to environment files that are placed  in
                     the /etc/portage/env/ directory and have the same format as make.conf(5).

                     Format:
                     - comment lines begin with # (no inline comments)
                     - one DEPEND atom per line followed by name(s) of environment file(s)

                     Example:
                     # use environment variables from /etc/portage/env/glibc.conf for the glibc package
                     sys-libs/glibc glibc.conf
Comment 6 Jeroen Roovers (RETIRED) gentoo-dev 2013-06-28 16:37:06 UTC
Or maybe emerge should not source these files as if they were bash scripts? I don't know.
Comment 7 Zac Medico gentoo-dev 2013-06-28 16:58:18 UTC
(In reply to Jeroen Roovers from comment #5)
> Oh. Maybe portage(5) should be updated? :)

See bug 388781.

> Apparently package.env is loaded during depcalc and such, and
> /etc/portage/env is only sources during build phases?

Right.

> Both configuration
> mechanisms using /etc/portage/env isn't helping here.

Yeah I know. I merged the package.env patches, people assured me that there wouldn't be any confusion because of that.

(In reply to Jeroen Roovers from comment #6)
> Or maybe emerge should not source these files as if they were bash scripts?
> I don't know.

It would be a little late to not source those files as bash scripts, since people have been relying on it for years. Actually, the bashrc thing was originally implemented in profiles/base/profile.bashrc a long time ago, and the code has since been migrated into portage itself.

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