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

Fix user home detection #141

Closed
1 task done
monsieurchico opened this issue Jul 29, 2014 · 3 comments
Closed
1 task done

Fix user home detection #141

monsieurchico opened this issue Jul 29, 2014 · 3 comments
Assignees
Labels

Comments

@monsieurchico
Copy link
Contributor

When the install is itself included in another script, the user home detection fails and we lose the user information.
We have to use logname instead.
The same problem comes when we want to clean.

  • Change user home detection

Add "sudo" in front of clean command (in install_executable function)
Add "sudo" in front of link command (in install_executable function)

monsieurchico added a commit that referenced this issue Jul 29, 2014
monsieurchico added a commit that referenced this issue Jul 29, 2014
@monsieurchico
Copy link
Contributor Author

Bon après investigation, le problème du sudo venait d'ailleurs. je rollback

monsieurchico added a commit that referenced this issue Jul 29, 2014
monsieurchico added a commit that referenced this issue Jul 29, 2014
@geoffroy-aubry
Copy link
Member

Le détecter avant git push eût été l'idéal ;-)

@monsieurchico
Copy link
Contributor Author

c'est pour ça qu'on utilise des branches, pour rollback easy !

Romain DEROCLE
rderocle@gmail.com
tel. 06 75 28 74 57

On Tue, Jul 29, 2014 at 4:33 PM, Geoffroy Aubry notifications@github.com
wrote:

Le détecter avant git push eût été l'idéal ;-)


Reply to this email directly or view it on GitHub
#141 (comment).

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

2 participants