After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 320869 - xfer fails silently
xfer fails silently
Status: RESOLVED FIXED
Product: gnome-vfs
Classification: Deprecated
Component: File operations
cvs (head)
Other Linux
: Normal major
: ---
Assigned To: gnome-vfs maintainers
gnome-vfs maintainers
Depends on:
Blocks:
 
 
Reported: 2005-11-07 12:12 UTC by Paolo Borelli
Modified: 2005-11-08 12:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (918 bytes, patch)
2005-11-07 12:16 UTC, Paolo Borelli
none Details | Review

Description Paolo Borelli 2005-11-07 12:12:52 UTC
among the others, try the following:

gnomevfs-copy README ftp://ftp.gnome.org/robots.txt
gnomevfs-copy /etc/fstab ssh://paolo:xxxx@localhost/foo.txt (where /foo.txt,
already exists)

ditto for http and others: these operations obviously fail because we don't have
write permissions, but xfer returns success.

This is a real major problem for gedit vfs support because means that we cannot
reliably tell if a file was saved successfully ot not
Comment 1 Paolo Borelli 2005-11-07 12:16:02 UTC
Created attachment 54412 [details] [review]
patch

this patch fixes the issue for us... it's a quick patch, and the comment makes
me wonder if there was a rationale behind the behavior. Or maybe the comment is
just on crack...

Another option is not to assing result = VFS_OK even when handle_error returns
true, since we pass it a &result and thus it should set the result to the
proper value on its own
Comment 2 Alexander Larsson 2005-11-08 12:07:38 UTC
I tried to track down the reason for the initial comment, but couldn't figure it
out. The patch looks right to me, so I'm commiting it to HEAD. But i'm always
wary about changes to gnome_vfs_xfer, so we should keep a lookout for problems
caused by this change.