256379 – app-text/scrollkeeper should install /etc/sandbox.d/ entry for /var/log/scrollkeeper.log

Bug 256379 - app-text/scrollkeeper should install /etc/sandbox.d/ entry for /var/log/scrollkeeper.log

Summary: app-text/scrollkeeper should install /etc/sandbox.d/ entry for /var/log/scrol...

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	[OLD] GNOME (show other bugs)
Hardware:	All All

Importance:	High normal (vote)
Assignee:	Gentoo Linux Gnome Desktop Team

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:	gnome2.eclass
	Show dependency tree

Reported:	2009-01-26 05:49 UTC by SpanKY
Modified:	2011-05-06 13:18 UTC (History)
CC List:	1 user (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-01-26 05:49:09 UTC

the scollkeeper ebuild should create a file in /etc/sandbox.d/ with an entry like so:
SANDBOX_WRITE="/var/log/scrollkeeper.log"

unless of course the sandbox issues with this util being stupid while running in an ebuild have been resolved ...

Comment 1 Gilles Dartiguelongue (RETIRED) gentoo-dev

2009-02-22 21:53:15 UTC

@gnome, should we get rid of gnome2_omf_fix in gnome2-utils.eclass and use this solution instead ? I think that since we now use rarian in place of scrollkeeper the fix is actually useless or will be when we kill scrollkeeper ebuilds.

Comment 2 Mart Raudsepp gentoo-dev

2009-06-19 15:55:26 UTC

well, I don't think scrollkeeper should be ran at all when in phases that are sandboxed as that means it's doing something it shouldn't (machine specific updating of stuff while still in the page of prepping the image dir which leads to a binary package as well potentially), so I don't think we'd want such a /etc/sandbox.d entry, do we?

Comment 3 SpanKY gentoo-dev

2009-06-19 21:34:29 UTC

sorry, i should have included a little bit of background in case it wasnt obvious ...

previous sandbox versions had all been including a write entry for /var/log/scrollkeeper.log.  ive dropped it in the latest versions.  packages that need these kind of things should be creating sandbox.d entries rather than pushing it into the sandbox package itself.

i would absolutely prefer to not have scrollkeeper create any sandbox entries, but you guys are familiar with the exact issues at hand here, so i defer to you.

Comment 4 Pacho Ramos gentoo-dev

2011-01-27 14:00:05 UTC

What should we do with this? Looks like scrollkeeper is not being used for a lot of time

Comment 5 SpanKY gentoo-dev

2011-01-30 00:37:50 UTC

then mask + punt the scrollkeeper ebuild ?

Comment 6 Pacho Ramos gentoo-dev

2011-04-14 18:04:37 UTC

(In reply to comment #5)
> then mask + punt the scrollkeeper ebuild ?

Dropping app-text/scrollkeeper-0.3.14-r2 should be enough, no?

Comment 7 Pacho Ramos gentoo-dev

2011-05-06 13:18:44 UTC

+  06 May 2011; Pacho Ramos <pacho@gentoo.org> -scrollkeeper-0.3.14-r2.ebuild,
+  -files/scrollkeeper-0.3.14-gcc2_fix.patch,
+  -files/scrollkeeper-0.3.14-gentoo.patch,
+  -files/scrollkeeper-0.3.14-nls.patch, -scrollkeeper-9999.ebuild,
+  scrollkeeper-9999-r1.ebuild, -files/scrollkeeper-logrotate:
+  As talked with Gilles days ago, drop old versions as they are no needed and
+  it allows us to finally solve bug #256379 (I have tried to mail and contact
+  m68k team without success). Also make repoman happier.
+