-
Notifications
You must be signed in to change notification settings - Fork 46
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
Pharo Launcher cmd line should allow to work in current working directory #659
Comments
@estebanlm What do you think of the proposition? |
Why a global setting? $ pl image create titi --here
My take, as a heavy user of command line with Pharo, is that I don't care, but this has some impact on other decisions:
|
Thank you for the feedback @guillep.
Could be an option too. I thought a global option would be easier for people that do not want to work with a central image repository.
in case of a global option, the current directory would be seen as the default image repository and by so, will be scanned to find folders with images inside. If there is only one, it will also work.
in CLI, this is the case.
In my proposition, there will be no conflict because the global repository will be the local repository. So, I think both options are open. |
By default, Pharo Launcher cmd line uses the standard image repository to store the newly created image when creating an image.
PL CLI could propose a (global) setting to do not use the image repository and by so, use the current working directory to host images.
It means image listing will probably only show one image but image creation and store could happen anywhere.
There is still one open question: do we store the image directly in the current folder or in a folder with the name of the image as Pharo Launcher already do?
I would favor the second option since the creation of an image will also create a
pharo.version
file, a sources file a changes file, ameta-inf.ston
file, a pharo-local folder, etc. Having 2 images at the same place would not be possible.@estebanlm WDYT?
The text was updated successfully, but these errors were encountered: