Skip to content
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

Fix a bug where substitutions were not done for runtime. #754

Merged
merged 1 commit into from
Oct 16, 2023

Conversation

vaikas
Copy link
Member

@vaikas vaikas commented Oct 13, 2023

Melange Pull Request Template

Fix a bug where substitutions were not done for runtime.

Functional Changes

  • This change can build all of Wolfi without errors (describe results in notes)

Notes:

SCA Changes

  • Examining several representative APKs show no regression / the desired effect (details in notes)

Notes:

Linter

  • The new check is clean across Wolfi
  • The new check is opt-in or a warning

Notes:

@vaikas vaikas merged commit 31c7281 into chainguard-dev:main Oct 16, 2023
26 checks passed
@vaikas vaikas deleted the package-version-fix branch October 16, 2023 16:02
vaikas added a commit to vaikas/os that referenced this pull request Oct 17, 2023
*DO NOT MERGE UNTIL chainguard-dev/melange#754 goes in.
*****

After installing with apk add php-8.3 also installs the php-8.3-config

```
22ea88231f3b:/work/packages# apk add ./aarch64/php-8.3-8.3_rc3-r14.apk
(1/8) Installing xz (5.4.4-r0)
(2/8) Installing libxml2 (2.11.5-r1)
(3/8) Installing php-8.3-config (8.3_rc3-r14)
(4/8) Installing ncurses-terminfo-base (6.4_p20230722-r1)
(5/8) Installing ncurses (6.4_p20230722-r1)
(6/8) Installing readline (8.2-r2)
(7/8) Installing sqlite (3.40.0-r1)
(8/8) Installing php-8.3 (8.3_rc3-r14)
OK: 32 MiB in 27 packages

22ea88231f3b:/work/packages# apk info -L php-8.3-config
php-8.3-config-8.3_rc3-r14 contains:
etc/php/php.ini
var/lib/db/sbom/php-8.3-config-8.3_rc3-r14.spdx.json

22ea88231f3b:/work/packages# apk info -L php-8.3
php-8.3-8.3_rc3-r14 contains:
bin/php
usr/bin/phar
usr/bin/phar.phar
usr/bin/php
usr/share/man/man1/phar.1
usr/share/man/man1/phar.phar.1
usr/share/man/man1/php-cgi.1
usr/share/man/man1/php-config.1
usr/share/man/man1/php.1
usr/share/man/man1/phpdbg.1
usr/share/man/man1/phpize.1
usr/share/man/man8/php-fpm.8
usr/share/php/fpm/status.html
var/lib/db/sbom/php-8.3-8.3_rc3-r14.spdx.json
```

Spot checking an extension:
```
22ea88231f3b:/work/packages# apk add ./aarch64/php-8.3-gettext-8.3_rc3-r14.apk
(1/2) Installing php-8.3-gettext-config (8.3_rc3-r14)
(2/2) Installing php-8.3-gettext (8.3_rc3-r14)

22ea88231f3b:/work/packages# apk info -L php-8.3-gettext
php-8.3-gettext-8.3_rc3-r14 contains:
usr/lib/php/modules/gettext.so
var/lib/db/sbom/php-8.3-gettext-8.3_rc3-r14.spdx.json

22ea88231f3b:/work/packages# apk info -L php-8.3-gettext-config
php-8.3-gettext-config-8.3_rc3-r14 contains:
etc/php/conf.d/gettext.ini
var/lib/db/sbom/php-8.3-gettext-config-8.3_rc3-r14.spdx.json
```

Signed-off-by: Ville Aikas <[email protected]>
github-merge-queue bot pushed a commit to wolfi-dev/os that referenced this pull request Oct 17, 2023
vaikas added a commit to vaikas/os that referenced this pull request Oct 17, 2023
*DO NOT MERGE UNTIL chainguard-dev/melange#754 goes in.
*****

After installing with apk add php-8.3 also installs the php-8.3-config

```
22ea88231f3b:/work/packages# apk add ./aarch64/php-8.3-8.3_rc3-r14.apk
(1/8) Installing xz (5.4.4-r0)
(2/8) Installing libxml2 (2.11.5-r1)
(3/8) Installing php-8.3-config (8.3_rc3-r14)
(4/8) Installing ncurses-terminfo-base (6.4_p20230722-r1)
(5/8) Installing ncurses (6.4_p20230722-r1)
(6/8) Installing readline (8.2-r2)
(7/8) Installing sqlite (3.40.0-r1)
(8/8) Installing php-8.3 (8.3_rc3-r14)
OK: 32 MiB in 27 packages

22ea88231f3b:/work/packages# apk info -L php-8.3-config
php-8.3-config-8.3_rc3-r14 contains:
etc/php/php.ini
var/lib/db/sbom/php-8.3-config-8.3_rc3-r14.spdx.json

22ea88231f3b:/work/packages# apk info -L php-8.3
php-8.3-8.3_rc3-r14 contains:
bin/php
usr/bin/phar
usr/bin/phar.phar
usr/bin/php
usr/share/man/man1/phar.1
usr/share/man/man1/phar.phar.1
usr/share/man/man1/php-cgi.1
usr/share/man/man1/php-config.1
usr/share/man/man1/php.1
usr/share/man/man1/phpdbg.1
usr/share/man/man1/phpize.1
usr/share/man/man8/php-fpm.8
usr/share/php/fpm/status.html
var/lib/db/sbom/php-8.3-8.3_rc3-r14.spdx.json
```

Spot checking an extension:
```
22ea88231f3b:/work/packages# apk add ./aarch64/php-8.3-gettext-8.3_rc3-r14.apk
(1/2) Installing php-8.3-gettext-config (8.3_rc3-r14)
(2/2) Installing php-8.3-gettext (8.3_rc3-r14)

22ea88231f3b:/work/packages# apk info -L php-8.3-gettext
php-8.3-gettext-8.3_rc3-r14 contains:
usr/lib/php/modules/gettext.so
var/lib/db/sbom/php-8.3-gettext-8.3_rc3-r14.spdx.json

22ea88231f3b:/work/packages# apk info -L php-8.3-gettext-config
php-8.3-gettext-config-8.3_rc3-r14 contains:
etc/php/conf.d/gettext.ini
var/lib/db/sbom/php-8.3-gettext-config-8.3_rc3-r14.spdx.json
```

Signed-off-by: Ville Aikas <[email protected]>
vaikas added a commit to vaikas/os that referenced this pull request Oct 17, 2023
*DO NOT MERGE UNTIL chainguard-dev/melange#754 goes in.
*****

After installing with apk add php-8.3 also installs the php-8.3-config

```
22ea88231f3b:/work/packages# apk add ./aarch64/php-8.3-8.3_rc3-r14.apk
(1/8) Installing xz (5.4.4-r0)
(2/8) Installing libxml2 (2.11.5-r1)
(3/8) Installing php-8.3-config (8.3_rc3-r14)
(4/8) Installing ncurses-terminfo-base (6.4_p20230722-r1)
(5/8) Installing ncurses (6.4_p20230722-r1)
(6/8) Installing readline (8.2-r2)
(7/8) Installing sqlite (3.40.0-r1)
(8/8) Installing php-8.3 (8.3_rc3-r14)
OK: 32 MiB in 27 packages

22ea88231f3b:/work/packages# apk info -L php-8.3-config
php-8.3-config-8.3_rc3-r14 contains:
etc/php/php.ini
var/lib/db/sbom/php-8.3-config-8.3_rc3-r14.spdx.json

22ea88231f3b:/work/packages# apk info -L php-8.3
php-8.3-8.3_rc3-r14 contains:
bin/php
usr/bin/phar
usr/bin/phar.phar
usr/bin/php
usr/share/man/man1/phar.1
usr/share/man/man1/phar.phar.1
usr/share/man/man1/php-cgi.1
usr/share/man/man1/php-config.1
usr/share/man/man1/php.1
usr/share/man/man1/phpdbg.1
usr/share/man/man1/phpize.1
usr/share/man/man8/php-fpm.8
usr/share/php/fpm/status.html
var/lib/db/sbom/php-8.3-8.3_rc3-r14.spdx.json
```

Spot checking an extension:
```
22ea88231f3b:/work/packages# apk add ./aarch64/php-8.3-gettext-8.3_rc3-r14.apk
(1/2) Installing php-8.3-gettext-config (8.3_rc3-r14)
(2/2) Installing php-8.3-gettext (8.3_rc3-r14)

22ea88231f3b:/work/packages# apk info -L php-8.3-gettext
php-8.3-gettext-8.3_rc3-r14 contains:
usr/lib/php/modules/gettext.so
var/lib/db/sbom/php-8.3-gettext-8.3_rc3-r14.spdx.json

22ea88231f3b:/work/packages# apk info -L php-8.3-gettext-config
php-8.3-gettext-config-8.3_rc3-r14 contains:
etc/php/conf.d/gettext.ini
var/lib/db/sbom/php-8.3-gettext-config-8.3_rc3-r14.spdx.json
```

Signed-off-by: Ville Aikas <[email protected]>
vaikas added a commit to vaikas/os that referenced this pull request Oct 17, 2023
*DO NOT MERGE UNTIL chainguard-dev/melange#754 goes in.
*****

After installing with apk add php-8.3 also installs the php-8.3-config

```
22ea88231f3b:/work/packages# apk add ./aarch64/php-8.3-8.3_rc3-r14.apk
(1/8) Installing xz (5.4.4-r0)
(2/8) Installing libxml2 (2.11.5-r1)
(3/8) Installing php-8.3-config (8.3_rc3-r14)
(4/8) Installing ncurses-terminfo-base (6.4_p20230722-r1)
(5/8) Installing ncurses (6.4_p20230722-r1)
(6/8) Installing readline (8.2-r2)
(7/8) Installing sqlite (3.40.0-r1)
(8/8) Installing php-8.3 (8.3_rc3-r14)
OK: 32 MiB in 27 packages

22ea88231f3b:/work/packages# apk info -L php-8.3-config
php-8.3-config-8.3_rc3-r14 contains:
etc/php/php.ini
var/lib/db/sbom/php-8.3-config-8.3_rc3-r14.spdx.json

22ea88231f3b:/work/packages# apk info -L php-8.3
php-8.3-8.3_rc3-r14 contains:
bin/php
usr/bin/phar
usr/bin/phar.phar
usr/bin/php
usr/share/man/man1/phar.1
usr/share/man/man1/phar.phar.1
usr/share/man/man1/php-cgi.1
usr/share/man/man1/php-config.1
usr/share/man/man1/php.1
usr/share/man/man1/phpdbg.1
usr/share/man/man1/phpize.1
usr/share/man/man8/php-fpm.8
usr/share/php/fpm/status.html
var/lib/db/sbom/php-8.3-8.3_rc3-r14.spdx.json
```

Spot checking an extension:
```
22ea88231f3b:/work/packages# apk add ./aarch64/php-8.3-gettext-8.3_rc3-r14.apk
(1/2) Installing php-8.3-gettext-config (8.3_rc3-r14)
(2/2) Installing php-8.3-gettext (8.3_rc3-r14)

22ea88231f3b:/work/packages# apk info -L php-8.3-gettext
php-8.3-gettext-8.3_rc3-r14 contains:
usr/lib/php/modules/gettext.so
var/lib/db/sbom/php-8.3-gettext-8.3_rc3-r14.spdx.json

22ea88231f3b:/work/packages# apk info -L php-8.3-gettext-config
php-8.3-gettext-config-8.3_rc3-r14 contains:
etc/php/conf.d/gettext.ini
var/lib/db/sbom/php-8.3-gettext-config-8.3_rc3-r14.spdx.json
```

Signed-off-by: Ville Aikas <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants