Gentoo Logo
Gentoo Spaceship




Note: Due to technical difficulties, the Archives are currently not up to date. GMANE provides an alternative service for most mailing lists.
c.f. bug 424647
List Archive: gentoo-dev
Navigation:
Lists: gentoo-dev: < Prev By Thread Next > < Prev By Date Next >
Headers:
To: gentoo-dev@g.o
From: "Paweł Hajdan, Jr." <phajdan.jr@g.o>
Subject: RFC: trivial chromium.eclass patch
Date: Tue, 29 May 2012 10:47:08 +0200
I'd like to commit the following chromium.eclass patch.

The rationale is that checked kernel config options are not needed for
SELinux sandbox.

After that patch gets committed, I plan to modify the ebuilds in tree.
Can the deprecated function be removed immediately after that, or should
it stay longer? If the latter, how long?

It seems gentoo-dev review is not strictly needed for non-general
eclasses (<http://devmanual.gentoo.org/eclass-writing/index.html>), but
I'd just make sure I'm making those changes correctly.

Paweł
--- chromium.eclass.orig	2012-05-29 10:33:45.000000000 +0200
+++ chromium.eclass	2012-05-29 10:36:59.000000000 +0200
@@ -17,11 +17,11 @@
 	IUSE+=" custom-cflags"
 fi
 
-# @FUNCTION: chromium_check_kernel_config
+# @FUNCTION: chromium_suid_sandbox_check_kernel_config
 # @USAGE:
 # @DESCRIPTION:
-# Ensures the system kernel is configured for full Chromium functionality.
-chromium_check_kernel_config() {
+# Ensures the system kernel supports features needed for SUID sandbox to work.
+chromium_suid_sandbox_check_kernel_config() {
 	has "${EAPI:-0}" 0 1 2 3 && die "EAPI=${EAPI} is not supported"
 
 	if [[ "${MERGE_TYPE}" == "source" || "${MERGE_TYPE}" == "binary" ]]; then
@@ -34,6 +34,16 @@
 	fi
 }
 
+# @FUNCTION: chromium_check_kernel_config
+# @USAGE:
+# @DESCRIPTION:
+# Ensures the system kernel is configured for full Chromium functionality.
+chromium_check_kernel_config() {
+	eqawarn "chromium_check_kernel_config is deprecated."
+	eqawarn "Please use chromium_suid_sandbox_check_kernel_config if applicable."
+	chromium_suid_sandbox_check_kernel_config
+}
+
 # @ECLASS-VARIABLE: CHROMIUM_LANGS
 # @DEFAULT_UNSET
 # @DESCRIPTION:
Attachment:
signature.asc (OpenPGP digital signature)
Navigation:
Lists: gentoo-dev: < Prev By Thread Next > < Prev By Date Next >
Previous by thread:
RFC: Enable FEATURES="userpriv usersandbox" by default?
Next by thread:
[PATCH] games.eclass: allow EAPI=4
Previous by date:
Re: RFC: Enable FEATURES="userpriv usersandbox" by default?
Next by date:
Re: [PATCH eutils] Move remove_libtool_files() from autotools-utils for wider use.


Updated Jun 29, 2012

Summary: Archive of the gentoo-dev mailing list.

Donate to support our development efforts.

Copyright 2001-2013 Gentoo Foundation, Inc. Questions, Comments? Contact us.