Skip to content
This repository has been archived by the owner on Aug 15, 2024. It is now read-only.

Update ferros/buffer to version 5 #68

Closed
wants to merge 3 commits into from
Closed

Conversation

venik
Copy link

@venik venik commented Aug 30, 2017

Hi,

Sorry previous PR #67 contained wrong email. Is it possible to update ferros/buffer to version 5? Problem is that versions 4.X refers to global

Buffer.TYPED_ARRAY_SUPPORT = global.TYPED_ARRAY_SUPPORT !== undefined
  ? global.TYPED_ARRAY_SUPPORT
  : typedArraySupport()

which leads to #60 and #31

this patch has fixed this problem and version 5.X works just fine.

We used shrinkwrap to work it around, but would be better to have updated version of the node-libs-browser (npm-shrinkwrap.json):

{
  "dependencies": {
    "webpack": {
      "version": "3.3.0",
      "dependencies": {
        "node-libs-browser": {
          "version": "2.0.0",
          "dependencies": {
            "buffer": {
              "version": "5.0.7"
            }
          }
        }
      }
    }
  }
}

Thanks in advance!

@jsf-clabot
Copy link

jsf-clabot commented Aug 30, 2017

CLA assistant check
All committers have signed the CLA.

@jhnns
Copy link
Member

jhnns commented Nov 15, 2017

This is a breaking change, see feross/buffer@5daca86#commitcomment-19698936. We will pull this in with the next major version bump. Users that still need to support IE9 and IE10 can use a polyfill.

@venik
Copy link
Author

venik commented Nov 15, 2017

@jhnns thx for the reply!

@blikblum
Copy link

IMO this is the right time to upgrade buffer.

Currently IE9 has a market share of 0.12% and IE10 0.11% (see https://caniuse.com/usage-table)

Within six month or so, when next webpack, browserify versions are released, these browsers will be even more irrelevant giving the trend

Postponing the upgrade can loose the opportunity to be shipped in bundlers next releases

@igrek8
Copy link

igrek8 commented Jul 14, 2018

Buffer v4 fails at simple case Buffer.alloc(1, !!true), but works in any nodejs env. I think it is time to move forward and update the dependency

@blikblum
Copy link

Bump. This is the time to upgrade

@flipflopsimsommer
Copy link

any plans when this will be merged?

@mvasilkov
Copy link

Hi, can we please integrate this?

@sokra
Copy link
Member

sokra commented Jun 12, 2019

This library is deprecated and won't accept Pull Requests that include Breaking Changes or new Features. Only bugfixes are accepted.

We can't add a breaking change to webpack 4.

webpack 5 will no longer use node-libs-browser, but allows you to use any buffer version.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants