Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 467216

Summary: sys-apps/pv: Replace sys-apps/usleep with app-admin/killproc
Product: Gentoo Linux Reporter: Samuli Suominen (RETIRED) <ssuominen>
Component: Current packagesAssignee: Jeroen Roovers (RETIRED) <jer>
Status: RESOLVED FIXED    
Severity: normal    
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 467212    

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. :)