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

Add some small advise about module's update #2253

Merged
merged 1 commit into from Aug 10, 2018
Merged

Add some small advise about module's update #2253

merged 1 commit into from Aug 10, 2018

Conversation

uzluisf
Copy link
Collaborator

@uzluisf uzluisf commented Aug 10, 2018

The problem

After struggling for half an hour with a change to a local module that wasn't being reflected,
I realized that I needed to uninstall and install the local module again for the change to take place.

If this is mentioned somewhere else, this can be disregarded.

Solution provided

Point out that one must uninstall and install a given local module for the latest changes to take place.

This seems pretty obvious (after you know it) but I learned it
the hard way after struggling for half an hour.
@zoffixznet zoffixznet merged commit 3de8620 into Raku:master Aug 10, 2018
@zoffixznet
Copy link
Contributor

👍 Thanks!

@ugexe
Copy link
Contributor

ugexe commented Aug 10, 2018

This isn't entirely true. You can force install ( part of the CompUnit::Repository api ), change the auth, or bump the version or api.

I would go one step further and say if you are doing development on a module such that you are changing the code without getting around to bumping the version that you should just be using -I. ( or -Ilib ) which would put the targeted repo in front of the other installation repos.

@zoffixznet
Copy link
Contributor

After struggling for half an hour with a change to a local module that wasn't being reflected,

What was the problem, actually? Doesn't zef say "already installed" and refuses to install again?

@uzluisf
Copy link
Collaborator Author

uzluisf commented Aug 10, 2018

@ugexe Thanks for the advice. I'll look into the CompUnit::Repository api.

@zoffixznet
It was more of a problem from my lack of knowledge. The thing is that I installed the local module, made some changes to it afterward but when I proceeded to use it, none of those changes were reflected. I decided to uninstall and then install the module again with the latest changes. However, that doesn't seem to be the most appropriate course of actions as per @ugexe's suggestion.

zoffixznet added a commit that referenced this pull request Aug 11, 2018
To address concerns on #2253
@zoffixznet
Copy link
Contributor

I reworded the text to address both ugexes comments and to reduce the chances of the problem that happened to you happening to users.

@uzluisf
Copy link
Collaborator Author

uzluisf commented Aug 11, 2018

That addresses my concern. Thanks!

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

Successfully merging this pull request may close these issues.

None yet

3 participants