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
install-nix: Solve #971 USER == NULL #1549
Conversation
lgtm |
Does project flow with squash on merge, - or I need to do squashing/acceptance squashing. |
LGTM. @Anton-Latukha maybe you want to squash it all into a single commit. |
That is what I asked above. Is there is no squash on merge enabled and no workflow to squash on merge in repository. It seems life the natural way to move. It is nature of someone who contribute pull request and waits on acceptance - to better it's pull request. To make fixes, add comments, polish, refactor code and text. With squash on pull request contributor - he needs to squash all commits every time, at every fix, at every improvement, to keep things clean and ready to merge. With squash on merge on maintainer side:
I created a relevant to this topic issue: |
echo "$0: \$USER is not set" >&2 | ||
exit 1 | ||
echo "$0: Environment variable USER is not set" >&2 | ||
USER="$(whoami)" # Taking username from UID |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't that also be readonly?
I was solving the real problem. Most of scripts was in very poor state. And they do not have constants used. And they still do not. I needed to merge at least something to have a progress in master. I offered myself to be a guy who would make a great quality Nix install scrips, and who would always give his time to maintain it. I submitted 4 PRs. I've made a lot of reasoning for them, documentation, demo setups, etc. And nothing got merged. In 8 month. If I can not get a tiniest changes. Changes that solve real world reported and myself experienced cases, like #971. Then, what can I do about that. Now it can be cleaned up. Anyway. Thanks to everybody. |
This addresses #971.
Which also encountered by me:
This is because Docker env not goes through
login
and login shell, so shell not sources login config, so shell not defined that env variables.We touch
USER
only when it is not defined. And as it is shell, anyway it is a copy variable in scope of this script.