Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"--copy" does not work on Windows #140

Closed
ThomasWeinert opened this issue May 5, 2018 · 5 comments
Closed

"--copy" does not work on Windows #140

ThomasWeinert opened this issue May 5, 2018 · 5 comments
Assignees
Labels
Milestone

Comments

@ThomasWeinert
Copy link
Contributor

"-c/--copy" copies the Phar into the tools directory, but without the file extension. The tools can not be called directly that way - only with "php tools\toolname". The BatPharActivator should be called to create a matching batch file.

Why is the copy flag stored in the phive.xml? It feels more like an environment specific option to me.

@theseer
Copy link
Member

theseer commented May 5, 2018

That indeed sounds like a bug. I thought on windows we always do a copy because windows can't do symlinks properly. But then, I don't use windows.

The flag is stored because it is a decision done on behalf of the project: Quite some people commit the binary of the tools along in their repository - which of course will only work when there is a copy rather than a symlink. And for updates we also want to maintain that state. I prefer to have that in a clean config file rather than having to guess from a file that may or may not exist.

@theseer theseer added the bug label May 5, 2018
@ThomasWeinert
Copy link
Contributor Author

That way copy as a package/project flag makes sense. I misunderstood it. Thanks for the explanation.

@theseer
Copy link
Member

theseer commented Jun 26, 2018

Is this issue still relevant, given #141 has been merged?

@ThomasWeinert
Copy link
Contributor Author

Yes. The activator is not triggered if the flag is set.

@sebastianheuer sebastianheuer self-assigned this Jul 6, 2018
@sebastianheuer sebastianheuer added this to the 0.12.0 milestone Jul 6, 2018
@ThomasWeinert
Copy link
Contributor Author

That looks nice, now copy=true seems to work better then just links (it provides IDE endpoints for the tools).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants