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

UpuConsole: -o ignored if directory already exists #3

Open
russt opened this issue Dec 9, 2014 · 0 comments
Open

UpuConsole: -o ignored if directory already exists #3

russt opened this issue Dec 9, 2014 · 0 comments

Comments

@russt
Copy link

russt commented Dec 9, 2014

This works okay:

$ mono /scm/p4proj/main/p4connect/UPU/UpuConsole.exe -o /tmp/foodir -i dist/P4Connect.unitypackage
Extracting dist/P4Connect.unitypackage to /tmp/foodir
Decompressed: P4Connect.unitypackage
Extracting /var/folders/lZ/lZCSnMPVHt4tXtOYjErkFE+++TI/-Tmp-/Upu/iyvrrmdy.ejr/P4Connect to /var/folders/lZ/lZCSnMPVHt4tXtOYjErkFE+++TI/-Tmp-/Upu/iyvrrmdy.ejr/content...
Creating directory /tmp/foodir/Assets/P4Connect/Editor...
Extracting file /tmp/foodir/Assets/P4Connect/Editor/libp4bridge.dylib...
Extracting file /tmp/foodir/Assets/P4Connect/Editor/PackageIcons.cs...
Extracting file /tmp/foodir/Assets/P4Connect/readme.txt...
Extracting file /tmp/foodir/Assets/P4Connect/Editor/Icons.pack...

However, if output directory already exists, UpuConsole confusingly ignores the -o argument, and unpacks to the default location with an incremented unpack count:

$ mono /scm/p4proj/main/p4connect/UPU/UpuConsole.exe -o /tmp/foodir -i dist/P4Connect.unitypackage
Extracting dist/P4Connect.unitypackage to /Volumes/tanami-ufs/p4proj/main/p4connect/dist/P4Connect.unitypackage_unpacked (2)
Decompressed: P4Connect.unitypackage
Extracting /var/folders/lZ/lZCSnMPVHt4tXtOYjErkFE+++TI/-Tmp-/Upu/sg7t3k6x.xky/P4Connect to /var/folders/lZ/lZCSnMPVHt4tXtOYjErkFE+++TI/-Tmp-/Upu/sg7t3k6x.xky/content...
Creating directory /Volumes/tanami-ufs/p4proj/main/p4connect/dist/P4Connect.unitypackage_unpacked (2)/Assets/P4Connect/Editor...
Extracting file /Volumes/tanami-ufs/p4proj/main/p4connect/dist/P4Connect.unitypackage_unpacked (2)/Assets/P4Connect/Editor/libp4bridge.dylib...
Extracting file /Volumes/tanami-ufs/p4proj/main/p4connect/dist/P4Connect.unitypackage_unpacked (2)/Assets/P4Connect/Editor/PackageIcons.cs...
Extracting file /Volumes/tanami-ufs/p4proj/main/p4connect/dist/P4Connect.unitypackage_unpacked (2)/Assets/P4Connect/readme.txt...
Extracting file /Volumes/tanami-ufs/p4proj/main/p4connect/dist/P4Connect.unitypackage_unpacked (2)/Assets/P4Connect/Editor/Icons.pack...

Would probably be better to unpack to:

/tmp/foodir (2)

to be consistent with the behaviour when the -o option is not specified.

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

No branches or pull requests

1 participant