Skip Menu |
 

This queue is for tickets about the Scalar-List-Utils CPAN distribution.

Report information
The Basics
Id: 60792
Status: resolved
Priority: 0/
Queue: Scalar-List-Utils

People
Owner: Nobody in particular
Requestors: smueller [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: 1.23_02
Fixed in: (no value)



Subject: Drop the pure Perl variant or make XS strong default?
MIME-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 861
Download (untitled) / with headers
text/plain 861b
Hi Graham, Scalar::Util and friends have been in the core since before 5.8. Additionally, almost all perl installations have access to a compiler in the unlikely case one needs to install an updated version of the module (without upgrading perl!). Because of this and the fact that the pure-Perl variant of the module does not offer the exact same features of the XS version and that this has caused a lot of grief for a lot of people in the past, I would like to suggest dropping the pure-Perl implementation altogether or at least making it very, very, very strongly discouraged. So strongly that the user would have to do edit Makefile.PL or some such to get it. I have talked to quite a few other core people via IRC about this and it seems I'm not at all alone in my assessment. Can we please remove the pure-Perl implementation? Best regards, Steffen
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-11064-1282903104-1840.60792-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 438
Download (untitled) / with headers
text/plain 438b
Oh, I forgot to add a pointer to this proposed patch for Makefile.PL which would implement what I'd consider "strong discouragement". Nicholas mentioned on IRC that the patch may have a bug or two, but I'm pointing to it as an example of what I'd consider agreeable if you would rather not lose the PP code. http://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/2010-08/msg00818.html Thanks for considering this! Best regards, Steffen
From gbarr [...] pobox.com Fri Aug 27 14: 04:12 2010
MIME-Version: 1.0 (Apple Message framework v1081)
X-Spam-Status: No, score=-8.749 tagged_above=-99.9 required=10 tests=[AWL=1.164, BAYES_00=-2.599, RCVD_IN_DNSWL_HI=-8, SPF_NEUTRAL=0.686] autolearn=ham
In-Reply-To: <rt-3.8.HEAD-11060-1282902191-410.60792-4-0 [...] rt.cpan.org>
X-Mailer: Apple Mail (2.1081)
X-Spam-Flag: NO
References: <RT-Ticket-60792 [...] rt.cpan.org> <rt-3.8.HEAD-11060-1282902191-410.60792-4-0 [...] rt.cpan.org>
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Content-Type: text/plain; charset="utf-8"
Message-ID: <B9A475D9-7049-4023-961B-66611A788162 [...] pobox.com>
X-RT-Original-Encoding: utf-8
X-Spam-Score: -8.749
Authentication-Results: hipster.bestpractical.com (amavisd-new); dkim=pass header.i= [...] pobox.com
Authentication-Results: hipster.bestpractical.com (amavisd-new); domainkeys=pass header.from=gbarr [...] pobox.com
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id CDDF2240AE1 for <cpan-bug+Scalar-List-Utils [...] hipster.bestpractical.com>; Fri, 27 Aug 2010 14:04:12 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id wW3DmyzfLaOK for <cpan-bug+Scalar-List-Utils [...] hipster.bestpractical.com>; Fri, 27 Aug 2010 14:04:07 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 454BD240ADE for <bug-Scalar-List-Utils [...] rt.cpan.org>; Fri, 27 Aug 2010 14:04:06 -0400 (EDT)
Received: (qmail 13450 invoked by uid 103); 27 Aug 2010 18:06:40 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 27 Aug 2010 18:06:40 -0000
Received: from a-pb-sasl-quonix.pobox.com (HELO sasl.smtp.pobox.com) (208.72.237.25) by 16.mx.develooper.com (qpsmtpd/0.80) with ESMTP; Fri, 27 Aug 2010 11:06:38 -0700
Received: from sasl.smtp.pobox.com (unknown [127.0.0.1]) by a-pb-sasl-quonix.pobox.com (Postfix) with ESMTP id C64E7D0818 for <bug-Scalar-List-Utils [...] rt.cpan.org>; Fri, 27 Aug 2010 14:06:34 -0400 (EDT)
Received: from a-pb-sasl-quonix. (unknown [127.0.0.1]) by a-pb-sasl-quonix.pobox.com (Postfix) with ESMTP id C26E8D0817 for <bug-Scalar-List-Utils [...] rt.cpan.org>; Fri, 27 Aug 2010 14:06:34 -0400 (EDT)
Received: from mail.goingon.net (unknown [70.85.157.106]) by a-pb-sasl-quonix.pobox.com (Postfix) with ESMTPA id 9DCF9D0816 for <bug-Scalar-List-Utils [...] rt.cpan.org>; Fri, 27 Aug 2010 14:06:34 -0400 (EDT)
Received: by mail.goingon.net (Postfix, from userid 99) id 3966D1003FE1; Fri, 27 Aug 2010 13:06:34 -0500 (CDT)
Received: from [192.168.111.4] (client-63.249.42.207.dfwtx.partnershipbroadband.com [63.249.42.207]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (No client certificate requested) by mail.goingon.net (Postfix) with ESMTP id 2AB6EFB4362 for <bug-Scalar-List-Utils [...] rt.cpan.org>; Fri, 27 Aug 2010 13:06:33 -0500 (CDT)
Delivered-To: cpan-bug+Scalar-List-Utils [...] hipster.bestpractical.com
Subject: Re: [rt.cpan.org #60792] Drop the pure Perl variant or make XS strong default?
Domainkey-Signature: a=rsa-sha1; c=nofws; d=pobox.com; h=content-type :mime-version:subject:from:in-reply-to:date :content-transfer-encoding:message-id:references:to; q=dns; s= sasl; b=uX3X+jUycNdZYOVNcLZeLFH1zNfXZP3mLZwNRnj3doc8/PwWsv/a0vyJ Nls3veIuzCnucS3JPl1YakMFPf4gyOjfNpH+U8STq2zTbQ7d47JAoLQ44Pq1FOuz qrEdIHs/aQyLuKppKO6jP2rMDgM7JrnjMAtqjTR7S/IUMyz/dqM=
Return-Path: <gbarr [...] pobox.com>
Dkim-Signature: v=1; a=rsa-sha1; c=relaxed; d=pobox.com; h=content-type :mime-version:subject:from:in-reply-to:date :content-transfer-encoding:message-id:references:to; s=sasl; bh= deIERCd0p+veyxHQu1esAeAmNwk=; b=RU1wEo72qrQ0ajjKekiyuzx/msOXOvy+ 0BFIXUWwLRJf+HI8UUr4YgPOTAX5RkDR2LIqs1Siwx8ENVGetnMI9pflBZLWExAo 5PJeQipgndIxeaUvxD1c1Ap1+vTqb2q4INgysze/YYAJMfdEvkdLgSf3icd7K9vi sQujFnWUEvQ=
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: cpan-bug+Scalar-List-Utils [...] hipster.bestpractical.com
X-RT-Mail-Extension: scalar-list-utils
X-Old-Spam-Status: No, score=-2.1 required=5.0 tests=AWL,BAYES_00,RDNS_DYNAMIC autolearn=no version=3.2.4
Date: Fri, 27 Aug 2010 13:06:32 -0500
X-Old-Spam-Level:
X-Spam-Level:
To: bug-Scalar-List-Utils [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
X-Pobox-Relay-ID: D41C42CA-B205-11DF-8FB3-030CEE7EF46B-36227416!a-pb-sasl-quonix.pobox.com
From: Graham Barr <gbarr [...] pobox.com>
RT-Message-ID: <rt-3.8.HEAD-11058-1282932408-277.60792-0-0 [...] rt.cpan.org>
Content-Length: 173
Download (untitled) / with headers
text/plain 173b
Yes I think it is time probably for the PP to go. Access to a compiler is not as bad as it used to be and as you note, it has been a core module for sometime now. Graham.
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-11058-1282932408-277.60792-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <RT-Ticket-60792 [...] rt.cpan.org> <rt-3.8.HEAD-11060-1282902191-410.60792-4-0 [...] rt.cpan.org> <B9A475D9-7049-4023-961B-66611A788162 [...] pobox.com> <rt-3.8.HEAD-11058-1282932408-277.60792-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-11062-1283023225-957.60792-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 576
Download (untitled) / with headers
text/plain 576b
Hi Graham, On Fri Aug 27 14:06:48 2010, gbarr@pobox.com wrote: Show quoted text
> Yes I think it is time probably for the PP to go. Access to a compiler > is not as bad as it used to be and as you note, it has been a core module > for sometime now.
thanks for that. This message of yours has made a lot of people happy. I just pushed a fork of the repository to github which removes the PP implementation. Tests pass on my system (with the p_ tests removed), but it was a quick removal, so you may want to double-check it. git://github.com/tsee/Scalar-List-Utils.git Best regards, Steffen
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-11062-1284477449-474.60792-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 86
::PP was dropped in Scalar-List-Utils-1.23_03.tar.gz just uploaded to CPAN for testing


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.