RTC PR-5.3-dev Pending

User tests: Successful: 0 Unsuccessful: 0

avatar brianteeman
brianteeman
18 Mar 2025

During the build process of a new release or update we remove various files that should not or do not need to be distributed

This PR extends the list of files to be removed from the libraries/vendor folder to include

.drone.yml
.drone.jsonnet

Examples can be seen in most of the libraries/vendor/joomla folders

@richard67 I assume that this will need to be added to the removed files list

avatar brianteeman brianteeman - open - 18 Mar 2025
avatar brianteeman brianteeman - change - 18 Mar 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Mar 2025
Category Repository
avatar exlemor
exlemor - comment - 18 Mar 2025

Hi @brianteeman, is there perhaps a missing label for this PR, I just tried testing it and Joomla Patch Tester said:

There are no files to patch from this pull request. This may mean that the files in the pull request are not present in your installation.

and the .drone.yml, .drone.jsonnet files were still present in the sub-folders..

(which led me to think, it perhaps should require a Download - Prebuilt package?)

avatar brianteeman
brianteeman - comment - 18 Mar 2025

@exlemor as stated in the PR this is in the script to build a release. it is not in the release of joomla

avatar brianteeman brianteeman - change - 18 Mar 2025
The description was changed
avatar brianteeman brianteeman - edited - 18 Mar 2025
avatar fgsw
fgsw - comment - 18 Mar 2025

@exlemor Look in tab "Files changed", if the file is not in path "/administrator/..." its not able to test by patchtester, prebuilt packages.

avatar exlemor
exlemor - comment - 18 Mar 2025

@exlemor Look in tab "Files changed", if the file is not in path "/administrator/..." its not able to test by patchtester, prebuilt packages.

Probably obvious but not something I knew, great insight - thanks @fgsw

avatar richard67
richard67 - comment - 18 Mar 2025

@richard67 I assume that this will need to be added to the removed files list

@brianteeman Correct. I will handle that regularly and so add them when this PR has been merged.

avatar brianteeman
brianteeman - comment - 18 Mar 2025

@exlemor Look in tab "Files changed", if the file is not in path "/administrator/..." its not able to test by patchtester, prebuilt packages.

Probably obvious but not something I knew, great insight - thanks @fgsw

not true as obviously changes in /plugins/ for example can be tested

avatar brianteeman
brianteeman - comment - 18 Mar 2025

@richard67 I assume that this will need to be added to the removed files list

@brianteeman Correct. I will handle that regularly and so add them when this PR has been merged.

thanks

avatar richard67
richard67 - comment - 18 Mar 2025

@exlemor Testing instructions: Download the full installation zip package created by Drone for this PR, and download a recent 5.3 nightly build, and compare the content of the 2 zip files regarding files which only exist in the nightly build package but not in the one for this PR.

Package for this PR: https://artifacts.joomla.org/drone/joomla/joomla-cms/5.3-dev/45152/downloads/83435/Joomla_5.3.0-beta3-dev+pr.45152-Development-Full_Package.zip

Nightly build: https://developer.joomla.org/nightlies/Joomla_5.3.0-beta3-dev-Development-Full_Package.zip

Result:

comparison

The comparison tool (Beyond Compare) shows the package of this PR on the left hand side and the package from the nightly build at the right hand side. It is adjusted to only show singles on the right hand side, i.e. only the files which exists on the right hand side but not at the left hand side. Other differences are not shown in this mode.

As you can see, the PR does what it claims to do.

avatar richard67 richard67 - test_item - 18 Mar 2025 - Tested successfully
avatar richard67
richard67 - comment - 18 Mar 2025

I have tested this item ✅ successfully on da9b83f

Details about how I've tested see my previous comment.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45152.

avatar exlemor exlemor - test_item - 19 Mar 2025 - Tested successfully
avatar exlemor
exlemor - comment - 19 Mar 2025

I have tested this item ✅ successfully on da9b83f

I was able to successfully test this (thanks to @richard67's help).

I used an alternative to BeyondCompare.

ScreenFloat Shot of ScreenFloat at 19 Mar 2025 at 05_53_59
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45152.
avatar QuyTon QuyTon - change - 19 Mar 2025
Status Pending Ready to Commit
Labels Added: PR-5.3-dev
avatar QuyTon
QuyTon - comment - 19 Mar 2025

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45152.

avatar QuyTon QuyTon - change - 19 Mar 2025
Labels Added: RTC
avatar laoneo laoneo - change - 19 Mar 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-03-19 06:37:23
Closed_By laoneo
avatar laoneo laoneo - close - 19 Mar 2025
avatar laoneo laoneo - merge - 19 Mar 2025
avatar laoneo
laoneo - comment - 19 Mar 2025

Thanks!

avatar brianteeman
brianteeman - comment - 19 Mar 2025

Thanks

Add a Comment

Login with GitHub to post a comment