-
-
Notifications
You must be signed in to change notification settings - Fork 13
src: Update Electron header download URL #43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The old atom.io Electron headers URL is going down, per the updated sunset announcement. Likewise, the old gh-contractor-zcbenz Amazon AWS/S3 bucket is no-longer guaranteed to be active, and may go away at any time. (See: https://github.blog/2022-06-08-sunsetting-atom/ and https://www.electronjs.org/blog/s3-bucket-change) So, this commit updates our package manager to get the Electron headers from the new 'artifacts.electronjs.org/headers/dist' location. This is the correct, official place to get the Electron headers, according to the blog post above at electronjs.org. Ensures we can continue to build native C/C++ code for certain Atom/Pulsar packages that use it. (For example: several of the language packages have C/C++ addon code.)
|
By the way: tests are failing here for the same reason they are failing on See #42 for the fix to that. Getting somewhat off-topic for this present PR, to talk about #42 (click to expand):These test failures have to do with the Node 16 bump, and I think it specifically has to do with trying to use old node-gyp to build C/C++ code for older Node (spec fixture version of Node is 10.12.1) while running much newer Node 16.0.0. I think newer node-gyp has fixes for this, by the way. But we can't really take advantage of that without also bumping the bundled npm, since our bundled npm 6 specifies the older node-gyp in its own package.json.) |
confused-Techie
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for lack of approval, lets get this merged
|
Copy-pasting from Discord, testing notes:
Best way I know of to test this is to use the changed code to install a Pulsar package that includes a native C/C++ module. For example, I run The |
Includes updated Electron headers download URL. (See: pulsar-edit/ppm#43)
Includes updated Electron headers download URL. (See: pulsar-edit/ppm#43)
The old atom.io Electron headers URL is going down, per the updated sunset announcement.
Likewise, the old gh-contractor-zcbenz Amazon AWS/S3 bucket is no-longer guaranteed to be active, and may go away at any time.
(See: https://github.blog/2022-06-08-sunsetting-atom/#if-im-using-atom-what-changes-can-i-expect-after-the-sunset and https://www.electronjs.org/blog/s3-bucket-change)
So, this commit updates our package manager to get the Electron headers from the new 'artifacts.electronjs.org/headers/dist' location.
This is the correct, official place to get the Electron headers, according to the blog post above at electronjs.org.
Ensures we can continue to build native C/C++ code for certain Atom/Pulsar packages that use it.
(For example: several of the language packages have C/C++ addon code.)