Skip to content
This repository was archived by the owner on Aug 15, 2024. It is now read-only.

Fixes #33#34

Closed
lavturo wants to merge 1 commit intophp:masterfrom
lavturo:fix_issue33
Closed

Fixes #33#34
lavturo wants to merge 1 commit intophp:masterfrom
lavturo:fix_issue33

Conversation

@lavturo
Copy link
Contributor

@lavturo lavturo commented May 28, 2019

@weltling This is a very small change, but it seems that when we SET PFTT_RELEASE = %PFTT_HOME%\pftt_release in the middle of the code instead of at the beginning, it doesn't set it right away. Since it isn't set right away, it refers to \bin instead of %PFTT_RELEASE%\bin.

This will make the script pack an empty dir instead of the required directories to make pftt run with minimal files.

@cmb69
Copy link
Member

cmb69 commented May 28, 2019

Yep, that fix makes sense. com.exe evaluates all variables in a parenthesized if body before processing each line inside. Another option would be to use enabledelayedexpansion and ! instead of %, but I'd prefer doing it your way.

@php-pulls
Copy link

Comment on behalf of cmb at php.net:

Applied as bd072f0.

@php-pulls php-pulls closed this May 28, 2019
@lavturo lavturo deleted the fix_issue33 branch October 23, 2019 18:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants