User tests: Successful: Unsuccessful:
build.php does not create zstd packages on macOS with default BSD tar:
In addition, the script does not stop in any environment if an error occurs, e.g. if zstd or the zip application is missing:
Building Joomla_5.3.1-Stable-Update_Package.zip... sh: 1: zip: not found
tar
and zstd
calls, working in all environments, including macOS BSD-tar.system()
command should be check for error status and the script failcp: build/fido.jwt: No such file or directory
php build/build.php --remote=5.3.1
Building Joomla_5.3.1-Stable-Full_Package.tar.zst... tar: Couldn't open zstd --ultra -22: No such file or directory
php build/build.php --remote=5.3.1 --include-bzip2
ls -l build/tmp/packages
php build/build.php --remote=5.3.1 --include-bzip2 --debug-build
ls -l build/tmp/packages
zstd
with zstdXXX
in line 653:php build/build.php --remote=5.3.1
You have to get something like:
Building Joomla_5.3.1-Stable-Full_Package.tar.zst... tar: Write error
failed.
ERROR: Command failed (127): tar --create * | zstdXXX --ultra -22 -o ../packages/Joomla_5.3.1-Stable-Full_Package.tar.zst
Output:
sh: zstdXXX: command not found
chmod
line after line 392 mkdir packages
:system('chmod 0 packages');
You have to get something like:
Building Joomla_5.3.1-Stable-Full_Package.zip ... failed.
ERROR: Command failed (15): zip -r ../packages/Joomla_5.3.1-Stable-Full_Package.zip *
Output:
zip I/O error: Permission denied
zip error: Could not create output file (../packages/Joomla_5.3.1-Stable-Full_Package.zip)
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
Status | New | ⇒ | Pending |
Category | ⇒ | Repository |
Maybe the testing instructions should be extended to test if everything works as well as before when using Linux or Windows with WSL, and by a test for some error message, e.g. when changing permissions of the "build/tmp/packages" folder just after it is created (i.e. when you see the message "Create list of changed files from git repository for version") so that there are no writing permissions, which should cause a visible error message.
@muhme Please fix the PHPCS errors reported here: https://ci.joomla.org/joomla/joomla-cms/85015/1/7
Title |
|
Labels |
Added:
bug
PR-5.3-dev
|
@richard67 Thank you for first view 👍 PHPCS is now green and test instructions are extended, do you like to test on Windows?
I have tested this item ✅ successfully on 37327e3
I have tested this item ✅ successfully on 37327e3
I've tested on Ubuntu.
Instead of using php build/build.php --remote=5.3.1
to build with the 5.3.1 tag, I've used php ./build/build.php --remote=HEAD
to build with the current local branch.
For the result without this PR, that was the 5.3-dev branch, for the result with this PR I had fetched this PR into a local branch and checked out that branch.
Test results 1. and 2.: Skipped because not a Mac.
3.a and b: Success. Error message is shown, and build script stops.
4.: I've unpacked and verified ALL packages and compared with and without this PR.
The packages show only the usual differences which you also get when running 2 builds on the same branch, e.g. 5.3-dev.
@tecpromotion I assume you have tested only with Mac, right? Or have you also tested with WSL?
@muhme I think we should get another test with WSL, then we are safe.
@tecpromotion I assume you have tested only with Mac, right? Or have you also tested with WSL?
@muhme I think we should get another test with WSL, then we are safe.
Yes, Mac only.
@tecpromotion and @richard67 Thanks for testing 👍 From my point of view two tests are enough, I have also tested on Intel Mac, Apple Silicon Mac and Windows 11 WSL 2, which in the end is also simply Ubuntu. But if someone wants to retest with Windows 11 WSL 2, welcome ...
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-05-29 14:25:47 |
Closed_By | ⇒ | HLeithner |
thanks
@tecpromotion may you test with Mac?