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

Override Environment.SpecialFolder.Personal with $HOME #1861

Merged
merged 1 commit into from
Jun 8, 2015

Conversation

saper
Copy link
Contributor

@saper saper commented Jun 7, 2015

Any port that writes to Environment.SpecialFolder.Personal during build
or when running tests may end up with files outside of stage area that
aren't cleaned up by the ports framework. The issue lies with Mono
looking in /etc/passwd first and only if no entry found there in HOME.

Patch based on discussion in

#371

Obtained from: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=193426

Any port that writes to Environment.SpecialFolder.Personal during build
or when running tests may end up with files outside of stage area that
aren't cleaned up by the ports framework. The issue lies with Mono
looking in /etc/passwd first and only if no entry found there in HOME.

Patch based on discussion in

mono#371

Obtained from: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=193426
@kumpera
Copy link
Contributor

kumpera commented Jun 8, 2015

After looking at the history of gmisc-unix (and gpath before that), plus the fact that well known env vars should override system defaults, I believe this to be the right behavior.

kumpera added a commit that referenced this pull request Jun 8, 2015
Override Environment.SpecialFolder.Personal with $HOME
@kumpera kumpera merged commit aac7aed into mono:master Jun 8, 2015
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Override Environment.SpecialFolder.Personal with $HOME

Commit migrated from mono/mono@aac7aed
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