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

package command -path option requires final foreslash #605

Closed
janihur opened this issue Oct 30, 2024 · 2 comments
Closed

package command -path option requires final foreslash #605

janihur opened this issue Oct 30, 2024 · 2 comments
Assignees
Labels
bug Something isn't working customer

Comments

@janihur
Copy link

janihur commented Oct 30, 2024

package command -path option requires final foreslash / (in Linux). This is not mentioned anywhere. The expected behaviour is both:

  • /home/irisowner (without a directory separator in the end)
  • /home/irisowner/ (with a directory separator in the end)

should work in the identical way.

zpm:IPMTEST1>version

%SYS>     zpm          0.7.3
IPMTEST1> zpm-registry 1.3.2
https://pm.community.intersystems.com - 1.0.6
http://localhost:52773/registry/ - 1.3.2

zpm:IPMTEST1>package -verbose -only -path /home/irisowner osex-ipm-hello

[IPMTEST1|osex-ipm-hello]       Package START
[osex-ipm-hello]        Package FAILURE
ERROR! Error creating directory chain /home/irisownersrc/OSEX/ipm/hello/: <4294967283> Unknown error -13
zpm:IPMTEST1>
@isc-tleavitt
Copy link
Contributor

This is an easy one, just need to use ##class(%Library.File).NormalizeDirectory.

@janihur
Copy link
Author

janihur commented Jan 8, 2025

I verified this is fixed in v0.9.1 and probably already earlier.

@janihur janihur closed this as completed Jan 8, 2025
@github-project-automation github-project-automation bot moved this from To do to Done in v1.0 targets Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working customer
Projects
Status: Done
Development

No branches or pull requests

3 participants