Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 262096 - provide elog binaries for programs like sandbox to call
Summary: provide elog binaries for programs like sandbox to call
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Sandbox (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Portage team
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks: 537128 210077 261959
  Show dependency tree
 
Reported: 2009-03-11 09:58 UTC by SpanKY
Modified: 2015-01-25 09:56 UTC (History)
2 users (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description SpanKY gentoo-dev 2009-03-11 09:58:19 UTC 
if the log used by elog() was available in an env var, then sandbox could append QA notices to it

atm i'm dumping to stderr, but that sometimes breaks things (Bug 261957)

so i'm changing it to default to /dev/tty, but a default of elog would be better
Comment 1 Zac Medico gentoo-dev 2009-03-11 20:13:57 UTC 
It would be a lot safer to make the e* functions into binaries that sandbox could call. That way portage can safely change the log file format and sandbox won't have to know the log file format.
Comment 2 SpanKY gentoo-dev 2009-03-11 22:22:57 UTC 
true ... tbh, i wasnt going to respect the log file format in the first place ;)
Comment 3 Zac Medico gentoo-dev 2009-03-12 01:47:35 UTC 
I've added binaries for eerror, einfo, elog, eqawarn, and ewarn.
Comment 4 SpanKY gentoo-dev 2009-03-12 02:24:55 UTC 
cool ... once the next version of portage is released, i'll change sandbox to first try running eqawarn.  if that fails, i'll call back to writing to /dev/tty.
Comment 5 Zac Medico gentoo-dev 2009-03-12 10:11:53 UTC 
This is fixed in 2.2_rc25 and it will also be in 2.1.6.8 when it's released.