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 322907 - Modified settings should be ignored after a Error message in Pilot Settings->Device Settings
Modified settings should be ignored after a Error message in Pilot Settings->...
Status: VERIFIED FIXED
Product: gnome-pilot
Classification: Other
Component: capplet
2.0.x
Other Solaris
: High major
: ---
Assigned To: Jerry Yu
gnome-pilot Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-12-01 09:21 UTC by Jerry Yu
Modified: 2006-03-10 09:33 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed patch (4.20 KB, patch)
2005-12-01 09:30 UTC, Jerry Yu
none Details | Review
alternative patch: don't close the dialog until the params are ok (1.08 KB, patch)
2006-01-12 08:07 UTC, Matt Davey
none Details | Review

Description Jerry Yu 2005-12-01 09:21:13 UTC
Please describe the problem:
Using Gnome-Pilot pkg for Solaris USB port on x86 platform.

When select Type as Serial in Pilot Settings -> Device Settings , a Error
message will display as 

" Read/Write permissions failed on 
Cradle(the Port you select, such as /dev/pilot)
Check the permissions on the device and retry ".

Then the modified settings are saved all the same in Device Settings,although it
shouldn't be stored.

Steps to reproduce:



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:30:11 UTC
Created attachment 55465 [details] [review]
proposed patch

It works for this bug on Solaris platform
Comment 2 Matt Davey 2006-01-12 08:06:53 UTC
I think there's a simpler patch (next attachment), that accomplishes the
same thing: just don't close the dialog until the parameter checking passes.

However, on reflection there may be a problem with this approach:
it prevent users from configuring a valid device that doesn't
exist at configuration time.  I can't test this, but I suspect
this could happen if a serial device is using a USB->serial converter.
Comments?
Comment 3 Matt Davey 2006-01-12 08:07:43 UTC
Created attachment 57201 [details] [review]
alternative patch: don't close the dialog until the params are ok
Comment 4 Matt Davey 2006-02-02 09:42:16 UTC
Thanks for the work on this one.  I've committed a patch to CVS.

(the actual patch commit was the bugfixed version of the patch in comment #3.  See also #328395)
Comment 5 Jerry Yu 2006-02-10 08:23:24 UTC
Hi,Matt

Can the bug be marked as resolved, since you have committed your patch for this bug. 

Thanks,
Jerry
Comment 6 Jerry Yu 2006-03-10 08:52:17 UTC
The bug can't be reproducible. I verified it with pilot-link-0.12.0-pre + gnome-pilot CVS.
Comment 7 Matt Davey 2006-03-10 09:33:09 UTC
ok, thanks.  I'll mark as verified.