FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Gentoo > Gentoo Development

 
 
LinkBack Thread Tools
 
Old 02-26-2010, 08:02 PM
Samuli Suominen
 
Default gentoo-x86 commit in app-forensics/zzuf: ChangeLog zzuf-0.13.ebuild

On 02/26/2010 10:50 PM, Patrick Lauer (patrick) wrote:
> patrick 10/02/26 20:50:29
>
> Modified: ChangeLog
> Added: zzuf-0.13.ebuild
> Log:
> Bump
> (Portage version: 2.2_rc63/cvs/Linux x86_64)
>
> 1.1 app-forensics/zzuf/zzuf-0.13.ebuild
>
> file : http://sources.gentoo.org/viewcvs.py/gentoo-x86/app-forensics/zzuf/zzuf-0.13.ebuild?rev=1.1&view=markup
> plain: http://sources.gentoo.org/viewcvs.py/gentoo-x86/app-forensics/zzuf/zzuf-0.13.ebuild?rev=1.1&content-type=text/plain
>
> Index: zzuf-0.13.ebuild
> ================================================== =================
> # Copyright 1999-2010 Gentoo Foundation
> # Distributed under the terms of the GNU General Public License v2
> # $Header: /var/cvsroot/gentoo-x86/app-forensics/zzuf/zzuf-0.13.ebuild,v 1.1 2010/02/26 20:50:28 patrick Exp $
>

[ .. ]

> src_test() {
> if hasq sandbox ${FEATURES}; then
> ewarn "zzuf tests don't work correctly when sandbox is enabled,"
> ewarn "skipping tests. If you want to run the testsuite, please"
> ewarn "disable sandbox for this build."
> return
> fi

Testing FEATURES from ebuild? You shouldn't do that.
 
Old 02-26-2010, 09:48 PM
Patrick Lauer
 
Default gentoo-x86 commit in app-forensics/zzuf: ChangeLog zzuf-0.13.ebuild

On 02/26/10 22:02, Samuli Suominen wrote:
> On 02/26/2010 10:50 PM, Patrick Lauer (patrick) wrote:

>> src_test() {
>> if hasq sandbox ${FEATURES}; then
>> ewarn "zzuf tests don't work correctly when sandbox is enabled,"
>> ewarn "skipping tests. If you want to run the testsuite, please"
>> ewarn "disable sandbox for this build."
>> return
>> fi
>
> Testing FEATURES from ebuild? You shouldn't do that.

I disagree. That's a good way not to fail there, unless someone has a
better idea how to make that work.

And I'd appreciate it if PMS would stop refusing to document FEATURES.
(Double negative? I mean: PMS should document reality)

I won't mind if someone "fixes" that in a way that still has the same
functionality, but I honestly don't see it as a bug, so I'll leave it as
it is.
 
Old 02-26-2010, 09:52 PM
Samuli Suominen
 
Default gentoo-x86 commit in app-forensics/zzuf: ChangeLog zzuf-0.13.ebuild

On 02/27/2010 12:48 AM, Patrick Lauer wrote:
> On 02/26/10 22:02, Samuli Suominen wrote:
>> On 02/26/2010 10:50 PM, Patrick Lauer (patrick) wrote:
>
>>> src_test() {
>>> if hasq sandbox ${FEATURES}; then
>>> ewarn "zzuf tests don't work correctly when sandbox is enabled,"
>>> ewarn "skipping tests. If you want to run the testsuite, please"
>>> ewarn "disable sandbox for this build."
>>> return
>>> fi
>>
>> Testing FEATURES from ebuild? You shouldn't do that.
>
> I disagree. That's a good way not to fail there, unless someone has a
> better idea how to make that work.
>
> And I'd appreciate it if PMS would stop refusing to document FEATURES.
> (Double negative? I mean: PMS should document reality)
>
> I won't mind if someone "fixes" that in a way that still has the same
> functionality, but I honestly don't see it as a bug, so I'll leave it as
> it is.
>

http://bugs.gentoo.org/show_bug.cgi?id=279465
http://bugs.gentoo.org/show_bug.cgi?id=174335

Even this is better,

RESTRICT="test" # Doesn't work with sandbox enabled, see bug 279465

My point is simple: If you don't like the situation / policy, try to
change it, don't go solo with singular ebuild.
 
Old 02-27-2010, 07:31 AM
Ciaran McCreesh
 
Default gentoo-x86 commit in app-forensics/zzuf: ChangeLog zzuf-0.13.ebuild

On Fri, 26 Feb 2010 23:48:32 +0100
Patrick Lauer <patrick@gentoo.org> wrote:
> And I'd appreciate it if PMS would stop refusing to document FEATURES.
> (Double negative? I mean: PMS should document reality)

PMS does document reality: FEATURES can't be used by ebuilds since its
values aren't reliable. Tests have to be made for the thing that
actually causes the problem, not a variable that indicates that the
user requested but may not have been given the thing that causes the
problem.

But then, we've had this discussion before, and you already know that,
so you're just trying to stir up trouble again.

--
Ciaran McCreesh
 

Thread Tools




All times are GMT. The time now is 07:21 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org