cli update failes on windows with message "error:cannot move ..."

[Speed4Trade-Admin]

When I try to update wp-cli.phar on windows, I get an error message that the temp file cannot be moved.

Command:
php.exe "C:\Program Files\wp\wp-cli.phar" cli update

Output:

You have version 2.8.0. Would you like to update to 2.8.1? [y/n] y
Downloading from https://github.com/wp-cli/wp-cli/releases/download/v2.8.1/wp-cli-2.8.1.phar...
md5 hash verified: a0cf91756fd46903c83c436ec1d462eb
New version works. Proceeding to replace.
Error: Cannot move C:\Users\xxx\AppData\Local\Temp\3/wp_647daad3256960.42338796.phar to C:\Program Files\wp\wp-cli.phar

The slash seems suspicious on windows and might be a possible reason (?)

Permissions are ok, I can move the file manually on cmd if I replace the / with \.
On the same console session, the command "cli update" fails however.

[Speed4Trade-Admin]

The slash doesn't matter.
I tested with a test.php and renamed the file successfully.
Not sure if a handle prevents the renaming or the cli can't replace itself (?)

[danielbachhuber]

Thanks for the report, @Speed4Trade-Admin

The slash doesn't matter.
I tested with a test.php and renamed the file successfully.

Can you share what commands you ran here? The slash is suspicious to me too. Here's where it's added:

wp-cli/php/utils.php

Line 1076 in ad16e53

$temp = trailingslashit( sys_get_temp_dir() );

wp-cli/php/utils.php

Lines 1008 to 1023 in ad16e53

	/**
	* Appends a trailing slash.
	*
	* @access public
	* @category System
	*
	* @param string $string What to add the trailing slash to.
	* @return string String with trailing slash added.
	*/
	function trailingslashit( $string ) {
	if ( ! is_string( $string ) ) {
	return '/';
	}
	return rtrim( $string, '/\\' ) . '/';
	}