Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 467216 - sys-apps/pv: Replace sys-apps/usleep with app-admin/killproc
Summary: sys-apps/pv: Replace sys-apps/usleep with app-admin/killproc
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Jeroen Roovers (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 467212
  Show dependency tree
 
Reported: 2013-04-25 14:36 UTC by Samuli Suominen (RETIRED)
Modified: 2013-04-25 15:33 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 Samuli Suominen (RETIRED) gentoo-dev 2013-04-25 14:36:08 UTC
I made a stupid mistake by adding the separate usleep command to Portage. I didn't realize the correct usleep command is already in the tree as part of the app-admin/killproc suite.

Please fix pv's ebuild to reference app-admin/killproc instead of sys-apps/usleep so I can remove usleep from portage.

Bug 467212
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2013-04-25 15:33:57 UTC
I've just gone and removed the faux dependency altogether. First of all, no one knows that installing something providing usleep will make a certain test in the pv package run slightly faster. I could add a notice in src_test() to make that clear, but without a usleep command, instead of a fifth of a second, the test will now wait a full second.

Instead of 3 seconds, tests/019-remote-cksum now takes 15 seconds. That's excluding the initial 2 second nap before that particular test even begins, so overall, useless sleep has gone up from 5 seconds to 17 seconds. :)