[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#761442: marked as done (ITP: node-readable-stream -- Streams2, user-land copy of the stream library from Node.js v0.10)

Your message dated Fri, 13 Mar 2015 22:13:53 +0100
with message-id <55035311.5010302@xs4all.nl>
and subject line Re: Bug#761442: ITP: node-readable-stream -- Streams2, user-land copy of the stream library from Node.js v0.10
has caused the Debian Bug report #761442,
regarding ITP: node-readable-stream -- Streams2, user-land copy of the stream library from Node.js v0.10
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
761442: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=761442
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: wnpp
Severity: wishlist

* Package name    : node-readable-stream
  Version         : 1.1.13
  Upstream Author : Isaac Z. Schlueter <i@izs.me>
* URL             : https://github.com/isaacs/readable-stream/
* License         : MIT/X
  Programming Lang: JavaScript
  Description     : a user-land copy of the stream library from Node.js v0.11.x

The readme says:

    This package is a mirror of the Streams2 and Streams3
    implementations in Node-core.

    If you want to guarantee a stable streams base, regardless
    of what version of Node you, or the users of your libraries
    are using, use readable-stream only and avoid the "stream"
    module in Node-core.

This is declared as dependency in node-htmlparser2, but I'm not
sure, whether it is really needed or could be patched out.

--- End Message ---
--- Begin Message --- 
noowner 761442
thanks

On 03/13/2015 09:39 PM, Andrew Kelley wrote:
> Don't package this, instead patch the module to require('stream') instead
> of require('readable-stream') since Debian has Node.js v0.10.x which has
> streams2 built in.

Having looked further into this issue, there is indeed no need for the
readable-stream module.

decompress-zip checks if the stream module is readable, and if not falls
back to the readable-stream module.

The other packages in the openlayers dependency chain that depend on
readable-stream (node-bl & node-htmlparser2) are already in the archive
and likewise don't require the readable-stream module.

I'm closing this issue since the stream module in the nodejs is sufficient.

Kind Regards,

Bas

-- 
 GPG Key ID: 4096R/6750F10AE88D4AF1
Fingerprint: 8182 DE41 7056 408D 6146  50D1 6750 F10A E88D 4AF1

--- End Message ---
Reply to: