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 322906 - Setting an inexistent backup directory in Pilot Settings->Conduit Settings cause backup failed
Setting an inexistent backup directory in Pilot Settings->Conduit Settings ca...
Status: RESOLVED FIXED
Product: gnome-pilot
Classification: Other
Component: conduit: backup
2.0.x
Other Solaris
: High major
: ---
Assigned To: Jerry Yu
gnome-pilot Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-12-01 09:11 UTC by Jerry Yu
Modified: 2006-03-13 22:41 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed patch (5.14 KB, patch)
2005-12-01 09:18 UTC, Jerry Yu
none Details | Review
proposed patch based on gnome-pilot CVS (5.09 KB, patch)
2006-03-09 08:25 UTC, Jerry Yu
none Details | Review

Description Jerry Yu 2005-12-01 09:11:24 UTC
Please describe the problem:
Edit Backup directory in Evolution->Tools->Pilot Settings -> Conduit Settings,
change the value from default one to an inexistent directory.
Press HotSync button on Palm, Backup failed and an error message displayed as
"backup conduit has no usable backupdir " when the background program run.

Steps to reproduce:
1.Startup "Email and Calendar" in Launch Menu.
2.In evolution,select Top Menu->Tools->Pilot Settings ->Conduits
3.Click "Settings..." Button to change the directory value from default one to
an inexistent directory.
4.Press HotSync button on Palm

Actual results:


Expected results:


Does this happen every time?
Yes

Other information:
Version details: gnome-pilot 2.0.13
Distribution/Version: Solaris NV 24
Comment 1 Jerry Yu 2005-12-01 09:18:18 UTC
Created attachment 55463 [details] [review]
proposed patch

It works for this bug on Solaris platform
Comment 2 Jerry Yu 2006-03-09 08:23:45 UTC
Hi,Matt

This bug isn't resolved yet. I rewrite a patch attached based on gnome-pilot CVS . It's similar with the previous patch and works well on my SUSE platform.  

The patch's idea is as follow:
Check the backup directory if it exists or it can be made after a directory name is set. If FALSE is returned, a error dialog will appear to remind the user.

Comments are truly appreciated.

-Jerry
Comment 3 Jerry Yu 2006-03-09 08:25:40 UTC
Created attachment 60949 [details] [review]
proposed patch based on gnome-pilot CVS

It works well on Solaris and Linux such as SUSE.
Comment 4 Jerry Yu 2006-03-09 08:29:03 UTC
The bug can be reproduced by setting the backup directory to an inexist one such as "/home/jerry/pilot/backup", where "/home/jerry" is the user home directory and "pilot" is an inexistent directory.
Comment 5 Matt Davey 2006-03-13 22:41:24 UTC
Thanks for this patch.  It's been applied to CVS.
I would have preferred to have used the behaviour as per
the capplet's own pilot dialog and device dialog, but it's
awkward because the conduit API doesn't seem to have a method
of checking the conduit config before applying changes.