Skip to content

node-gyp 5.0 breaks support for Electron versions < 4 #1778

Description

@MarshallOfSound
  • Node Version: Unrelated
  • Platform: All
  • Compiler: Unrelated
  • Module: Any

This PR --> #1670 removes support for obsolete io.js versions, this is perfectly valid however because of the way that logic was originally implemented, over on the Electron team for our 1.x/2.x/3.x releases, we had to publish our headers with the iojs prefix instead of the node prefix.

As of node-gyp@5.0 these versions no longer work and currently we still support Electron 3 so ideally this would be fixed. I've raised a simple "revent the PR that broke it" PR, if that's acceptable it would be great to merge and release that, otherwise I can craft a PR to specifically introduce support back for these headers.

Revert PR: #1777
Refs: electron/electron#18787

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions