Bug 324191

Summary:	new portage option: FEATURES=compress-build-logs
Product:	Portage Development	Reporter:	Zac Medico <zmedico>
Component:	Core - Configuration	Assignee:	Portage team <dev-portage>
Status:	RESOLVED FIXED
Severity:	normal	CC:	dhp_gentoo, esigra, flameeyes, Liganic, toralf
Priority:	High	Keywords:	InVCS
Version:	2.2
Hardware:	All
OS:	All
See Also:	https://bugs.gentoo.org/show_bug.cgi?id=554240
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	335925

Description Zac Medico gentoo-dev

2010-06-16 00:19:33 UTC

The idea is to compress build logs as they are written, in order to facilitate the publishing of compressed tinderbox logs on a web server. Since the portage code opens the build log many times in many places, it's not very convenient to pass an open file descriptor to all those places. However, it's possible to open the file in append mode with the gzip.GzipFile() class. A helper function can be used to open the file based on the current FEATURES settings.

Comment 1 SpanKY gentoo-dev

2010-06-16 00:38:47 UTC

probably best to not encode a compression schema in the feature name

Comment 2 DEMAINE Benoît-Pierre, aka DoubleHP 2010-06-20 12:30:32 UTC

(In reply to comment #1)
> probably best to not encode a compression schema in the feature name
> 

I agree; otherwise, we could end up in collision. One feature for compression, and one extra variable for the algo (default to gz please, to easily use zcat and zgrep).

Comment 3 DEMAINE Benoît-Pierre, aka DoubleHP 2010-06-20 12:34:57 UTC

Different approach proposed in bug 324729 .

Comment 4 Sebastian Luther (few) 2010-07-13 08:13:17 UTC

*** Bug 328063 has been marked as a duplicate of this bug. ***

Comment 5 Zac Medico gentoo-dev

2010-08-21 18:30:00 UTC

This is in git now:

http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=42cab46428e69cf7423053b10940a275c38c04ff

Comment 6 Zac Medico gentoo-dev

2010-08-23 06:24:55 UTC

This is in 2.2_rc68, but I'll leave this bug open until it's in an unmasked version.

Comment 7 Zac Medico gentoo-dev

2010-09-04 08:32:58 UTC

This is fixed in 2.1.9.