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.
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.)