Skip to content

Conversation

malice00
Copy link
Collaborator

@malice00 malice00 commented Sep 9, 2025

PHP and ruby were not in the SBOM, because they were not working in the workflows. I made some changes and added them back now:

  • Changed the images for the linux build to php-images
  • Installed composer for Darwin and Linux
  • Fixed the call that checks for composer on Windows

@malice00 malice00 requested a review from prabhu as a code owner September 9, 2025 19:13
@malice00 malice00 added bug Something isn't working build Build, action or package manager changes labels Sep 9, 2025
apt install -y curl
- libc: musl
image: alpine:3.16
image: php:8.4-alpine3.21
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this make our musl builds >= 3.21?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should not, but will check when I download a binary

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Runs just fine on 3.16. I believe it's the node binary that counts, which is still produced on an older images. I don't think the choice of image really influences the binary that is getting created, tbh.

Copy link
Collaborator

@prabhu prabhu Sep 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about debian 12? I am surprised it will run on debian 11 since node-sqlite3 would be built against the latest glibc?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I changed back to debian 11 in the latest commit. Not sure what you mean though, we've been using that for quite some time?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

my bad. didn't see the change.

@malice00 malice00 force-pushed the binary_builds/types branch from 26fd0d8 to faee040 Compare September 9, 2025 21:30
@malice00 malice00 merged commit 938a63c into master Sep 9, 2025
80 checks passed
@malice00 malice00 deleted the binary_builds/types branch September 9, 2025 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working build Build, action or package manager changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants