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.

Sign up for GitHub

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

Jump to bottom

Temporary files are left over in the filesystem #80

Closed
jtheoof opened this issue Feb 13, 2021 · 1 comment
Closed

Temporary files are left over in the filesystem #80

jtheoof opened this issue Feb 13, 2021 · 1 comment
Assignees
@jtheoof
Labels
bug Something isn't working

Comments

@jtheoof
Copy link
Owner

jtheoof commented Feb 13, 2021

This issue was brought up by @apiraino

Ways to reproduce:

grim -g "$(slurp)" - | swappy -f -

The temporary file created in (usually) /tmp remains there after swappy exits.
@jtheoof jtheoof self-assigned this Feb 13, 2021
@jtheoof jtheoof added the bug Something isn't working label Feb 13, 2021
jtheoof added a commit that referenced this issue Feb 13, 2021
@jtheoof
fix(application): unlink temp file coming from stdin
c66d933 
Stop mutating `file_str`, use a dedicated `temp_file_str` to store the
location of the temporary file when using swappy with `-f -` option.

Closes #80
jtheoof added a commit that referenced this issue Feb 13, 2021
@jtheoof
fix(application): unlink temp file coming from stdin
fefe3ec 
Stop mutating `file_str`, use a dedicated `temp_file_str` to store the
location of the temporary file when using swappy with `-f -` option.

Closes #80
jtheoof added a commit that referenced this issue Feb 13, 2021
@jtheoof
fix(application): unlink temp file coming from stdin
7ef38db 
Stop mutating `file_str`, use a dedicated `temp_file_str` to store the
location of the temporary file when using swappy with `-f -` option.

Closes #80
jtheoof added a commit that referenced this issue Feb 13, 2021
@jtheoof
fix(application): unlink temp file coming from stdin
f0321df 
Stop mutating `file_str`, use a dedicated `temp_file_str` to store the
location of the temporary file when using swappy with `-f -` option.

Closes #80
@apiraino
Copy link
Contributor

thank you @jtheoof for fixing this!

@jtheoof jtheoof mentioned this issue Feb 17, 2021
Closed
lelgenio pushed a commit to lelgenio/swappy that referenced this issue Feb 21, 2021
@jtheoof
fix(application): unlink temp file coming from stdin
6338752 
Stop mutating `file_str`, use a dedicated `temp_file_str` to store the
location of the temporary file when using swappy with `-f -` option.

Closes jtheoof#80
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jtheoof jtheoof

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jtheoof @apiraino