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: void Xsession fail when $HOME is full #1

Conversation

black-desk
Copy link
Contributor

When user home is full,
scripts write to a file in user home will failed.

After redirect stdout and stderr to ERRFILE,
aka $HOME/.xsession-errors,
this script echo a line to that file.

This echo might failed and make this script exit with error.
In that case, the session leader will not able to even start.

Releated: linuxdeepin/developer-center#4685
Releated: linuxdeepin/dde-daemon#526

When user home is full,
scripts write to a file in user home will failed.

After redirect stdout and stderr to ERRFILE,
aka $HOME/.xsession-errors,
this script echo a line to that file.

This echo might failed and make this script exit with error.
In that case, the session leader will not able to even start.

Releated: linuxdeepin/developer-center#4685
Releated: linuxdeepin/dde-daemon#526
Signed-off-by: black-desk <me@black-desk.cn>
@deepin-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: black-desk
Once this PR has been reviewed and has the lgtm label, please assign hudeng-go for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@black-desk black-desk force-pushed the black-desk/dev/fix-fail-when-home-full branch from f29b208 to d9cfc28 Compare June 21, 2023 02:00
@github-actions
Copy link

github-actions bot commented Jun 21, 2023

TAG Bot

TAG: 1%7.7+23-deepin1
EXISTED: no
DISTRIBUTION: stable

Signed-off-by: black-desk <me@black-desk.cn>
@black-desk black-desk force-pushed the black-desk/dev/fix-fail-when-home-full branch from 7e9bfaa to 5b18592 Compare June 21, 2023 02:05
@Zeno-sole
Copy link
Contributor

/integrate

2 similar comments
@Zeno-sole
Copy link
Contributor

/integrate

@Zeno-sole
Copy link
Contributor

/integrate

@github-actions
Copy link

AutoIntegrationPr Bot
auto integrate with pr url: deepin-community/Repository-Integration#205
PrNumber: 205
PrBranch: auto-integration-5329764117

@hudeng-go
Copy link

/lgtm

@Zeno-sole Zeno-sole merged commit 1ada546 into deepin-community:master Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants