satanas / Turpial

Twitter client written in Python. Light, fast, fully functional and integrated to the user desktop
http://turpial.org.ve
GNU General Public License v3.0
174 stars 47 forks source link

Delete config file if it's no valid #269

Open ruiztulio opened 10 years ago

ruiztulio commented 10 years ago

Suddendly appear a message indicating that there was no connection to tne internet and an option to retry, that didn't work. Ran it with -d option, but didn't show any usefull information, first tried to reinstall then my fix was to remove all tupial config files in my home.

Running: lsb_release -a Distributor ID: ManjaroLinux Description: Manjaro Linux Release: 0.8.8 Codename: Ascella

uname -a Linux 3.12.7-1-MANJARO #1 SMP PREEMPT Fri Jan 10 17:28:06 UTC 2014 x86_64 GNU/Linux

Sorry if I can't give you more usefull information about this.

Richzendy commented 10 years ago

I report this issue too with libturpial 1.6.0 over Fedora 19 x86_64

satanas commented 10 years ago

@Richzendy Did you try deleting ~/.config/turpial/config?

Richzendy commented 10 years ago

deleting ~/.config/turpial/config turpial runs fine, this is the content of file config:

[Notifications] login = on updates = on icon = on actions = on

[Window] position = -1,-1 state = windowed visibility = show size = 1600,852

[Proxy] username = password = port = server =

[Services] shorten-url = is.gd upload-pic = twitpic

[Sounds] login = on updates = on

[Browser] cmd =

[Advanced] socket-timeout = 20 show-user-avatars = on

[General] update-interval = 5 profile-color = on statuses = 60 minimize-on-close = on queue-interval = 30

[Columns] column4 = Richzendy-twitter-favorites column1 = Richzendy-twitter-timeline column3 = Richzendy-twitter-directs column2 = Richzendy-twitter-replies

This problem looks very important, if an user update libturpial and turpial don't run, I do not think putting a Fedora version 1.6.0 of libturpial if users can have this problem.

which steps you think might be next?

satanas commented 10 years ago

Maybe I could try to check if config file is valid and delete it otherwise. This way user doesn't need to perform this action manually

Richzendy commented 10 years ago

this seems better

satanas commented 10 years ago

Great! Then I should fix that issue and add lot of tests. I don't want to release another buggy version

ruiztulio commented 10 years ago

+1

satanas commented 10 years ago

Renamed issue for a better localization