[krusader] [Bug 401064] Temp files created instead open real files when comparing file content differences

2020-05-15 Thread Toni Asensi Esteve
https://bugs.kde.org/show_bug.cgi?id=401064

--- Comment #6 from Toni Asensi Esteve  ---
Git commit d8a5556e65ab376f3096859932d6956c0f471675 by Toni Asensi Esteve, on
behalf of Alexander Bikadorov.
Committed on 15/05/2020 at 23:47.
Pushed by asensi into branch 'master'.

Add Kompare as compare application with KIO support

It can use the original KIO URLs. No need for using temporary local files.
And remove the obsolete space character check for file paths for KDiff3.

Differential Revision: https://phabricator.kde.org/D24987

M  +3-3krusader/krslots.cpp

https://commits.kde.org/krusader/d8a5556e65ab376f3096859932d6956c0f471675

-- 
You are receiving this mail because:
You are watching all bug changes.

[krusader] [Bug 401064] Temp files created instead open real files when comparing file content differences

2019-10-27 Thread Rafael Linux User
https://bugs.kde.org/show_bug.cgi?id=401064

--- Comment #5 from Rafael Linux User  ---
Thank you @Alex. I understand your situation. If you read  the thread I posted,
you will understand why I'm more frustated each day with KDE (Plasma) and
Gnome. Instead of work togheter to solve issues, they let developers and users
to fight with system base issues at their own. That is not the best way to get
more users to use Linux desktops. Very sad.  :(

-- 
You are receiving this mail because:
You are watching all bug changes.

[krusader] [Bug 401064] Temp files created instead open real files when comparing file content differences

2019-10-27 Thread Alex Bikadorov
https://bugs.kde.org/show_bug.cgi?id=401064

Alex Bikadorov  changed:

   What|Removed |Added

 Ever confirmed|0   |1
   Severity|normal  |wishlist
 Status|REPORTED|CONFIRMED

--- Comment #4 from Alex Bikadorov  ---
Kompare will be added as application supporting KIO
(https://phabricator.kde.org/D24987).

For the write support for non-KIO applications KioFuse
(https://techbase.kde.org/Projects/KioFuse) would be -as you pointed out- the
general solution. So to be honest i don't think somebody will implement this as
new feature in Krusader.

-- 
You are receiving this mail because:
You are watching all bug changes.

[krusader] [Bug 401064] Temp files created instead open real files when comparing file content differences

2019-10-21 Thread Rafael Linux User
https://bugs.kde.org/show_bug.cgi?id=401064

Rafael Linux User  changed:

   What|Removed |Added

 CC||rafael.linux.u...@gmail.com

--- Comment #3 from Rafael Linux User  ---
I understand that indeed, is not a bug of Krusader, but of Plasma. I reported
and discussed about it several times in some threads like this 
https://bugs.kde.org/show_bug.cgi?id=75324 
It's October now, so ... it's supposed it should be solved but, is it?? I
understand that is not, cause the linked thread is not closed. I'm bored to
claim for a fix for a bug that affect to all work depending on KIO, including
Krusader.

Thank you anyway for your consideration

-- 
You are receiving this mail because:
You are watching all bug changes.

[krusader] [Bug 401064] Temp files created instead open real files when comparing file content differences

2019-10-20 Thread Alex Bikadorov
https://bugs.kde.org/show_bug.cgi?id=401064

Alex Bikadorov  changed:

   What|Removed |Added

 CC||alex.bikado...@kdemail.net

--- Comment #2 from Alex Bikadorov  ---
Kompare is supporting KIO protocols. So simply doing the same "whitelisting" as
for kdiff would make it work as well. (Done in KRslots::compareContent() i
think).

But Meld (actually also my favourite diff app) is GTK based and adding write
support for it (and other diff tools) is complicated as Davide wrote.

-- 
You are receiving this mail because:
You are watching all bug changes.

[krusader] [Bug 401064] Temp files created instead open real files when comparing file content differences

2019-07-12 Thread Davide Gianforte
https://bugs.kde.org/show_bug.cgi?id=401064

Davide Gianforte  changed:

   What|Removed |Added

 CC||dav...@gengisdave.org

--- Comment #1 from Davide Gianforte  ---
The code confirm your analysis (KRslots::compareContent(const QUrl&, const
QUrl&)). Only Kdiff works on remote files - and their url must not have spaces.

I thought two ways to solve this situation:

- we can let the diff utility to handle files with the original path, anyway
any password would be cleared and you'll have to insert it again; less code to
write, any bugs should be opened to the util devs;

- whenever the local file changes are saved, Krusader will ask to copy it on
the original location; this will require to write more code;

I'll investigate and make some test for the first solution.

-- 
You are receiving this mail because:
You are watching all bug changes.