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

private-tmp in meld breaks diff view #3551

Closed
bbhtt opened this issue Jul 26, 2020 · 6 comments
Closed

private-tmp in meld breaks diff view #3551

bbhtt opened this issue Jul 26, 2020 · 6 comments

Comments

@bbhtt
Copy link
Contributor

bbhtt commented Jul 26, 2020

Firejail 0.9.62,0.9.63
Meld 3.20.2

System:Kernel: 5.4.0-42-generic x86_64 bits: 64 compiler: gcc v: 9.3.0 Desktop: Xfce 4.14.2 
Distro: Ubuntu 20.04.1 LTS  

Seems like files are sourced from /tmp and private-tmp is creating problems in the diff view: https://imgur.com/SlIahe8. Not sure if this has something to with CLI/frontends using git. Other diff viewers seem to work.The repo is cloned in /home/git/GitHub/.
Maybe a comment should be written above it.

@rusty-snake
Copy link
Collaborator

With diffview you mean what? Which command or menu?

@bbhtt
Copy link
Contributor Author

bbhtt commented Jul 26, 2020

With diffview you mean what? Which command or menu?

The comparison view you get when you change a file.

@rusty-snake
Copy link
Collaborator

For me it works, can you give more detailed steps.

@bbhtt
Copy link
Contributor Author

bbhtt commented Jul 26, 2020

It works in CLI I think. Set meld as diff tool in it, open a file in text editor through cola, change something, right click on unstaged chages, view using diff tool.

Commenting private-tmp solves the issue.

@rusty-snake
Copy link
Collaborator

git difftool file_with_changes

@bbhtt
Copy link
Contributor Author

bbhtt commented Jul 26, 2020

Doesn't work without that patch, I had it commented already in another terminal when I said "CLI" works. Thanks,

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 a pull request may close this issue.

2 participants