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

get real home directory #171

Merged
merged 1 commit into from
Apr 5, 2017
Merged

get real home directory #171

merged 1 commit into from
Apr 5, 2017

Conversation

morph027
Copy link
Contributor

@morph027 morph027 commented Feb 2, 2017

We are running workstations with powerbroker active directory integration, where the home directory is somehing like /home/local/$DOMAIN_REALM/$USERNAME, so the scripted version throws errors due to nonexistent default home dirs.

@palob
Copy link
Member

palob commented Feb 2, 2017

But the path is still $HOME, right?

@morph027
Copy link
Contributor Author

morph027 commented Feb 2, 2017

This happens for root (in scripted apt context), see #101

touch: cannot touch '/home//.config/numix-folders': No such file or directory
gtk-update-icon-cache: Cache file created successfully.
gtk-update-icon-cache: Cache file created successfully.
Folder change complete!
./numix-folders: line 321: /home//.config/numix-folders: No such file or directory
chown: cannot access '/home//.config/numix-folders': No such file or directory

@Foggalong Foggalong added the bug label Feb 3, 2017
@Foggalong Foggalong added the merge label Apr 5, 2017
@Foggalong Foggalong merged commit 05c9693 into numixproject:master Apr 5, 2017
@morph027
Copy link
Contributor Author

morph027 commented Apr 6, 2017

❤️ Thanks!

Output now is fine:

...
Unpacking numix-icon-theme-circle (2.0.3+14~201704051631~ubuntu16.04.1) over (2.0.3+13~201611240401~ubuntu16.04.1) ...
...
restoring Numix icons (Style: 6, Color: blue)
gtk-update-icon-cache: Cache file created successfully.
gtk-update-icon-cache: Cache file created successfully.
Folder change complete!

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

Successfully merging this pull request may close these issues.

3 participants