winter icon indicating copy to clipboard operation
winter copied to clipboard

Mix:watch webpack config not being cleaned - Windows OS

Open damsfx opened this issue 1 year ago • 1 comments

Winter CMS Build

dev-develop

PHP Version

8.1

Database engine

MySQL/MariaDB

Plugins installed

No response

Issue description

When running the command php artisan mix:watch on Windows OS native terminal, the halt signal is not well handle and the mix.webpack.js is not automatically cleaned up.

The getSubscribedSignals method of the Winter's trait is never fired on Windows.

My investigations : In the command process execution the Symfony console in his constructor set a signalsToDispatchEvent private variable if the SignalRegistry::isSupported() test is successful. Application.php#L99-L102

The SignaRegistry::isSupported() method perform a test for PHP pcntl_signal support. SignalRegistry.php#L40-L43

The PHP PCNTL documentation says :

Currently, this module will not function on non-Unix platforms (Windows).

So ... No way to get halting signals on Windows OS while running Symfony console v6.2.xxx

In the next Synfony console version (6.3.xxx) a new architecture was introduced to better handle exit signals from command that works with Winter's commands.

But for now the 6.3.x Symfony console can't be installed due to Winter.Storm requirements : "symfony/console": ">=6.0.9 <6.3.0",

Steps to replicate

Run a php artisan mix:watch ... command and press CONTROL+C to escape the run.

Workaround

Use WSL on Windows OS .

damsfx avatar Jul 31 '24 14:07 damsfx

We'll have to address this in 1.3 as Symofny/Console 6.3 doesn't support PHP 8.0 so we can't switch to it yet on 1.2: https://github.com/wintercms/storm/commit/facb27d7486406306dab491ed3d9eb95245fd0b2

LukeTowers avatar Aug 01 '24 06:08 LukeTowers

This should be resolved now.

LukeTowers avatar Jun 02 '25 04:06 LukeTowers