Success

User tests: Successful: Unsuccessful:

avatar richard67
richard67
19 Oct 2019

Pull Request for Issue #240 .

Summary of Changes

Use DIRECTORY_SEPARATOR for expode and implode of the path to files to be backed up so that later creating the directories for the backups doesn't fail.

I don't know if this is the right way to fix it, but it works here.

Testing Instructions

See issue #240 .

avatar richard67 richard67 - open - 19 Oct 2019
avatar richard67
richard67 - comment - 20 Oct 2019

@Hackwar If you don't like DIRECTORY_SEPARATOR we might have to use '/'? If so, let me know here and I'll change this PR.

P.S.: Unfortunately I don't have a Windows test system. Someone should test if '/' or DIRECTORY_SEPARATOR works on Windows, too.

avatar richard67
richard67 - comment - 22 Oct 2019

Sorry, my previous post was in German first by accident. Have changed it to Englisdh yesterday. Unfortunately still no reaction. So the Patchtester 4 with CI enabled is not usable on Linux or other Unixoids. What a pity.

avatar Hackwar Hackwar - close - 24 Oct 2019
avatar Hackwar Hackwar - merge - 24 Oct 2019
avatar Hackwar Hackwar - reference | a2b354a - 24 Oct 19
avatar Hackwar Hackwar - merge - 24 Oct 2019
avatar Hackwar Hackwar - change - 24 Oct 2019
Status New Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-10-24 11:55:25
Closed_By Hackwar
avatar Hackwar Hackwar - close - 24 Oct 2019
avatar richard67 richard67 - head_ref_deleted - 27 Nov 2019

Add a Comment

Login with GitHub to post a comment